Skip to content

Conversation

@ringods
Copy link
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
Contributor

Looks good!

Copy link
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
Contributor Author

ringods commented Sep 18, 2017

@GeorgeWL which dropdown issue are you talking about?

@GeorgeWL
Copy link
Contributor

#87

@dabibbit
Copy link

dabibbit commented Dec 9, 2017

Any progress on getting this merged?

@ringods
Copy link
Contributor Author

ringods commented Feb 9, 2018

@ryanfox1985 @adrianmo when could this be merged?

@ringods
Copy link
Contributor Author

ringods commented Mar 3, 2018

@ryanfox1985 @adrianmo ping.

@ctindel
Copy link

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
Contributor

@ringods fix the merge conflicts and we can merge

@ctindel
Copy link

ctindel commented Dec 21, 2019

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

#257

@ryanfox1985
Copy link
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