Skip to content

Support for menus with sections and optional image.#111

Closed
ringods wants to merge 2 commits into
devcows:masterfrom
ringods:menu-sections
Closed

Support for menus with sections and optional image.#111
ringods wants to merge 2 commits into
devcows:masterfrom
ringods:menu-sections

Conversation

@ringods
Copy link
Copy Markdown
Contributor

@ringods ringods commented Sep 10, 2017

@GeorgeWL @adrianmo,

I reviewed the menu navigation to support more of the upstream theme. This PR offers:

  • dropdown menu with single list of menu items
  • dropdown menu with 4 columns of menu items, grouped in sections
  • dropdown menu with 2 column wide image, followed by 2 columns of menu items grouped in sections.

Theme adapted and documentation added.

Part of #101

@ryanfox1985
Copy link
Copy Markdown
Contributor

Looks good!

Copy link
Copy Markdown
Contributor

@GeorgeWL GeorgeWL left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ooks pretty good.

Does this help fix the dropdown issue too btw?

@ringods
Copy link
Copy Markdown
Contributor Author

ringods commented Sep 18, 2017

@GeorgeWL which dropdown issue are you talking about?

@GeorgeWL
Copy link
Copy Markdown
Contributor

#87

@dabibbit
Copy link
Copy Markdown

dabibbit commented Dec 9, 2017

Any progress on getting this merged?

@ringods
Copy link
Copy Markdown
Contributor Author

ringods commented Feb 9, 2018

@ryanfox1985 @adrianmo when could this be merged?

@ringods
Copy link
Copy Markdown
Contributor Author

ringods commented Mar 3, 2018

@ryanfox1985 @adrianmo ping.

@ctindel
Copy link
Copy Markdown

ctindel commented Dec 12, 2019

Is there any chance of getting this merged @ryanfox1985 @adrianmo looks like its been sitting for a while and it would be awesome to have this functionality!

@salim-b salim-b mentioned this pull request Dec 12, 2019
@GeorgeWL
Copy link
Copy Markdown
Contributor

@ringods fix the merge conflicts and we can merge

@ctindel
Copy link
Copy Markdown

ctindel commented Dec 21, 2019

@GeorgeWL I merged this into a new PR to remove the merge conflicts

#257

@ryanfox1985
Copy link
Copy Markdown
Contributor

closed in favour #307

@ryanfox1985 ryanfox1985 closed this Aug 4, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants