Skip to content

Shell navbar menu extension #430

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

Closed
wants to merge 31 commits into from
Closed

Conversation

pchemguy
Copy link
Contributor

This modified "shell" component extends menu functionality (primarily adds support for icons in menu items and styling menu items as buttons). This pull request also contains a sample hybrid menu demo file menu_demo.sql.

pchemguy added 11 commits June 19, 2024 10:46
When menu items are specified as objects, "icon" property will be used to add an icon to the left (or instead) of  menu title with standard spacing between the icon and title if both are defined.
If "button" attribute is set to True on top-level menu item, format it as a single "button". Supported attributes are copied from the "button" component.
Otherwise, preserve previous behavior. Icon can still be added independently, if defined.
If "button" attribute is set to True on submenu item, format it as a single "button". Supported attributes are copied from the "button" component. Otherwise, preserve previous behavior. Icon can still be added independently, if defined.
Define "compact"=True on submenu, to remove extra spaing on the right and make sure that icon-only button are stacked (placed one per line).
@pchemguy pchemguy marked this pull request as draft June 20, 2024 05:28
@pchemguy pchemguy marked this pull request as ready for review June 20, 2024 10:23
@pchemguy
Copy link
Contributor Author

I have updated this pull request to include all changes made in "Parse JSON menu_item for SQLite and skip blank items", so that this pull request could be merged without conflicts after the other one is accepted. Please take a look at the extended "shell" component and accept the pull request, if you think it is OK, or let me know of any issues.

pchemguy added 6 commits June 20, 2024 22:14
If "fixed_top_menu" boolean is set to True on the "shell" component, the top navigation bar will be fixed at top. However, it spans the entire page width (probably, by design) regardless of layout used and works best with the fluid layout.
@pchemguy
Copy link
Contributor Author

I have resolved current conflicts.

@lovasoa
Copy link
Collaborator

lovasoa commented Jun 21, 2024

Thanks for the PR !

There are multiple things here.

I'm still not convinced by the looks of this

image

But I clearly see the interest of fixed_top for instance.

Can you open distinct pull requests ? This will make it easier to merge what can be merged now, and to discuss what still needs to be discussed.

@pchemguy
Copy link
Contributor Author

pchemguy commented Jun 22, 2024

To clarify, the three new PRs are just that (parts of this PR).

@lovasoa
Copy link
Collaborator

lovasoa commented Jun 22, 2024

Great, I already merged the sticky top menus. I'm closing this one in favor of the others, then

@lovasoa lovasoa closed this Jun 22, 2024
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

Successfully merging this pull request may close these issues.

2 participants