redchamps / clean-admin-menu

Magento 2 Extension to cleanup admin menu and Store > Configuration area by arranging third party extension items.
https://redchamps.com
MIT License
141 stars 13 forks source link

Sub Menu feature #8

Closed kpitn closed 4 years ago

kpitn commented 4 years ago

Hi, I'm not a big fan of Amasty, but they make something nice to see to regroup menu.

menu

Actually for example with elastic suite menu, the subtitle are link to dashboard : menu2

rav-redchamps commented 4 years ago

Hi @kpitn

Thanks for submitting this.

  1. Are you referring to the design/colors used in amasty's menu screenshot? If you can elaborate a little further that will be great.

  2. Regarding Elasticsuite: Are you referring to this issue https://github.com/redchamps/clean-admin-menu/issues/5 ?

rav-redchamps commented 4 years ago

I think I got your point for referring Amasty menu's screenshot.

You are looking to have a submenu for each vendor. If yes, then the good news is, extension already works like this. It appears to us as shown in below image

clean-submenu

You might haven't redeployed static contents after extension installation due to which extensions CSS code is not active. If this is the case, then please redeploy the static assets and let me know if it solves the purpose.

kpitn commented 4 years ago

For the submenu, I use capistrano to deploy magento, and the result is the second screenshot of my first message (with 1.0.2 packagist version.)

On local i just pull master branch, and i have a submenu, but i see icon / name, it looks a little broken :)

sub_menu

kpitn commented 4 years ago

After redeploying static icon it looks better.

One of the item is in a second colum, but it's way better, thank for you module good job

menu

menu2

menu3

rav-redchamps commented 4 years ago

Glad it worked and thanks for appreciating our work.

So, can we mark this issue as closed as of now?

kpitn commented 4 years ago

You can close it.

I'm using sass compiler instead of less, so i'm missing css on server.