You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Using doiuse in my project, I encountered the fact that when using the text-decoration rule I received an error message like mdn-text-decoration-shorthand. I read mdn about text-decoration and realized that:
The check is not correct since the text-decoration property itself has been used since Chrome 1
Checking in general is not necessary, since each of the text-decoration-... properties has its own separate checking rules (that's not accurate)
I can do a PR with the edits, but I couldn’t come to the conclusion that it’s better: remove this check altogether or fix it so that it only works on declaration like text-decoration-...: ... . What do you think?
The text was updated successfully, but these errors were encountered:
Hi! Thanks for filing the issue. Do you have a small css snippet so I can better understand the issue? Also note that we support more than what's on the MDN page going bas far as IE8 (though maybe we shouldn't).
and these styles work for the browser versions listed in the first line in the screenshot, but doiuse returns an error
I can also give an example caniuse (an important clarification is that only one word-value can be passed into it, so check in file text-decoration.js on 'text-decoration': /\w \w/ seems correct to me)
Using
doiuse
in my project, I encountered the fact that when using thetext-decoration
rule I received an error message likemdn-text-decoration-shorthand
. I read mdn abouttext-decoration
and realized that:text-decoration
property itself has been used since Chrome 1text-decoration-...
properties has its own separate checking rules (that's not accurate)I can do a PR with the edits, but I couldn’t come to the conclusion that it’s better: remove this check altogether or fix it so that it only works on declaration like
text-decoration-...: ...
. What do you think?The text was updated successfully, but these errors were encountered: