Skip to content

Conversation

@sanne-san
Copy link
Contributor

Changes

  • Improve button styles and consistency in top bar
  • Update filter pill style
  • Update filter menu style
  • Change realtime view to show current visitors and last updated in top bar rather than in the graph tabs
  • Update tooltip style and add delay to show tooltip
  • Update main graph tabs style

Tests

  • Automated tests have been added
  • This PR does not require tests

Dark mode

  • The UI has been tested both in dark and light mode

- Improve button styles and consistency in top bar
- Update filter pill style
- Update filter menu style
- Change realtime view to show current visitors and last updated in top bar rather than in the graph tabs
- Update tooltip style and add delay to show tooltip
- Update main graph tabs style
@github-actions
Copy link

github-actions bot commented Nov 3, 2025

Preview environment👷🏼‍♀️🏗️
PR-5858

: null,
seeMoreWidth:
SEE_MORE_LEFT_MARGIN_PX +
SEE_MORE_WIDTH_PX +
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I know this is a draft, so feel free to ignore, but for the math to work out, this value needed to be known and used in the calculations.

If you open https://pr-5858.review.plausible.io/plausible.io?f=is,country,US&f=is,page,/open-source-website-analytics&f=is,region,US-TX&l=US,United%20States&l=US-TX,Texas at 550px width (just random example), we expect the query period menu not to overflow the bar, but it does, even though it would be possible to avoid overflow if the Country is United States filter wrapped into the vertical menu.
Left: PR, right: staging
Image

I know it's weird to hard code the width of the button in PX: I used to dynamically measure it, but it created a loop where it first needed to be drawn (let's say with size-5), then measured, then maybe thrown away.

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 this pull request may close these issues.

3 participants