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

Re-design Menus #2586

Closed
mikeyarce opened this issue Sep 21, 2023 · 3 comments · Fixed by #2612
Closed

Re-design Menus #2586

mikeyarce opened this issue Sep 21, 2023 · 3 comments · Fixed by #2612
Assignees
Labels

Comments

@mikeyarce
Copy link
Member

mikeyarce commented Sep 21, 2023

  • New Icon (should be base64 SVG icon so it's compatible with Calypso)
  • Should have all the current same submenu items (Except Add New, remove those)
  • Add "Applications pro" and "Resumes pro" items
  • The Pro upsell items will link to pages that have more information on that add-on
  • There will be a separate task for creating those pages.

I'm not sure if we will rename our "Add-ons" page to "Extensions" so I asked here: p6r3EZ-1Nu-p2#comment-3358
Decision: Will be called "Extensions"

Related: https://github.com/Automattic/wpjm-addons/issues/118

Design:
Image

@yscik yscik self-assigned this Sep 28, 2023
@yscik yscik removed their assignment Oct 2, 2023
@fjorgemota fjorgemota self-assigned this Oct 2, 2023
@fjorgemota
Copy link
Member

For reference, Sensei did have changes similar to this one last year.

Here's the post of the related project thread: p6rkRX-2xl-p2

@yscik
Copy link
Contributor

yscik commented Oct 4, 2023

Also here is the section separator CSS added recently to the Applications plugin:

https://github.com/Automattic/wpjm-addons/blob/8476676fc27dbf1ba833f544ec6eec33771f671f/src/applications/assets/css/menu.scss#L26-L30

@fjorgemota fjorgemota linked a pull request Oct 10, 2023 that will close this issue
@github-actions
Copy link

Closed by #2612.

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 a pull request may close this issue.

3 participants