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

Ellipsis have "cursor: pointer" style even when the content is short #1299

Closed
ouuan opened this issue Oct 8, 2021 · 1 comment · Fixed by #1302
Closed

Ellipsis have "cursor: pointer" style even when the content is short #1299

ouuan opened this issue Oct 8, 2021 · 1 comment · Fixed by #1302
Labels
bug Something isn't working

Comments

@ouuan
Copy link
Contributor

ouuan commented Oct 8, 2021

TuSimple/naive-ui version (版本)

2.19.5

Vue version (Vue 版本)

3.2.19

Browser and its version (浏览器及其版本)

Mozilla Firefox 93.0

System and its version (系统及其版本)

Arch Linux 5.14.9-zen2-1-zen

Node version (Node 版本)

v16.10.0

Reappearance link (重现链接)

https://codesandbox.io/s/naive-ui-ellipsis-short-content-cursor-pointer-hfn1p?file=/src/App.vue

Reappearance steps (重现步骤)

Use <n-ellipsis expand-trigger="click"> with a short content.

Expected results (期望的结果)

It doesn't have the cursor: pointer style if the content is short.

Actual results (实际的结果)

It has the cursor: pointer style, but clicking it has no effect.

Remarks (补充说明)

If the content is usually short but sometimes long, it will be confusing to have the cursor: pointer style.

@github-actions github-actions bot added the untriaged need to sort label Oct 8, 2021
@07akioni 07akioni added bug Something isn't working and removed untriaged need to sort labels Oct 8, 2021
@ouuan
Copy link
Contributor Author

ouuan commented Oct 11, 2021

It still has cursor: pointer when :tooltip="false".

https://codesandbox.io/s/naive-ui-ellipsis-short-content-cursor-pointer-forked-m923q?file=/src/App.vue

@07akioni 07akioni reopened this Oct 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants