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] Add test ensuring border-image is displayed with a transparent border-color #26350

Conversation

@twilco
Copy link
Contributor

@twilco twilco commented Oct 31, 2020

A transparent border-color should not prevent a border-image from being rendered, but this is the case in WebKit today. Gecko and Chromium do not exhibit this behavior.

Picture showing WebKit not rendering blue-orange linear-gradient border-image while Chromium and Firefox do.

WebKit bug: https://bugs.webkit.org/show_bug.cgi?id=217900

Spec: https://www.w3.org/TR/2017/CR-css-backgrounds-3-20171017/#border-images

@wpt-pr-bot wpt-pr-bot requested review from dbaron and fantasai Oct 31, 2020
@twilco twilco force-pushed the twilco:border-image-should-beat-border-color branch from 7abba4d to d01a958 Oct 31, 2020
…transparent border-color
@twilco twilco force-pushed the twilco:border-image-should-beat-border-color branch from d01a958 to 44e9cfe Nov 4, 2020
@twilco twilco changed the title [css-backgrounds] Add test ensuring border-image beats a transparent border-color [css-backgrounds] Add test ensuring border-image is displayed with a transparent border-color Nov 4, 2020
@twilco
Copy link
Contributor Author

@twilco twilco commented Nov 4, 2020

Fixed both comments, @smfr.

@smfr
smfr approved these changes Nov 4, 2020
@gsnedders gsnedders merged commit acfda5e into web-platform-tests:master Feb 22, 2021
31 checks passed
31 checks passed
Azure Pipelines Build #20201104.26 succeeded
Details
Azure Pipelines (./wpt test-jobs) ./wpt test-jobs succeeded
Details
Azure Pipelines (affected tests without changes: Safari Technology Preview) affected tests without changes: Safari Technology Preview succeeded
Details
Azure Pipelines (affected tests: Safari Technology Preview) affected tests: Safari Technology Preview succeeded
Details
Azure Pipelines (wpt.fyi hook: safari-preview-affected-tests) wpt.fyi hook: safari-preview-affected-tests succeeded
Details
Azure Pipelines (wpt.fyi hook: safari-preview-affected-tests-without-changes) wpt.fyi hook: safari-preview-affected-tests-without-changes succeeded
Details
download-firefox-nightly Community-TC (pull_request)
Details
infrastructure/ tests Community-TC (pull_request)
Details
infrastructure/ tests (Python 3) Community-TC (pull_request)
Details
lint Community-TC (pull_request)
Details
resources/ tests (Python 2) Community-TC (pull_request)
Details
resources/ tests (Python 3.6) Community-TC (pull_request)
Details
resources/ tests (Python 3.8) Community-TC (pull_request)
Details
sink-task Community-TC (pull_request)
Details
tools/ integration tests (Python 2) Community-TC (pull_request)
Details
tools/ integration tests (Python 3.6) Community-TC (pull_request)
Details
tools/ integration tests (Python 3.8) Community-TC (pull_request)
Details
tools/ unittests (Python 2) Community-TC (pull_request)
Details
tools/ unittests (Python 3.6) Community-TC (pull_request)
Details
tools/ unittests (Python 3.8) Community-TC (pull_request)
Details
update-built Community-TC (pull_request)
Details
wpt-chrome-dev-results Community-TC (pull_request)
Details
wpt-chrome-dev-results-without-changes Community-TC (pull_request)
Details
wpt-chrome-dev-stability Community-TC (pull_request)
Details
wpt-decision-task Community-TC (pull_request)
Details
wpt-firefox-nightly-results Community-TC (pull_request)
Details
wpt-firefox-nightly-results-without-changes Community-TC (pull_request)
Details
wpt-firefox-nightly-stability Community-TC (pull_request)
Details
wpt.fyi - chrome[experimental] Chrome results
Details
wpt.fyi - firefox[experimental] Firefox results
Details
wpt.fyi - safari[experimental] Safari results
Details
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

6 participants