Semantic-ui: [Message] - Compact class incompatible with icon class

Created on 17 Nov 2016  路  6Comments  路  Source: Semantic-Org/Semantic-UI

Steps to Reproduce

  1. Create <div class="ui compact icon message" />

Expected
The Message should do be producing an icon message that is compact

Result
The compact class is ignored and element is displayed as block (full width)

Testcase
(fork this to get started)
http://jsfiddle.net/95kzk6mr/1/

Confirmed Bug stale

Most helpful comment

@jlukic Hey there ... ui compact icon message still doesn't work. See the original Testcase above (http://jsfiddle.net/95kzk6mr/1/), it still renders at 100% width.

Neither 2.2.11 (where the fix was introduced) nor the latest 2.2.13 solve the issue.

I think what's happening is that .ui.icon.message sets width:100%.
So this problem may simply be fixed by adding the following CSS:

.ui.compact.icon.message {
    width: auto;
}

All 6 comments

Confirmed ... +1

Fixed in 2.2.11

@jlukic Hey there ... ui compact icon message still doesn't work. See the original Testcase above (http://jsfiddle.net/95kzk6mr/1/), it still renders at 100% width.

Neither 2.2.11 (where the fix was introduced) nor the latest 2.2.13 solve the issue.

I think what's happening is that .ui.icon.message sets width:100%.
So this problem may simply be fixed by adding the following CSS:

.ui.compact.icon.message {
    width: auto;
}

There has been no activity in this thread for 90 days. While we care about every issue and we鈥檇 love to see this fixed, the core team鈥檚 time is limited so we have to focus our attention on the issues that are most pressing. Therefore, we will likely not be able to get to this one.

However, PRs for this issue will of course be accepted and welcome!

If there is no more activity in the next 90 days, this issue will be closed automatically for housekeeping. To prevent this, simply leave a reply here. Thanks!

Seems like a simple fix, though I'm sure they don't want a lot of special case css in the library because it would make it harder to maintain. I override it with a similar rule as above for now. Maybe Fomantic will fix it

Maybe Fomantic will fix it

We finally did :wink:

Was this page helpful?
0 / 5 - 0 ratings