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

Update nav, header, and related elements #293

Closed
18 tasks done
Tracked by #722
codecovdesign opened this issue Jul 19, 2023 · 8 comments
Closed
18 tasks done
Tracked by #722

Update nav, header, and related elements #293

codecovdesign opened this issue Jul 19, 2023 · 8 comments
Assignees
Labels
Dev-Ready This means the UX is reviewed and ready for prioritization scheduling. epic this label is used to mark issues as epics
Milestone

Comments

@codecovdesign
Copy link
Contributor

codecovdesign commented Jul 19, 2023

Problem to solve

context: the nav element (not to confuse with org-selector>breadcrumb) was mostly untouched through the migration (was a former pattern on GH we mirrored). Today, those migration constraints have been lifted and present an opportunity to improve the nav/header/elements relationship. current UX: navigating the app starts at the org selection; then as user navigates there is a persistent breadcrumb:

The problem is the nav is not serving as a navigation tool, but rather displaying external links (doc, support, blog). Important items, but they are secondary and raises question whether they take precedent in the top nav?

While the elements do see some high traffic (pendo reporting), they may be equally or more as effective in alternative context/location (or out of the way for primary nav). Additionally, the org selector and resulting breadcrumb are persistant as user navigates app; but visually are sub-nav (secondary) from the top-nav (primary). While this is a pattern familiar on repo providers (GH/GL); for our purposes this may be unnecessary to be visualized as secondary. Other questions to looks at:

  • Is the “All orgs and repos” view helpful? is it worth constraining to specific org selection?

    • Consider we seen UX bugs with adding plan to wrong org
    • For third part apps, its familiar to be restricted to org context only
    • If this was removed, where could the plan page live (when org is not selected)
  • Consideration on how the top nav and header appear for both signed in and not signed in experience.

    • For not sign in experience, there is a high visibility and marketing opportunity of features (think open source dev that may use be interested in their org)

Ask for pendo related data to marketing team when ready to start this work.

Solution and discovery ideation

ideation details - Look at merging current org selection > breadcrumb to top nav - Reconsider use and location of other elements - docs, support, blog, and feedback - profile dropdown (related to Pavillion updates, but better to be captured here) - Revisit previous user testing: - We didn’t see major nav/breadcrumb issues in the testing; however, we did see some short-lived confusion and opportunity for potentially applying improved visual treatment on the org selector. - Removing elements reduces visual friction - We did see users navigate to docs from the top nav.
  • Previously look at left-nav to show persistent org selector (common pattern that allows for selector + other in-app items + external links). Downsides are 1) large overhaul/UX change, 2) departure from current learned pattern (not gradual change), 3) constrains, or reduces the body container width allowed.
    • Worth visualizing wireframe for for team feedback/exploration, but the downsides are pretty high

🎨 design overview

Screen.Recording.2023-09-19.at.7.55.26.AM.mov

Nice-to-haves:

@codecovdesign codecovdesign added the epic this label is used to mark issues as epics label Jul 19, 2023
@codecovdesign
Copy link
Contributor Author

codecovdesign commented Jul 19, 2023

@katia-sentry katia-sentry transferred this issue from another repository Aug 9, 2023
@codecovdesign codecovdesign self-assigned this Sep 5, 2023
@katia-sentry
Copy link
Contributor

@codecov/design is this a duplicate of #229?

@aj-codecov aj-codecov added the P1: will do priority 7-9 label Sep 6, 2023
@aj-codecov aj-codecov moved this from Idea to Design Complete in Codecov's Roadmap Sep 6, 2023
@aj-codecov aj-codecov added this to the Q3'23 milestone Sep 6, 2023
@aj-codecov
Copy link

@katia-sentry Yes, I updated to use this w/ milestone and labels as needed

@katia-sentry katia-sentry added the deprioritized Work is deprioritized for current quarter label Oct 12, 2023
@katia-sentry
Copy link
Contributor

katia-sentry commented Oct 24, 2023

[Applications] Deprioritized for Q3

@aj-codecov aj-codecov moved this from Design Complete to Selected for development in Codecov's Roadmap Jan 4, 2024
@aj-codecov aj-codecov moved this from Selected for development to Idea/Deprioritized for the quarter in Codecov's Roadmap Jan 4, 2024
@codecovdesign codecovdesign added Dev-Ready This means the UX is reviewed and ready for prioritization scheduling. and removed deprioritized Work is deprioritized for current quarter labels Feb 14, 2024
@codecovdesign codecovdesign added in discovery The design, product, and specifications require refinement and removed Dev-Ready This means the UX is reviewed and ready for prioritization scheduling. labels Apr 17, 2024
@codecovdesign
Copy link
Contributor Author

@Adal3n3 from our sync to prep this issue for dev ready

@Adal3n3 Adal3n3 added Dev-Ready This means the UX is reviewed and ready for prioritization scheduling. and removed in discovery The design, product, and specifications require refinement labels May 2, 2024
@katia-sentry katia-sentry removed the P1: will do priority 7-9 label May 3, 2024
@Adal3n3
Copy link

Adal3n3 commented May 20, 2024

Here's the latest design and please view the design mocks for more details.

  • The font-size for Org selector remains “text-xl/Semibold”, but please double check the consistency across different plans.
  • On the footer, add "Submit a Github issue" link for free plan customers only. No change on the paid plans.
  • Enlarge font size to “text-xl/Semibold” for Repo breadcrumb.
  • We have an on-going investigate branch display on breadcrumb BUG: for bundles / commit list mismatch branches show #1600
  • The placement of a topBanner should stick to the top of the nav bar.

Screenshot 2024-05-20 at 8 37 04 AM

@codecovdesign @spalmurray-codecov @katia-sentry

@Adal3n3
Copy link

Adal3n3 commented May 21, 2024

Making a note from the slack 🧵: link

  • We need to regroup and relocate the support links from the footer to the new support icon/dropdown
  • relocate the Sentry widget

Screenshot 2024-05-21 at 10 50 58 AM
Screenshot 2024-05-21 at 10 51 39 AM
Screenshot 2024-05-21 at 10 52 07 AM

@Adal3n3
Copy link

Adal3n3 commented May 23, 2024

Making a note here from the sync with @spalmurray-codecov @nicholas-codecov @katia-sentry 5/23:

  • Components will be use in this update: Breadcrumb, multi-selection, link with outbound link icon
  • On the onboarding flow, we will remove top nav and add "back" and "cancel" button to make customers focus on the flow. We will keep the sentry feedback widget on the onboarding screens with no change.
  • Sentry feedback widget will be reposition to the new support links icon.
  • New footer, all links center-aligned the page.

Figma: link

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Dev-Ready This means the UX is reviewed and ready for prioritization scheduling. epic this label is used to mark issues as epics
Projects
Status: Done
Development

No branches or pull requests

6 participants