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

small in alert-danger #12782

Closed
HardCoderNET opened this issue Feb 19, 2014 · 4 comments
Closed

small in alert-danger #12782

HardCoderNET opened this issue Feb 19, 2014 · 4 comments

Comments

@HardCoderNET
Copy link

I checked only .alert-danger
http://jsfiddle.net/HardCoderNET/ns64W/

@cvrebert cvrebert added the css label Feb 19, 2014
@cvrebert cvrebert added this to the v3.2.0 milestone Feb 19, 2014
@saas786
Copy link
Contributor

saas786 commented Feb 20, 2014

Could you please be more specific about what you want? and whats wrong?

I reviewed your fiddle, and it seems like you are making mess of you code.

You have

small,

inside alert
inside heading
and you also applying text-* utility class to small?

Why would you do that?

heading small have its own color,
if you are using alert class on container, then why you would need text-danger class?

you should either use text-danger or use alert, don't use both in same context, makes no sense to me (atleast).

Also don't stuff classes in html to achieve this kind of result, instead achieve these results with css, its standard and recommended method.

As far as consider this as a bug, I don't think so,

as we can't provide flexibility and consistency both together for some elements.

It was all three

headings color, and than small colors (inside heading and such) and text-* utility classes (danger, success etc) are highly desired features, and core team tried to make them work together as much as it could be possible without sacrificing core standards, so please don't ask for too much out of core code, rather do these hacks on your on specific to your project.

There is not many projects where you would like to achieve same results, so stuffing this in core makes no sense.

Remember the more clean you keep your core (no hacks, tricks applied to core code) the more browsers, devices compatibility you will get. And it will also makes your life easier to trace the issues, as with hacks, its hard to fix the issues if they are part of core.

Framework is meant to give you just kind of roadmap, its not a final solution, so any specific changes should be done in your own custom files.

@HardCoderNET
Copy link
Author

I just think that the tag "small" and class "small" should have the same color as the rest of the text inside the "div.alert" depending by type (danger/warning/primary). Now, the text inside the "small" tag is gray, even if we assign the class "text-danger".

Я понимаю что все это из за заголовка, и проблема не критична. В любом случае, тему можно закрыть.

@mdo
Copy link
Member

mdo commented Feb 24, 2014

The shitty part about this is there's no real way to go about fixing this without busting something else. We have a couple options:

  1. Add !important to our text classes, but that's just overkill as we know from utility classes.
  2. Remove the color all together from small/.small within headings, and don't set it at all.
  3. Set the color on all small/.small` elements, regardless of where they appear.

The only viable one is the second in my opinion. That, or we make a note of this and punt to v4. Thoughts?

@mdo
Copy link
Member

mdo commented Mar 9, 2014

Going to have to punt this to v4. I don't want to get into more specificity wars with ourselves for stuff like this. We'll be doing more to avoid this come v4.

@mdo mdo closed this as completed Mar 9, 2014
@mdo mdo removed this from the v3.2.0 milestone Mar 9, 2014
@mdo mdo mentioned this issue Aug 19, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants