Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[css-backgrounds] Specify exact pixel values for border-width: thin, medium, thick #7254

Closed
zcorpan opened this issue May 5, 2022 · 3 comments
Labels
Closed Accepted by CSSWG Resolution Commenter Satisfied Commenter has indicated satisfaction with the resolution / edits. css-backgrounds-3 Current Work Tested Memory aid - issue has WPT tests

Comments

@zcorpan
Copy link
Member

zcorpan commented May 5, 2022

See this Twitter thread: https://twitter.com/tabatkins/status/1522261879762014223

Since browsers agree on 1px, 3px and 5px for border-width: thin, medium, and thick, respectively, it seems to me the spec could require this mapping for all UAs.

cc @tabatkins @bfgeek

@tabatkins
Copy link
Member

I suspect the current undefinedness was one of those things where browsers didn't agree when we were nailing down 2.1 and we weren't sure they could agree by our deadline.

@fantasai
Copy link
Collaborator

fantasai commented May 5, 2022

I think it was intentionally left UA-defined, just like the various font-size keywords. My suggestion would be to put “For Web compatibility, it is recommended that these keywords be mapped to 1px, 3px, and 5px, respectively.” or somesuch.

@css-meeting-bot
Copy link
Member

The CSS Working Group just discussed [css-backgrounds] Specify exact pixel values for border-width: thin, medium, thick, and agreed to the following:

  • RESOLVED: thin, medium, and thick will be defined as 1px, 3px, and 5px respectively.
The full IRC log of that discussion <emeyer> Topic: [css-backgrounds] Specify exact pixel values for border-width: thin, medium, thick
<emeyer> Github: https://github.com//issues/7254
<emeyer> TabAtkins: Was noticed that we still leave the keyword thicknesses undefined. But all browsers agree on widths at 1px, 3px, 5px.
<emeyer> …Not sure of the exact history, but absent that, the widths are very consistent and don’t see a good reason to leave them undefined.
<astearns> ack fantasai
<emeyer> fantasai: Maybe it was that these were like font-size keywords and left open to interpretation. We should maybe recommend but not require specific widths.
<fantasai> My suggested wording was “For Web compatibility, it is recommended that these keywords be mapped to 1px, 3px, and 5px, respectively.”
<emeyer> TabAtkins: On the other hand, we specify a lot of things that could be adjusted by e-readers. I don’t know that this needs to be left undefined.
<emeyer> …I argue we should go stronger than suggested.
<astearns> ack dbaron
<emeyer> dbaron: This is one of those things that was left up in the air in CSS1, and I don’t remember a proposal to define more precisely. I support the precise definition.
<emeyer> fantasai: If everyone else agrees, that’s fine.
<emeyer> RESOLVED: thin, medium, and thick will be defined as 1px, 3px, and 5px respectively.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Closed Accepted by CSSWG Resolution Commenter Satisfied Commenter has indicated satisfaction with the resolution / edits. css-backgrounds-3 Current Work Tested Memory aid - issue has WPT tests
Projects
None yet
Development

No branches or pull requests

4 participants