Bug 256528 - Sync `hr` element UA stylesheet for missing 'color: gray'
Summary: Sync `hr` element UA stylesheet for missing 'color: gray'
Status: RESOLVED DUPLICATE of bug 258725
Alias: None
Product: WebKit
Classification: Unclassified
Component: CSS (show other bugs)
Version: Safari Technology Preview
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL: http://wpt.live/html/rendering/non-re...
Keywords: BrowserCompat, InRadar, WPTImpact
Depends on:
Blocks:
 
Reported: 2023-05-09 09:24 PDT by Ahmad Saleem
Modified: 2023-07-14 12:06 PDT (History)
5 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Ahmad Saleem 2023-05-09 09:24:32 PDT
HI Team,

While looking into failing testcase on WPT, I noticed that Firefox is passing this test case:

WPT Link - https://wpt.fyi/results/html/rendering/non-replaced-elements/the-hr-element-0/hr.html?label=experimental&label=master&aligned

Color - !EQ("rgb(128, 128, 128)", "rgb(0, 0, 0)")

While looking at HTML Standard: https://html.spec.whatwg.org/#the-hr-element-2

I noticed that it has:

color: gray;

^ it is also present in 'Firefox' html.css file (UA stylesheet).

Adding this in local build and checking above testcase, we start to pass failing test case.

Just wanted to raise so we can fix this and if we have any reason to not do this then we can do so.

NOTE - we are doing opposite in this regard for <table> border to ask for removing 'color: gray' while asking to have it for 'hr' element.

Thanks!
Comment 1 Karl Dubost 2023-05-15 01:28:36 PDT
We could do it. I don't think there is an issue against it.
Comment 2 Ahmad Saleem 2023-05-15 01:44:12 PDT
(In reply to Karl Dubost from comment #1)
> We could do it. I don't think there is an issue against it.

My only concern is that WebKit had ‘table-border: gray’ and standards said a ‘Nope’, you have ‘black’ as default but nothing from UA Stylesheet and now we have web-spec saying for ‘gray’ to be in UA stylesheet. It seems like two opposite things to do in forcing colors.
Comment 3 Anne van Kesteren 2023-05-15 04:44:22 PDT
Given that Chromium and WebKit match I would suggest raising an issue against whatwg/html to get the standard changed.
Comment 4 Tim Nguyen (:ntim) 2023-05-15 17:53:34 PDT
I think all browsers display it gray, or some variation. Blink & WebKit display it at rgb(154,154,154) which is close to gray (rgb(128,128,128)).

The only difference is that the spec suggests to use `border-color: currentColor` and then set the `color` to the actual color (which is nicer, since you can tweak the color the HR using the color property as well).
Comment 5 Radar WebKit Bug Importer 2023-05-16 09:25:20 PDT
<rdar://problem/109409002>
Comment 6 Ahmad Saleem 2023-07-14 12:06:07 PDT

*** This bug has been marked as a duplicate of bug 258725 ***