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

feat(client): show full text on hover for component state field viewer #630

Merged
merged 1 commit into from
Oct 16, 2024

Conversation

xuzuodong
Copy link
Contributor

This PR slightly improves UX for string value inspect.

Currently sometimes when text is too long and may contains multiple line, it's a bit inconvenient to inspect the whole value.

This PR adds title attribute to solve this.

2024-10-12.15.34.57.mov

Copy link

netlify bot commented Oct 12, 2024

Deploy Preview for vue-devtools-docs canceled.

Name Link
🔨 Latest commit bd44178
🔍 Latest deploy log https://app.netlify.com/sites/vue-devtools-docs/deploys/670a2854a663520008d5d1f3

Copy link

pkg-pr-new bot commented Oct 12, 2024

Open in Stackblitz

@vue/devtools-applet

pnpm add https://pkg.pr.new/@vue/devtools-applet@630

@vue/devtools-core

pnpm add https://pkg.pr.new/@vue/devtools-core@630

@vue/devtools

pnpm add https://pkg.pr.new/@vue/devtools@630

@vue/devtools-api

pnpm add https://pkg.pr.new/@vue/devtools-api@630

@vue/devtools-kit

pnpm add https://pkg.pr.new/@vue/devtools-kit@630

vite-plugin-vue-devtools

pnpm add https://pkg.pr.new/vite-plugin-vue-devtools@630

commit: bd44178

@webfansplz webfansplz changed the title chore: in component module's state field viewer, show full text on hover feat(client): show full text on hover for component state field viewer Oct 16, 2024
@webfansplz webfansplz merged commit c3ec0bc into vuejs:main Oct 16, 2024
7 checks passed
@Tofandel
Copy link

Tofandel commented Dec 5, 2024

We could have an expand value icon just like for objects. As this PR introduces a vulnerability, I suggest it is reverted until a better solution and maybe a refactor of the v-html into a component can be made

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.

3 participants