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

Admin main menu possible issue in DEV RC #1120

Closed
tzyganu opened this issue Mar 24, 2015 · 4 comments
Closed

Admin main menu possible issue in DEV RC #1120

tzyganu opened this issue Mar 24, 2015 · 4 comments
Labels
bug report Issue: Ready for Work Gate 4. Acknowledged. Issue is added to backlog and ready for development

Comments

@tzyganu
Copy link
Contributor

tzyganu commented Mar 24, 2015

There might be a possible issue with the new design for the admin menu in dev RC.
If I add too many top level menu items I don't get to see the bottom ones.
I know this is a bad practice and you should not have too many top level menu items, but for sure this will happen eventually for someone.
I think a vertical scroll is needed in case there are a lot of menu items.
I attached a screenshot to explain it better.
Notice that "menu 6" is cut off and there are other menu items below that and I have no way of accessing them.

Imgur
@gotbahn
Copy link

gotbahn commented Mar 24, 2015

Hey @tzyganu

Yeah, for the moment it's fixed and overlaps if menu height is bigger than window. We are planning use smart scroll here in next releases.

How do you feel with new admin area design?

@tzyganu
Copy link
Contributor Author

tzyganu commented Mar 24, 2015

@godban As a developer used to the old magento style I'm not very fond of it, but I'll get over it 😄
but I'm trying to view it through the eyes of a user that sees this for the first time. If I look at it that way I kind of like it.

@tkacheva
Copy link

@tzyganu thank you for feedback and reporting the issues. New Magento Admin look&feel work is in progress and menu issues will be addressed soon

MAGETWO-34929

@sshrewz sshrewz added Issue: Ready for Work Gate 4. Acknowledged. Issue is added to backlog and ready for development CS bug report labels Mar 24, 2015
@gotbahn gotbahn mentioned this issue Apr 9, 2015
Closed
@tzyganu
Copy link
Contributor Author

tzyganu commented Jun 16, 2015

This looks like is fixed in 0.74.0-beta13. It may have been fixed earlier but I just noticed it by mistake.
I will close the issue. Thanks for the fix.

@tzyganu tzyganu closed this as completed Jun 16, 2015
magento-team pushed a commit that referenced this issue May 19, 2017
@ishakhsuvarov
Merge pull request #1120 from magento-engcom/2.1-develop-prs
e1e4617 
[EngCom] Public Pull Requests - 2.1
 - MAGETWO-69238: Keep transparency when resizing images [2.1 backport] #9662
 - MAGETWO-69236: Add configurations for change email templates [2.1 backport] #9661
 - MAGETWO-69235: Do not di:compile tests/ folder #9660
 - MAGETWO-69223 Fix for #9646 #9647
 - MAGETWO-69234: Backport of MAGETWO-69152: Removed workaround for old Webkit bug #9655
VitaliyBoyko pushed a commit to VitaliyBoyko/magento2 that referenced this issue Jun 22, 2018
@maghamed
Merge pull request magento#1120 from magento-engcom/1117-Fix-hard-dep…
b50a893 
…endency-InventoryCache-module-on-Inventory

Fix hard dependency InventoryCache module on Inventory module
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug report Issue: Ready for Work Gate 4. Acknowledged. Issue is added to backlog and ready for development
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@tzyganu @gotbahn @tkacheva @sshrewz