Skip to content

fix: chart improvements#230

Merged
patak-cat merged 4 commits intonpmx-dev:mainfrom
graphieros:main
Jan 28, 2026
Merged

fix: chart improvements#230
patak-cat merged 4 commits intonpmx-dev:mainfrom
graphieros:main

Conversation

@graphieros
Copy link
Copy Markdown
Contributor

@graphieros graphieros commented Jan 28, 2026 •
edited
Loading

Related to #223

  • bump vue-data-ui from 3.13.4 to 3.13.5
  • expose the zoom tooltip sides to format weekly dates accordingly with a shorter label (left tooltip with weekStart; right tooltip with weekEnd):
image
  • add start & end dates in filenames

@vercel
Copy link
Copy Markdown

vercel Bot commented Jan 28, 2026 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
docs.npmx.dev Ready Ready Preview, Comment Jan 28, 2026 1:25pm
npmx.dev Ready Ready Preview, Comment Jan 28, 2026 1:25pm

Request Review

@graphieros graphieros marked this pull request as draft January 28, 2026 12:47
@graphieros graphieros mentioned this pull request Jan 28, 2026
Closed
3 tasks
@graphieros graphieros marked this pull request as ready for review January 28, 2026 13:29
@graphieros
Copy link
Copy Markdown
Contributor Author

graphieros commented Jan 28, 2026

One item from #223 is not fixed yet, but these improvements could probably be included in the meanwhile.
I don't really have a clue on how to solve the date selection thing. Hoping for someone with a better brain than mine to have a go at it ^^

@dominikg
Copy link
Copy Markdown

dominikg commented Jan 28, 2026

maybe calculating the other end of date range from interval is too much to ask for. better labeling and filename support are great. thank you so much

patak-cat
patak-cat approved these changes Jan 28, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@patak-cat patak-cat left a comment

Choose a reason for hiding this comment

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

nice! let's merge and iterate

@patak-cat patak-cat added this pull request to the merge queue Jan 28, 2026
@graphieros
Copy link
Copy Markdown
Contributor Author

graphieros commented Jan 28, 2026

maybe calculating the other end of date range from interval is too much to ask for. better labeling and filename support are great. thank you so much

I do believe a cool date picker component which limits selections based on granularity could provide ease of use (if such a component exists)

Merged via the queue into npmx-dev:main with commit 192efd5 Jan 28, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@patak-cat patak-cat patak-cat approved these changes

+1 more reviewer

@dominikg dominikg dominikg approved these changes

Reviewers whose approvals may not affect merge requirements

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

@graphieros @dominikg @patak-cat