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

Groups in menuBox #1

Closed
StuartNicholls opened this issue May 2, 2023 · 1 comment
Closed

Groups in menuBox #1

StuartNicholls opened this issue May 2, 2023 · 1 comment

Comments

@StuartNicholls
Copy link

StuartNicholls commented May 2, 2023

As this is now a component, theoretically we don't need groups functionality, although I'm assuming we can add multiple instances to the page?

If this is the case, suggest adding a component header. This will provide the same functionality as groups and allow components to sit between menu items.

Requires:

adaptlearning/adapt-contrib-core#366

@oliverfoster
Copy link
Member

#4 (comment)

I don't think there would be multiple instances of this particular plugin. This is effectively just the boxMenu behaviour, but as a component, including the grouping behaviour as standard.
Multiple instances are much harder to configure as one would need to refer to sub-pages or sub-menus (groups) by id, or some other filter category. The standard menuBox should be easy to configure in the AAT, by just dropping it onto a page.
[...]

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