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

Update -webkit-appearance button and textfield values #4290

Merged
merged 1 commit into from
Jun 13, 2019

Conversation

ddbeck
Copy link
Collaborator

@ddbeck ddbeck commented Jun 12, 2019

According to https://trac.webkit.org/changeset/13874 both values were included when -webkit-appearance first shipped in WebKit.

I set the Safaris to the release where -webkit-appearance shipped. I set Chromium-derived browsers' values to the first release that they shipped (as the feature predates Chromium entirely).

This contributes to #3804.

According to https://trac.webkit.org/changeset/13874 both values were included
when -webkit-appearance first shipped.

I set the Safaris to the release where `-webkit-appearance` shipped. I set
Chromium-derived browsers' values to the first release that they shipped (as
the feature predates Chromium entirely).
@ddbeck ddbeck added the data:css Compat data for CSS features. https://developer.mozilla.org/docs/Web/CSS label Jun 12, 2019
ddbeck added a commit to ddbeck/browser-compat-data that referenced this pull request Jun 12, 2019
While working on mdn#4290, I found that `-webkit-background-clip` was added to
WebKit at the same time as `appearance`:
https://trac.webkit.org/changeset/13874/webkit

Because this predates Chromium, I've set the `-webkit-background-clip` version
on Chromium and derived browsers to their first releases.
Copy link
Member

@Elchi3 Elchi3 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks Daniel! 👍

@Elchi3 Elchi3 merged commit ed0c8df into mdn:master Jun 13, 2019
@ddbeck ddbeck deleted the webkit-appearance branch June 13, 2019 08:44
Elchi3 pushed a commit that referenced this pull request Jun 13, 2019
* Update `-webkit-background-clip`

While working on #4290, I found that `-webkit-background-clip` was added to
WebKit at the same time as `appearance`:
https://trac.webkit.org/changeset/13874/webkit

Because this predates Chromium, I've set the `-webkit-background-clip` version
on Chromium and derived browsers to their first releases.

* Update `-webkit-background-clip: content-box`

This blog post on the WebKit post establishes that the `content-box` value was
original to `-webkit-background-clip`, so I also updated the Chromiums
accordingly:
https://webkit.org/blog/164/background-clip-text/

* Update `-webkit-background-clip: text`

This page establishes that Safari 4 first shipped the `text` value:
https://developer.apple.com/library/archive/documentation/AppleApplications/Reference/SafariCSSRef/Articles/StandardCSSProperties.html

I derived the iOS version from Safari 4's WebKit engine number.

The discussion on this Chromium bug suggests that the `text` value started
working in Chrome 3:
https://crbug.com/8872

* Use correct browser names in background-clip notes

* Fix notes/prefix data for "background-clip: text"
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
data:css Compat data for CSS features. https://developer.mozilla.org/docs/Web/CSS
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants