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

[CSS2] Added a border-collapse edge case (mono-cell-table) #27753

Closed
wants to merge 2 commits into from

Conversation

@TalbotG
Copy link
Contributor

@TalbotG TalbotG commented Feb 24, 2021

border-collapse-mono-cell-table-001.html
reference/border-collapse-mono-cell-table-001-ref.html

On my website:

http://www.gtalbot.org/BrowserBugsSection/css21testsuite/border-collapse-mono-cell-table-001.html

http://www.gtalbot.org/BrowserBugsSection/css21testsuite/reference/border-collapse-mono-cell-table-001-ref.html

Efforts worth noting:
[css-backgrounds] How borders of different colors of table cell join at corners in border-collapse model
https://lists.w3.org/Archives/Public/www-style/2018Feb/0017.html

[css22] [css-tables] [border-collapse] How borders of different colors join at corners?
https://lists.w3.org/Archives/Public/www-style/2018Mar/0014.html

@TalbotG TalbotG added the CSS2 label Feb 24, 2021
@TalbotG TalbotG requested review from frivoal and fantasai Feb 24, 2021
@TalbotG TalbotG self-assigned this Feb 24, 2021
@wpt-pr-bot wpt-pr-bot requested review from dbaron, kojiishi and svgeesus Feb 24, 2021
@fantasai
Copy link
Contributor

@fantasai fantasai commented Feb 25, 2021

I don't think the behavior at corners in collapsed borders mode is defined enough to accept this test. And also it probably shouldn't be, because intersections for collapsed borders model has lots of considerations. Ideally in a single-celled table it would be identical to separate borders model, but since the general code to handle collapsed borders has to consider many things I don't think making this a requirement is a good idea.

@fantasai fantasai closed this Feb 25, 2021
@TalbotG TalbotG deleted the TalbotG:CSS2-GT-PR11 branch Feb 26, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants