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

[Segment] Colored Segment Should Work in Segment Group #4013

Closed
tubia opened this issue May 11, 2016 · 3 comments
Closed

[Segment] Colored Segment Should Work in Segment Group #4013

tubia opened this issue May 11, 2016 · 3 comments
Milestone

Comments

@tubia
Copy link

tubia commented May 11, 2016

Hi,
I'm trying to understand if it's possible to have a group of segments with one or more of them coloured.
Something like this:

<div class="ui segments">
    <div class="ui red segment"></div>
    <div class="ui segment"></div>
    <div class="ui segment"></div>
</div>

Currently, this rule override the border-top colour:

.ui.segments:not(.horizontal) > .segment:first-child {
border-top: none;
//...
}

Is this the expected result?

Thank you!

@tubia tubia changed the title Group segments Group segments colour May 11, 2016
@jlukic
Copy link
Member

jlukic commented May 15, 2016

@jlukic jlukic changed the title Group segments colour [Segment] Colored Segment Should Work in Segment Group May 15, 2016
@jlukic jlukic added this to the 2.2.1 milestone May 15, 2016
@jlukic jlukic modified the milestones: 2.2.3, 2.2.4 Aug 22, 2016
@jlukic jlukic modified the milestones: 2.2.5, 2.2.6, 2.2.7 Oct 28, 2016
@jlukic jlukic modified the milestones: 2.2.8, 2.2.9 Feb 21, 2017
@kaurinz
Copy link

kaurinz commented Mar 18, 2017

There seems to be some functional overlap between groups of segments, and attached segments. When the example above is implemented as a group of attached segments, it functions as expected.

Not sure if it would work for all use cases, but using attached segments might be an appropriate workaround.

@jlukic jlukic modified the milestones: 2.2.10, 2.2.11 Mar 28, 2017
jlukic added a commit that referenced this issue Jul 3, 2017
@jlukic
Copy link
Member

jlukic commented Jul 3, 2017

Fixed in 2.2.11

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

3 participants