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

Minor UI fixes #312

Merged
merged 1 commit into from Feb 17, 2024
Merged

Minor UI fixes #312

merged 1 commit into from Feb 17, 2024

Conversation

svc-shorpo
Copy link
Contributor

Screenshot 2024-02-16 at 7 39 30 PM

Minor tweaks and fixes:

  • [AppNav] Use ScrollArea component to re-style scrollbars for consistency across platforms
  • [AppNav] Show search inputs even if there's less than 1 saved search/dashboard (for styling and for better CMD+K discoverability)
  • [AppNav] Tweak text and icon colors
  • [Search Page] Add spacing between icon and text in "Zoom Out", "Zoom In" btns
  • [Search Page] Fix Tags count text appearing on top of Details panel
  • [Tags Popup] Open by clicking instead of by hover for 1) better ux 2) better accessibility
  • [Tags Popup] Sort tags alphabetically, otherwise they rearrange constantly while using
  • [Tags Popup] Align tags popup to bottom-left so it doesn't reposition when parent element changes size (like when adding filters in Alerts page)
  • [Dashboard Page] Use Button instead of Badge for tags for better accessibility

Copy link

changeset-bot bot commented Feb 17, 2024

🦋 Changeset detected

Latest commit: 595ec18

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages
Name Type
@hyperdx/app Patch
@hyperdx/api Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@svc-shorpo svc-shorpo merged commit 95f5041 into main Feb 17, 2024
4 checks passed
@svc-shorpo svc-shorpo deleted the sr/navbar-update branch February 17, 2024 21:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants