Skip to content

Update the page header in admin dashboard #16238

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

Closed
gtsiolis opened this issue Feb 6, 2023 · 2 comments · Fixed by #16307
Closed

Update the page header in admin dashboard #16238

gtsiolis opened this issue Feb 6, 2023 · 2 comments · Fixed by #16307
Assignees
Labels
aspect: navigation feature: admin dashboard team: webapp Issue belongs to the WebApp team type: improvement Improves an existing feature or existing code

Comments

@gtsiolis
Copy link
Contributor

gtsiolis commented Feb 6, 2023

Problem to solve

Following up from #16164 (comment), the admin dashboard:

  1. No longer uses an active state for the admin menu at the top.
  2. Does not use a generic settings page header as the rest of the product.

Proposal

Here's a list of MVC steps that could be tackled all in one step or separate PRs.

  1. Use a generic page header for all top-level pages in the admin dashboard, see screenshot below.
  2. Keep the admin menu at the top active when navigating through the top-level pages in the admin dashboard.
  3. Remove the sidebar menu and use the tabs for the admin dashboard #7879
Page header (BEFORE) Page header (AFTER)
Screenshot 2023-02-06 at 4 35 22 PM Screenshot 2023-02-06 at 4 33 07 PM
user-before user-after
@Devansu-Yadav
Copy link
Contributor

@gtsiolis I would like to work on this issue. As I do not have access to the admin dashboard, how do you suggest I could see the changes I make getting reflected on my dev environment dashboard?

@gtsiolis
Copy link
Contributor Author

gtsiolis commented Feb 7, 2023

Hey @Devansu-Yadav! You can open a PR, and I'll spin up a preview environment for it—then I can help make you an admin.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
aspect: navigation feature: admin dashboard team: webapp Issue belongs to the WebApp team type: improvement Improves an existing feature or existing code
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants