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

place feedback icon lower, only if it really need #14005

Merged
merged 1 commit into from
Jul 3, 2014
Merged

place feedback icon lower, only if it really need #14005

merged 1 commit into from
Jul 3, 2014

Conversation

just-boris
Copy link
Contributor

Now you have default space from top, and remove when if label is hidden.
If it will be changed like this, feedback-icon would be placed correctly even if the label is omitted.

.has-feedback label.sr-only ~ .form-control-feedback {
top: 0;
// Reposition feedback icon if input has visible label above
.has-feedback label:not(.sr-only) ~ .form-control-feedback {
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:not() isn't supported by IE8...

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@cvrebert, oh sorry.
Work around it by additional rule

@cvrebert cvrebert added the css label Jul 1, 2014
@mdo
Copy link
Member

mdo commented Jul 2, 2014

Have a jsbin or something to demo the before and after?

@just-boris
Copy link
Contributor Author

@mdo
http://jsbin.com/hiqiyomu/1 - current master
http://jsbin.com/hiqiyomu/2 - with my fix
the difference only in the last field.

@twbs-lmvtfy
Copy link

Hi @just-boris!

You appear to have posted a live example (http://jsbin.com/hiqiyomu/1), which is always a good first step. However, according to the HTML5 validator, your example has some validation errors, which might be causing your issue:

  • line 6264, column 63: Duplicate ID inputSuccess5.
  • line 6257, column 63: The first occurrence of ID inputSuccess5 was here.

You'll need to fix these errors and post a revised example before we can proceed further.
Thanks!

(Please note that this is a fully automated comment.)

@twbs-lmvtfy
Copy link

Hi @just-boris!

You appear to have posted a live example (http://jsbin.com/hiqiyomu/2), which is always a good first step. However, according to the HTML5 validator, your example has some validation errors, which might be causing your issue:

  • line 6267, column 63: Duplicate ID inputSuccess5.
  • line 6260, column 63: The first occurrence of ID inputSuccess5 was here.

You'll need to fix these errors and post a revised example before we can proceed further.
Thanks!

(Please note that this is a fully automated comment.)

@just-boris
Copy link
Contributor Author

Fixed examples:
http://jsbin.com/hiqiyomu/6 - current master
http://jsbin.com/hiqiyomu/7 - with my fix

@just-boris
Copy link
Contributor Author

This fix is important to me because I want to make an multi-value editor. In this case I have a single common label and many inputs representing each item in list.
http://jsbin.com/doqayodo/1 - example of my UI

@mdo
Copy link
Member

mdo commented Jul 3, 2014

Updated example with all your stuff and a few more tweaks to remove some unnecessary CSS after your changes: http://jsbin.com/hiqiyomu/8.

@mdo mdo added this to the v3.2.1 milestone Jul 3, 2014
@mdo mdo merged commit 77c1a52 into twbs:master Jul 3, 2014
@cvrebert cvrebert mentioned this pull request Jul 3, 2014
@just-boris just-boris deleted the patch-1 branch July 3, 2014 08:37
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