Skip to content
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.

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

Mobile nav with new menu items #285

Merged
merged 9 commits into from Dec 13, 2021

Conversation

JakubKopys
Copy link
Contributor

Description

This is a combination of 2 PRs: #283 and #284

  1. Merges two navigations for mobile. Currently, we have a hamburger menu and a dropdown menu that's displayed when the user clicks on the avatar.
  2. Change the breakpoint for the mobile navbar. Currently, the mobile navbar is displayed only for mobile (width <= 768px). After these changes, mobile navbar will be displayed on both mobile and tablet viewports (width <= 1024px)
  3. Additionally, fixes the a11y issue with desktop version of nav - all items are now displayed inside a list.

Other changes:
User dropdown menu:

  • Use {{ t 'profile'}} in the my_profile helper in order to update the wording from "My profile" to "Profile".
  • Use "contributions" linker instead of the "my_activities" linker, since the "Request" link is added as a separate menu item.
  • Add "requests" linker.

Requests page:

  • Remove sub-navigation panel altogether

Contributions page

  • Exclude the "Requests" link from the sub-navigation panel.

Subscriptions page

  • Exclude the "Requests" link from the sub-navigation panel.

Screenshots

New navbar - admin:
https://user-images.githubusercontent.com/12835055/144049443-9870aaf6-cd5d-4cb2-8a03-e7048b215773.mov
New navbar - end user:
https://user-images.githubusercontent.com/12835055/144049466-8b5d1496-7883-43de-b205-0286ac5dc4b3.mov

Mobile nav screenshots:

Screenshot 2021-11-30 at 13 40 49

Screenshot 2021-11-30 at 13 40 32

Screenshot 2021-11-30 at 13 40 19

Screenshot 2021-11-30 at 13 40 10

Changed menu items screenshots:

image

Screenshot 2021-11-26 at 11 47 08

Screenshot 2021-11-26 at 11 47 20

Checklist

  • πŸ“— all commit messages follow the conventional commits standard
  • πŸ’… SASS files are compiled
  • ⬅️ changes are compatible with RTL direction
  • β™Ώ changes are accessible
  • πŸ“ changes are tested in Chrome, Firefox, Safari and Edge
  • πŸ“± changes are responsive and tested in mobile
  • πŸ‘ PR is approved by @zendesk/vikings

JakubKopys and others added 8 commits November 30, 2021 13:56
- merges two navigations for mobile. Currently, we have a hamburger menu and a dropdown menu that's displayed when the user clicks on the avatar.
- change the breakpoint for the mobile navbar. Currently, the mobile navbar is displayed only for mobile (width <= 768px). After these changes, mobile navbar will be displayed on both mobile and tablet viewports (width <= 1024px)
…ers' into jkopys/mobile-nav-with-new-menu-items
@JakubKopys JakubKopys requested a review from a team as a code owner December 8, 2021 12:29
@JakubKopys JakubKopys self-assigned this Dec 8, 2021
@Anifacted Anifacted mentioned this pull request Dec 8, 2021
7 tasks
@JakubKopys JakubKopys force-pushed the jkopys/mobile-nav-with-new-menu-items branch from 0f057b6 to ab584c3 Compare December 13, 2021 08:02
@JakubKopys JakubKopys merged commit 1c851b0 into master Dec 13, 2021
@JakubKopys JakubKopys deleted the jkopys/mobile-nav-with-new-menu-items branch December 13, 2021 08:09
zd-svc-github-copenhagen-theme added a commit that referenced this pull request Dec 13, 2021
# [2.13.0](v2.12.2...v2.13.0) (2021-12-13)

### Features

* Mobile nav improvements and new menu items ([#285](#285)) ([1c851b0](1c851b0))
@zd-svc-github-copenhagen-theme
Copy link
Collaborator

πŸŽ‰ This PR is included in version 2.13.0 πŸŽ‰

The release is available on GitHub release

Your semantic-release bot πŸ“¦πŸš€

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
4 participants