Skip to content

Merge plugin-related navigation items #5652

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.

Sign up for GitHub

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

Jump to bottom
Closed
gtsiolis opened this issue Jul 6, 2020 · 2 comments · Fixed by #5655
Closed

Merge plugin-related navigation items #5652

gtsiolis opened this issue Jul 6, 2020 · 2 comments · Fixed by #5655
Labels
contribution welcome documentation

Comments

@gtsiolis
Copy link
Contributor

gtsiolis commented Jul 6, 2020 •
edited
Loading

What problem does this feature solve?

The plugin development documentation sections are linked from a separate menu item at the top navigation, see Plugin Dev Guide. This makes it a) harder to use the top navigation on medium viewports and b) slower to locate the most relevant information regarding plugins or plugin development.

What does the proposed change look like?

It could be worth considering merging the Plugin Dev Guide with the Plugins top navigation menu to provide more clear navigation and make the top navigation more accessible on small viewports. WDYT?

Before (1) Before (2) After
nav-before-1 nav-before-2 nav-after
@haoqunjiang
Copy link
Member

Yeah, the last one looks much cleaner to me. Looks good!

@gtsiolis gtsiolis mentioned this issue Jul 7, 2020
Merged
9 tasks
@gtsiolis
Copy link
Contributor Author

gtsiolis commented Jul 7, 2020

Thanks @sodatea! I've added #5655 to resolve this, feel free to review or forward this to someone. 🏀

/cc @NataliaTepluhina because #2973

/cc @Akryum because #1535

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
contribution welcome documentation
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

docs: merge plugin-related navigation items gtsiolis/vue-cli
2 participants
@gtsiolis @haoqunjiang