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

IE8 button group support #13549

Merged
merged 1 commit into from
May 10, 2014
Merged

IE8 button group support #13549

merged 1 commit into from
May 10, 2014

Conversation

danielhusar
Copy link
Contributor

Hi,

is there any particular reason for not supporting 0 opacity for button groups on IE8 ?
(i can't have them hidden with display:none)

If there is any reason behind it, ignore :)

@cvrebert cvrebert added the css label May 9, 2014
@cvrebert cvrebert added this to the v3.2.0 milestone May 9, 2014
@hnrch02
Copy link
Collaborator

hnrch02 commented May 10, 2014

This should probably utilize the opacity mixin.

@danielhusar
Copy link
Contributor Author

Yes you are right, I havent noticed it.

commit 81324491f4913d6797c3c9e4df83a1aefd7d9116
Author: Daniel <[email protected]>
Date:   Sat May 10 17:37:29 2014 +0100

    use opacity mixin

commit 694b0a3fcd196e8cba9e4c23850f903c0804cb44
Author: Daniel Husar <[email protected]>
Date:   Fri May 9 13:47:35 2014 +0100

    Update button-groups.less
mdo added a commit that referenced this pull request May 10, 2014
@mdo mdo merged commit 696844f into twbs:master May 10, 2014
@mdo mdo mentioned this pull request May 10, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants