Skip to content
This repository was archived by the owner on Oct 2, 2019. It is now read-only.

Fix to allow select type multiple using the bootstrap theme in Firefox/IE #196

Merged
merged 2 commits into from
Oct 7, 2014

Conversation

FoxBuchele
Copy link
Contributor

The bootstrap theme uses a button for the ui-select-match-item element (the pill at the top after selecting one of the options). This causes the click event on the close 'x' to be swallowed in Firefox and IE.

Here's the multiple select demo with the original bootstrap theme. Open in firefox, click on the 'x' next to blue or red, observe that the pill does not go away.

http://plnkr.co/edit/he6uWGFARZsw66K2ERr5?p=preview

Here's the multiple select demo with my suggested updates.

http://plnkr.co/edit/3duaRkJZkJle9MH0Z3XV?p=preview

Fix to prevent clicks being swallowed when clicking on the 'x' within the multiple select pill.
Prevent the dotted line from appearing around the span element after clicking; this behavior did not occur when the span element was a button.
dimirc added a commit that referenced this pull request Oct 7, 2014
Fix to allow select type multiple using the bootstrap theme in Firefox/IE
@dimirc dimirc merged commit 4b4d2f2 into angular-ui:master Oct 7, 2014
@dimirc dimirc mentioned this pull request Feb 16, 2015
2 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants