Skip to content

feat: improve charts#214

Merged
danielroe merged 5 commits intonpmx-dev:mainfrom
graphieros:main
Jan 28, 2026
Merged

feat: improve charts#214
danielroe merged 5 commits intonpmx-dev:mainfrom
graphieros:main

Conversation

@graphieros
Copy link
Copy Markdown
Contributor

@graphieros graphieros commented Jan 28, 2026

  • bump vue-data-ui to 3.13.4 with maxWidth support for the large chart
  • set higher height for larger chart in mobile bp
  • add a close button close to thumb in chart modal in mobile

@vercel
Copy link
Copy Markdown

vercel Bot commented Jan 28, 2026

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 8:39am
npmx.dev Ready Ready Preview, Comment Jan 28, 2026 8:39am

Request Review

Copy link
Copy Markdown
Member

@9romise 9romise Jan 28, 2026

Choose a reason for hiding this comment

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

Hi! 👋 Thanks for your work! ❤️
I think we can use useResizeObserver useElementSize here.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Thanks :)
So much better indeed!

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.

sweet!

@danielroe danielroe enabled auto-merge January 28, 2026 08:38
@danielroe danielroe added this pull request to the merge queue Jan 28, 2026
Merged via the queue into npmx-dev:main with commit 7331083 Jan 28, 2026
8 checks passed
This was referenced Jan 28, 2026
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.

4 participants