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-ui-3] Should outline-style:auto follow border-radius curve? #4671

Open
BillGoldstein opened this issue Jan 14, 2020 · 2 comments
Open

[css-ui-3] Should outline-style:auto follow border-radius curve? #4671

BillGoldstein opened this issue Jan 14, 2020 · 2 comments

Comments

@BillGoldstein
Copy link

@BillGoldstein BillGoldstein commented Jan 14, 2020

Should outline-style:auto follow border-radius curve?
https://drafts.csswg.org/css-ui-3/#outline-props

If not, should there be spec language that says following border-radius curve should only apply to
non-auto <border-style>?

Note that UAs might choose to apply a (possibly platform-specific) default curve with outline-style:auto

@MatsPalmgren

This comment has been minimized.

Copy link

@MatsPalmgren MatsPalmgren commented Jan 14, 2020

I tend to think it shouldn't. The whole purpose of outline-style:auto is to provide a platform-specific outline. Both Safari and Firefox ignores the specified color and width in this example: data:text/html,<div style="outline:auto red 10px;">hello</div>. Chrome applies the color but not the width. I consider that a bug in Chrome (it shouldn't apply the color if it supports outline-style:auto on the given platform).

(NOTE: Firefox doesn't support outline-style:auto by default yet. You have to toggle the hidden pref layout.css.outline-style-auto.enabled to true to enable it. It will be on by default in Nightly soon though.)

@BillGoldstein

This comment has been minimized.

Copy link
Author

@BillGoldstein BillGoldstein commented Jan 14, 2020

It will be on by default in Nightly soon

Actually, it's on by default as of today.
https://bugzilla.mozilla.org/show_bug.cgi?id=1031664

I think I agree with Mats (and Emilio) as to how it should act; just think there should be spec text making it explicit.

Background info: https://bugzilla.mozilla.org/show_bug.cgi?id=1609195

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
You can’t perform that action at this time.