Skip to content

fix custom-select-indicator in IE10#28882

Closed
mort3za wants to merge 304 commits into
twbs:v4-devfrom
mort3za:master
Closed

fix custom-select-indicator in IE10#28882
mort3za wants to merge 304 commits into
twbs:v4-devfrom
mort3za:master

Conversation

@mort3za

@mort3za mort3za commented Jun 9, 2019

Copy link
Copy Markdown

Inline svg of custom-select is broken in IE10 because there is no width and height attributes.
Tested on a real IE10 in browserstack, it works.

Current: https://getbootstrap.com/docs/4.3/components/forms/#select-menu
Fix: https://deploy-preview-28882--twbs-bootstrap.netlify.com/docs/4.3/components/forms/#select-menu

Johann-S and others added 30 commits February 20, 2019 22:05
jaypinho and others added 20 commits May 29, 2019 18:17
Co-Authored-By: Mark Otto <otto@github.com>
Co-Authored-By: Martijn Cuppens <martijn.cuppens@gmail.com>
Notably:

* bundlesize 0.15.3 → ^0.17.2
* eslint-plugin-unicorn ^8.0.2 → ^9.0.0
* terser ^3.17.0 → ^4.0.0
…callout (twbs#28851)

- Add same advice about keyboard/AT handling to popovers as a callout; also add information about not adding excessive content or interactive controls (and the suggestion to opt for a modal)
- Tweak the tooltips callout for consistency
* Remove unused classes/styles and use utilities instead of specific styles
* Use `getElementById` instead of `querySelector`
Co-Authored-By: Shohei Yoshida <fellows3@gmail.com>
Co-Authored-By: Shohei Yoshida <fellows3@gmail.com>
@mort3za mort3za requested a review from a team as a code owner June 9, 2019 09:29
@mort3za mort3za closed this Jun 9, 2019
@mort3za mort3za reopened this Jun 9, 2019
@MartijnCuppens MartijnCuppens changed the base branch from master to v4-dev June 9, 2019 10:05
@MartijnCuppens MartijnCuppens changed the base branch from v4-dev to master June 9, 2019 10:07
@MartijnCuppens

Copy link
Copy Markdown
Member

Hi @mort3za,

Thanks for the fix! We're dropping IE10 support in Bootstrap v5 but will fix this for the next hotfix for v4. Could you target the v4-dev branch and rebase?

@mort3za

mort3za commented Jun 9, 2019

Copy link
Copy Markdown
Author

Yes, I'll

@mort3za mort3za changed the base branch from master to v4-dev June 9, 2019 10:48
@mort3za mort3za closed this Jun 9, 2019
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.