Skip to content

merge_pr_58982

Choose a tag to compare

View all tags
@github-actions github-actions released this 03 Apr 22:58
merge_pr_58982
d1bf16a

Remove border-color: gray from UA stylesheet for

**** GARDENERS! ****
If this change causes a failure or two on the CI, please do me
a huge favor and simply mark the offending tests disabled or
failing, and assign me a bug. I'll take care of cleaning it up.
It's a pain to get such a large CL landed. Thanks in advance.

The spec does not contain this rule:

https://un5nj90kzk5vf152hgyfw29h1eja2.julianrbryant.com/multipage/rendering.html#tables-2

and it causes the borders to incorrectly not default to
currentColor.

This patch implements the change behind the
TableDefaultBorderColorCurrentColor flag, which is experimental
for now. I will need to send an I2S for this change before shipping
it, because of the observable change.

Bug: 494554835
Change-Id: I16881309334193d091c6e0f9b6c6580f05fa3ebe
Reviewed-on: https://un5x4n0kwa1t164zhzvdp2b4bu49r4r40ry9xdr.julianrbryant.com/c/chromium/src/+/7708734
Auto-Submit: Mason Freed masonf@chromium.org
Reviewed-by: Joey Arhar jarhar@chromium.org
Commit-Queue: Mason Freed masonf@chromium.org
Cr-Commit-Position: refs/heads/main@{#1609982}

Assets 8
Loading