Skip to content

Refine message for unrecognized at-rule #93

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

Merged
merged 1 commit into from
Dec 11, 2017

Conversation

sideshowbarker
Copy link
Member

This changes the following error message:

Sorry, the at-rule %s is not implemented.

...to this:

Unrecognized at-rule %s


We don’t need the word “Sorry” in there, and the “is not implemented” part
makes users wonder if it means the at-rule isn’t implemented in browsers.

So let’s just make it “Unrecognized” — which covers both the cases of a valid
at-rule like @supports, which the CSS Validator doesn’t yet recognize because
no support for it has been added to the validator yet, and also the case of a
completely invalid at-rule like @key-frames, which is typo for @Keyframes.

This changes the following error message:

> Sorry, the at-rule %s is not implemented.

...to this:

> Unrecognized at-rule %s
@ylafon
Copy link
Member

ylafon commented Dec 11, 2017

Will do the french translation as well.

@ylafon ylafon merged commit 5170a94 into master Dec 11, 2017
@sideshowbarker sideshowbarker deleted the sideshowbarker/at-rule-unrecognized branch December 11, 2017 08:10
@xfq
Copy link
Member

xfq commented Dec 11, 2017

I'll do the Chinese translation after finding out a good Chinese name for "at-rule". (The current one doesn't look good enough to me.)

xfq added a commit that referenced this pull request Dec 12, 2017
ylafon added a commit that referenced this pull request Dec 12, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants