search

infor-design / enterprise-wc

Enterprise-grade web component library for the Infor Design System
Apache License 2.0
27 stars 26 forks source link

IdsAppMenu: Close menu on each click in Mobile Mode #2372

Open OlejnikKamil opened 3 months ago

OlejnikKamil commented 3 months ago

Is your feature request related to a problem or use case? Please describe. This use case indicates use of mobile mode ( around 600 px) In our solution, as can be seen on attached video, user is able to select some position in menu. With current implementation user might be frustrated to click few times to access program, which user was able to open without closing menu before (in ids-enterprise-ng). This solution is not acceptable by our clients and currently we are not able to finish our work on appMenu component. Our solution closes menu, when user pick desired program.

Describe the solution you'd like It would be great to make some parameter to be attached to app-menu, which in case of mobile mode allow us to not close app menu on each click, as it is currently. In our case we need to move into folders to find some applications to load.

Describe alternatives you've considered As suggested by Tim, we tried app-menu-nav, but we end up with previous solution (ids-app-menu).

Additional context

https://github.com/infor-design/enterprise-wc/assets/161036498/a134f59c-6924-4a99-8aaa-b9d763c1f93b

Application: Blending

tmcconechy commented 3 months ago

@OlejnikKamil ok maybe this makes sense, if they are not selecting or navigating might not need to close.

The go forward is more the module nav. Should we add there? Are you adopting that?

OlejnikKamil commented 3 months ago

As I mentioned, we want to stay with this solution, as it suits our needs.

tmcconechy commented 3 months ago

@OlejnikKamil its more about the styles of the new one (app styles in sync). But ok just mentioning it. Whatever we do here will need to on both.

OlejnikKamil commented 1 week ago

Hi @tmcconechy ! Is there any progress with this task? My team is close to the end of development and we would like to release new version soon.

tmcconechy commented 1 week ago

@OlejnikKamil No, i didnt think this needed to be a priority from the sounds of the issue. We have over 500 issues in the backlog so hard to get to everything. Is this a priority for you? Realistically could slot this in next month for Oct release if that works?

OlejnikKamil commented 1 week ago

Would be great @tmcconechy ! As this is one of not so many left, that we need to release.