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

@ringods ringods commented Sep 10, 2017

Copy link
Copy Markdown
Contributor

@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!

@GeorgeWL GeorgeWL left a comment

Copy link
Copy Markdown
Contributor

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

ringods commented Sep 18, 2017

Copy link
Copy Markdown
Contributor Author

@GeorgeWL which dropdown issue are you talking about?

@GeorgeWL

Copy link
Copy Markdown
Contributor

#87

@dabibbit

dabibbit commented Dec 9, 2017

Copy link
Copy Markdown

Any progress on getting this merged?

@ringods

ringods commented Feb 9, 2018

Copy link
Copy Markdown
Contributor Author

@ryanfox1985 @adrianmo when could this be merged?

@ringods

ringods commented Mar 3, 2018

Copy link
Copy Markdown
Contributor Author

@ryanfox1985 @adrianmo ping.

@ctindel

ctindel commented Dec 12, 2019

Copy link
Copy Markdown

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

ctindel commented Dec 21, 2019

Copy link
Copy Markdown

@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