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

Sidebar Menu UI Bugs #1346

Closed
subodhpareek18 opened this issue Feb 19, 2017 · 1 comment
Closed

Sidebar Menu UI Bugs #1346

subodhpareek18 opened this issue Feb 19, 2017 · 1 comment

Comments

@subodhpareek18
Copy link

Some comparisons with a pure semantic-ui project (non react):

<Sidebar 
  as={Menu} 
  animation='push' 
  width='very thin' 
  icon='labeled' 
  vertical 
  inverted>

Expected Sidebar Behavior
image

Actual Sidebar Behavior
image

As you can see the menu is getting hidden under Sidebar.Pusher


<Menu.Item disabled link name='/jobs'>
  <Icon name='briefcase' />
  Jobs
</Menu.Item>

Expected Behavior:
image

Actual Behavior:
image

Disabled prop is having no effect on the menu item.


Also facing issues with most of the animations (only push seems to be usable)
feb 19 2017 7-00 pm

Now I can see in the docs that all of the animations are working fine, so what can I do to achieve better performance?

@levithomason
Copy link
Member

levithomason commented Feb 21, 2017

A very thin sidebar simply does not have enough space to fit a labeled icon menu. You'd need to customize the CSS theme and set @labeledIconMinWidth: 0; in the Menu overrides. Otherwise, the menu items are set to have a min width that is wider than the sidebar.

Another option is to not use an icon='labeled' menu, but just an icon menu (size='massive' in this case):

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants