r7-labs / R7.Epsilon

Highly customizable, multiportal, responsive skin for DNN Platform based on Bootstrap 4
GNU Affero General Public License v3.0
5 stars 2 forks source link

Main menu rework #96

Closed roman-yagodin closed 4 years ago

roman-yagodin commented 5 years ago

New main menu features:

Optional:

roman-yagodin commented 5 years ago

Current concept is a semi-megamenu:

Pluses:

Снимок экрана от 2019-07-10 15-26-57

Keyboard navigation going between levels 0 and 1 in the current concept is wrong, and so is the level 1 menu placement in mobile view.

roman-yagodin commented 5 years ago

Variant with separate buttons for navigation and dropdown turned up not so promissing:

So we should stick to current concept, just need to figure out how to make navigation to the parent pages from dropdown clearer - e.g. add some text like "Go to ..." / "Open page..." or make additional row for level 1 items.

roman-yagodin commented 5 years ago

I think current main menu concept is OK for the beta. The only thing currently missing is proper keyboard navigation - all menu items is accessible, but not in correct order, so it require additional steps to navigate in- or out- of level 1 submenus.

roman-yagodin commented 5 years ago

Need to explore passing additional parameters via MenuNode.CommandName and MenuNode.CommandArgument through the NodeManipulator. This would allow to display dates in badges for news entries, star featured pages, mark new pages, etc.

roman-yagodin commented 4 years ago

Two-level dropdowns issues in trunk:

roman-yagodin commented 4 years ago

On second thought, it's looks better to implement second-level dropdowns with just collapse.