Skip to content

Change $input-border-color to HEX color value #21318

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

Closed
wants to merge 1 commit into from
Closed

Change $input-border-color to HEX color value #21318

wants to merge 1 commit into from

Conversation

kvlsrg
Copy link
Contributor

@kvlsrg kvlsrg commented Dec 8, 2016

This PR fixes two problems with input groups.

First is hover change of middle (not rounded) border color of input because of input & button borders overlay:

search-hover

Second one is that now input color is not compatible with dark background:

hex-border-color-before

after this PR:

hex-border-color-after

Sorry if I didn't get why now $input-border-color has RGBA value. But it will be great to fix this issues without changing z-indexes logic or any other major changes. Thanks!

@mdo mdo modified the milestone: v4.0.0-beta Dec 20, 2016
@bastienmoulia
Copy link
Contributor

twbs-savage pushed a commit to twbs-savage/bootstrap that referenced this pull request Apr 6, 2017
Add `$btn-secondary-border` border color for form control in input group in the navbar when there is a dark background.

Because the normal border color is `rgba(0,0,0,.15)` and depend of the background color the input in input group seems to have no border.

Fix twbs#21098
Close twbs#21318
@mdo
Copy link
Member

mdo commented May 26, 2017

I'm actually liking #22612 more than this PR. Will revisit after Beta 1 ships though, so punting this to the next release cycle.

@mdo mdo modified the milestones: v4.0.0-beta.2, v4.0.0-beta May 26, 2017
@mdo mdo removed this from the v4.0.0-beta.2 milestone Jun 18, 2017
@mdo
Copy link
Member

mdo commented Jul 2, 2017

Going with #22612 in Beta 2.

@mdo mdo closed this Jul 2, 2017
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.

3 participants