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.

Sign up for GitHub

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

Jump to bottom

[menu] Option wrapping in secondary menu with border-right in last-child item #2772

Closed
ursoforte opened this issue May 3, 2023 · 1 comment
Closed

[menu] Option wrapping in secondary menu with border-right in last-child item #2772

ursoforte opened this issue May 3, 2023 · 1 comment
Labels
type/bug Any issue which is a bug or PR which fixes a bug
Milestone
2.9.3

Comments

@ursoforte
Copy link

Bug Report

The Wrapping option on new in 2.9.2 is perfect. But when using with the secondary.menu option, the last item is gettin a border-right. - https://fomantic-ui.com/collections/menu.html#wrapping

Steps to reproduce

In the collections Menu, adding CSS class wrapping with secondary option. The last item has a border-right. If you add the CSS centered the border-right is removed, but the last item centered is not wanted.

Expected result

The correct is with border-right none.

Actual result

Testcase

https://jsfiddle.net/crg0qtfo/

Screenshot (if possible)

The second example is correct.
image

Version

2.9.2
@ursoforte ursoforte added state/awaiting-investigation Anything which needs more investigation state/awaiting-triage Any issues or pull requests which haven't yet been triaged type/bug Any issue which is a bug or PR which fixes a bug labels May 3, 2023
@lubber-de lubber-de mentioned this issue May 3, 2023
Merged
@lubber-de
Copy link
Member

Fixed by #2774

@lubber-de lubber-de added state/has-pr An issue which has a related PR open and removed state/awaiting-investigation Anything which needs more investigation state/awaiting-triage Any issues or pull requests which haven't yet been triaged labels May 3, 2023
@lubber-de lubber-de added this to the 2.9.3 milestone May 3, 2023
@lubber-de lubber-de added tag/next-release/nightly Any issue which has a corresponding PR which has been merged and is available in the nightly build and removed state/has-pr An issue which has a related PR open labels May 5, 2023
@lubber-de lubber-de removed the tag/next-release/nightly Any issue which has a corresponding PR which has been merged and is available in the nightly build label Sep 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
type/bug Any issue which is a bug or PR which fixes a bug
Projects
None yet
Milestone
2.9.3
Development

No branches or pull requests
2 participants
@ursoforte @lubber-de