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

vertically attached dropdown #3542

Closed
wants to merge 4 commits into from
Closed

vertically attached dropdown #3542

wants to merge 4 commits into from

Conversation

larsbo
Copy link
Contributor

@larsbo larsbo commented Jan 7, 2016

add styling for top attached dropdown and bottom attached dropdown
(similar to attached buttons)

example fiddle: http://jsfiddle.net/rr3Ltdzh/

add styling for top attached dropdown and bottom attached dropdown
(similar to attached buttons)
@jlukic
Copy link
Member

jlukic commented Jan 11, 2016

Hi, you might also want to verify other attachments: table, menu, etc. as these get a bit complex.

http://semantic-ui.com/examples/attached.html

@jlukic jlukic added this to the 2.2 milestone Jan 11, 2016
@larsbo
Copy link
Contributor Author

larsbo commented Jan 11, 2016

Yes, I will test the other elements & modules in the next time. For my use case I just needed the segment attached version and thought it might be a good extension for the framework. Thus I created the PR to find out if this is useful at all.

to specify the exact closable behavior of a modal you can now set
closable to ```click```to allow closing only by clicking on the dimmer
or you can set it to ```escapeKey```to allow closing only by pressing
the escape key.
…le-option' into vertically-attached-dropdown
@jlukic jlukic modified the milestones: 2.2.1, 2.2 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.7 Oct 19, 2016
@jlukic jlukic modified the milestones: 2.2.7, 2.2.8 Oct 28, 2016
@jlukic jlukic modified the milestones: 2.2.10, 2.2.11 Mar 28, 2017
@awgv
Copy link
Member

awgv commented May 2, 2017

From what I’ve tested—I copied the example above and tried appending dropdowns to the elements—it works as expected. The only problem is when an attached dropdown is placed between other attached elements like tables or segments, there’s a double border issue, which, I believe, can be fixed with something like that:

.ui.attached + .ui.attached.dropdown:not(.bottom) {
  margin-top: -1px;
}

There’re an issue with attached menus being offset by 1px (#4248), maybe it’s better to deal with it first. Also, the fork was updated from the original, some merge bubbles appeared, and since it doesn’t exist anymore, this PR can’t be updated to remove them.

EDIT: Found something else, when a dropdown is between other attached content and is opened .upward, it retains rounded borders:

.ui.attached + .ui.upward.selection.attached.dropdown:not(.bottom) {
  border-radius: 0 !important;
}

@larsbo
Copy link
Contributor Author

larsbo commented May 2, 2017

thank you for validation, @Banandrew :)
I deleted the branch because I had no time for checking all the other attached elements and the pr didn't get much attention at all. Moreover the pr was cluttered with non related modifications as you already mentioned.

If desired, you can use my changes for an own pr 😃

@jlukic jlukic removed this from the 2.2.11 milestone Jul 11, 2017
@jlukic jlukic modified the milestones: 2.2.12, 2.2.11 Jul 11, 2017
@jlukic jlukic modified the milestones: 2.2.14, 2.2.15 Jan 29, 2018
@jlukic jlukic closed this Feb 20, 2018
@jlukic jlukic modified the milestones: 2.2.15, 2.3.0 Feb 20, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants