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

overflow-x and overflow-y are wrongly flagged #289

Closed
gian1200 opened this issue Jul 10, 2023 · 4 comments
Closed

overflow-x and overflow-y are wrongly flagged #289

gian1200 opened this issue Jul 10, 2023 · 4 comments

Comments

@gian1200
Copy link

After upgrading the dependency to 7.0,0, I'm facing the following error:

Unexpected browser feature "css-overflow" is only partially supported by Safari 15.6, Safari on iOS 15.6, UC Browser for Android 15.5, QQ Browser 13.1, KaiOS Browser 2.5

image

Can I use says:

Effectively all browsers support the CSS 2.1 definition for single-value overflow as well as overflow-x & overflow-y and values visible, hidden, scroll & auto
https://caniuse.com/?search=css-overflow

Will try to make a reproducible project later

./.browserslistrc:

defaults
not op_mini all

./stylelint.config.cjs:

/**
 * @type {import("stylelint").Config}
 */
module.exports = {
	"extends": "stylelint-config-standard",
	"plugins": [
		"stylelint-no-unsupported-browser-features",
		"stylelint-stylistic"
	],
	"rules": {
		"stylistic/indentation": ["tab", { "severity": "warning" }],
		"plugin/no-unsupported-browser-features": [true, { "severity": "error" }]
	}
};
@ghost
Copy link

ghost commented Jul 10, 2023

Have you checked whether the issue also occurs if you use doiuse directly?

@gian1200
Copy link
Author

I though that checking "Can I use" page was the only thing required.

Do you mean this?

>npx doiuse@latest "src\app\index.css"
src\app\index.css:23:1: CSS Logical Properties only partially supported by: QQ Browser (13.1) (css-logical-props)
src\app\index.css:59:1: Media Queries: Range Syntax not supported by: Safari (15.6,16.3), Safari on iOS (15.6,16.0,16.1,16.2,16.3), UC Browser for Android (15.5), QQ Browser (13.1) (css-media-range-syntax)
src\app\index.css:70:1: CSS overflow property only partially supported by: Safari (15.6), Safari on iOS (15.6), UC Browser for Android (15.5), QQ Browser (13.1) (css-overflow)
src\app\index.css:71:1: CSS overflow property only partially supported by: Safari (15.6), Safari on iOS (15.6), UC Browser for Android (15.5), QQ Browser (13.1) (css-overflow)

Interesting, the output is the same 4 new errors I got with [email protected].

So, should I move my report to DoIUse, or to which project?

Thanks

@ghost
Copy link

ghost commented Jul 10, 2023

@gian1200 Yes if the error is identical to what you're getting from this plugin then it's coming from doiuse. This plugin leverages doiuse for checking css support, so if you feel that there is a bug in doiuse's detection then reporting it there would be the best thing to do. Once it's fixed there (assuming it's a bug), then it'll automatically be fixed for this plugin as well.

This plugin is merely the glue between doiuse and stylelint. So if anything is wrong with that integration, it's most likely this plugin. Anything that shows up as well when using doiuse directly is most likely occuring in doiuse. I'll close this for now, but feel free to let me know if it's an issue with this plugin.

@ghost ghost closed this as completed Jul 10, 2023
@gian1200
Copy link
Author

gian1200 commented Oct 9, 2023

For future reference, the issue on DoIUse is anandthakker/doiuse#166; and the PR is anandthakker/doiuse#160

This issue was closed.
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

No branches or pull requests

1 participant