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

Remove outdated CSS vendor prefix #430

Merged
merged 1 commit into from
Oct 28, 2024
Merged

Conversation

DRSDavidSoft
Copy link
Contributor

As requested in #429 (comment):

This pull request includes a change to the resources/sass/_base.scss file, removing the redundant CSS vendor prefixes for the ::selection pseudo-element styles.

CSS improvements:

  • Removed redundant ::selection styles for ::-moz-selection and ::-webkit-selection to simplify the code.

  • These vendor prefixes have not been required for a long time, according to caniuse:
    https://caniuse.com/?search=selection


I argue that vendor prefixes must not even be directly included in the Sass code, instead we should introduce tools such as Autoprefixer into our tooling to do this based on Browserslist data.

Lastly, as kint is mainly a development tool rather than production tool, I argue it's fair to assume the dev would be running on a fairly recent browser that would not be outdated beyond just a couple of versions.

@jnvsor jnvsor merged commit c9fb557 into kint-php:master Oct 28, 2024
17 checks passed
@DRSDavidSoft DRSDavidSoft deleted the no-prefix branch October 28, 2024 12:14
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.

2 participants