hr normalize applying wrong overflow value

[brady-shinners-bby]

HTML5 spec for hr is to apply overflow: hidden, but normalize is currently applying overflow: visible. Please either remove what appears to be an unneeded override of a now-uniform default style or apply the expected overflow: hidden.

[garrettw]

Are you confirming that most browsers are respecting the spec now, or that the override definitely isn't needed anymore?

[brady-shinners-bby]

The override does the opposite of the spec. I'm not familiar with what normalize's target browsers are, so not sure on whether overflow: visible should be replaced with overflow: hidden or just removed. For my use-case, all browsers we support adhere to overflow: hidden.

[garrettw]

In my normalize in https://github.com/garrettw/vitals I'm making the change to hidden until I know for sure that it isn't needed anymore.