Skip to content

Update branding, hover states, and padding of side nav #2833

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
Merged
laurakwhit merged 6 commits into from
Jul 29, 2025
Merged

Update branding, hover states, and padding of side nav #2833

laurakwhit merged 6 commits into from
Jul 29, 2025

Conversation

@laurakwhit
Copy link
Collaborator

@laurakwhit laurakwhit commented Jul 28, 2025 •
edited
Loading

Description & motivation 💭

  • Updates hover and selected state in the side nav
  • Updates border radius to 0 for hover and selected states
  • Updates font weight to normal for nav items
  • Updates ordering of nav links
  • Updates the position of on hover chevron to collapse the nav
  • Removes extra padding between title and first nav link
  • Adds “Cloud” beside Temporal logo for Cloud UI and “Self-Hosted” for OSS UI

Screenshots (if applicable) 📸

Before After
Screenshot 2025-07-28 at 12 06 47 PMScreenshot 2025-07-28 at 12 07 01 PM Screenshot 2025-07-28 at 11 52 56 AMScreenshot 2025-07-28 at 11 53 13 AM
Screenshot 2025-07-28 at 12 07 26 PMScreenshot 2025-07-28 at 12 07 40 PM Screenshot 2025-07-28 at 11 53 29 AMScreenshot 2025-07-28 at 11 53 51 AM

Design Considerations 🎨

Testing 🧪

How was this tested 👻

  • Manual testing
  • E2E tests added
  • Unit tests added

Steps for others to test: 🚶🏽‍♂️🚶🏽‍♀️

Checklists

Draft Checklist

Merge Checklist

Issue(s) closed

DT-2974

Docs

Any docs updates needed?

@vercel
Copy link

vercel bot commented Jul 28, 2025 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
holocene ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 29, 2025 3:52pm

@laurakwhit laurakwhit force-pushed the DT-2974-update-nav-state branch from d7b51fa to b93d2a4 Compare July 28, 2025 18:32
@laurakwhit laurakwhit marked this pull request as ready for review July 28, 2025 19:11
@laurakwhit laurakwhit requested review from a team and rossedfort as code owners July 28, 2025 19:11
@Alex-Tideman
Copy link
Collaborator

Alex-Tideman commented Jul 28, 2025 •
edited
Loading

Not blocking at all, but thoughts on this instead? Just feels a little more natural to have the dropdowns together at the top

Screenshot 2025-07-28 at 2 22 33 PM Screenshot 2025-07-28 at 2 22 41 PM

laurakwhit
laurakwhit commented Jul 29, 2025
View reviewed changes
@laurakwhit laurakwhit merged commit dc034f1 into main Jul 29, 2025
16 checks passed
@laurakwhit laurakwhit deleted the DT-2974-update-nav-state branch July 29, 2025 16:48
@Alex-Tideman Alex-Tideman mentioned this pull request Sep 3, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Alex-Tideman Alex-Tideman Alex-Tideman approved these changes

@rossedfort rossedfort Awaiting requested review from rossedfort rossedfort is a code owner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@laurakwhit @Alex-Tideman