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

Add !important to .select2-hidden-accessible. #3301

Merged
merged 1 commit into from
Apr 30, 2015
Merged

Conversation

fk
Copy link
Member

@fk fk commented Apr 30, 2015

.select2-hidden-accessible is the CSS class that is added to the original <select> element (that Select2 is initialized on) to hide it.
In cases where the original <select> has other style declarations assigned which conflict with those of .select2-hidden-accessible, the latter styles may be overridden, possibly resulting in the original <select> element not being properly hidden.

Similar to what #1549 did for Select2 v3's CSS, this adds the !important rule to all style declarations for .select2-hidden-accessible to make it exponentially harder to override them.

`.select2-hidden-accessible` is the CSS class that is added to the original `<select>` element (that Select2 is initialized on) to hide it.
In cases where the original `<select>` has other style declarations assigned which conflict with those of `.select2-hidden-accessible`, the latter styles may be overridden, possibly resulting in the original `<select>` element not being properly hidden.

Similar to what select2#1549 did for Select2 v3's CSS, this adds the `!important` rule to all style declarations for `.select2-hidden-accessible` to make it exponentially harder to override them.
kevin-brown added a commit that referenced this pull request Apr 30, 2015
Add `!important` to `.select2-hidden-accessible`.
@kevin-brown kevin-brown merged commit 1973843 into select2:master Apr 30, 2015
@kevin-brown
Copy link
Member

Thanks!

@kevin-brown kevin-brown added this to the 4.0.1 milestone May 23, 2015
fk pushed a commit to select2/select2-bootstrap-theme that referenced this pull request Jun 18, 2015
… until this lands in Select2's default CSS with v4.0.1 (select2/select2#3301).
Fix bad documentation along the way (hopefully ;-)).
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