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.

Sign up for GitHub

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

Jump to bottom

list-item button type and default cursor #5045

Closed
kkachniarz220 opened this issue Oct 2, 2023 · 0 comments · Fixed by #5051 or #5005 · May be fixed by X-oss-byte/pigweed#61
Closed

list-item button type and default cursor #5045

kkachniarz220 opened this issue Oct 2, 2023 · 0 comments · Fixed by #5051 or #5005 · May be fixed by X-oss-byte/pigweed#61
Assignees
@asyncLiz

Comments

@kkachniarz220
Copy link
Contributor

kkachniarz220 commented Oct 2, 2023 •
edited

What is affected?

Component

Description

<md-list-item type=button component has default cursor .

Reproduction

https://lit.dev/playground/#project=W3sibmFtZSI6Im1hdGVyaWFsLWltcG9ydHMuanMiLCJjb250ZW50IjoiaW1wb3J0IFwiQG1hdGVyaWFsL3dlYi9saXN0L2xpc3QuanNcIjtcbmltcG9ydCBcIkBtYXRlcmlhbC93ZWIvbGlzdC9saXN0LWl0ZW0uanNcIjsifSx7Im5hbWUiOiJpbmRleC5odG1sIiwiY29udGVudCI6IjwhRE9DVFlQRSBodG1sPlxuPHNjcmlwdCB0eXBlPVwibW9kdWxlXCIgc3JjPVwiLi9tYXRlcmlhbC1pbXBvcnRzLmpzXCI-PC9zY3JpcHQ-XG5cbjxtZC1saXN0PiAgXG4gIDxtZC1saXN0LWl0ZW0gdHlwZT1cImJ1dHRvblwiPlxuICAgIDxkaXYgc2xvdD1cImhlYWRsaW5lXCI-dGVzdCBsaXN0IGl0ZW08L2Rpdj5cbiAgPC9tZC1saXN0LWl0ZW0-XG4gICAgPG1kLWxpc3QtaXRlbSB0eXBlPVwiYnV0dG9uXCI-XG4gICAgPGRpdiBzbG90PVwiaGVhZGxpbmVcIj50ZXN0IGxpc3QgaXRlbTwvZGl2PlxuICA8L21kLWxpc3QtaXRlbT5cbjwvbWQtbGlzdD5cbiJ9LHsibmFtZSI6InBhY2thZ2UuanNvbiIsImNvbnRlbnQiOiJ7XG4gIFwiZGVwZW5kZW5jaWVzXCI6IHtcbiAgICBcImxpdFwiOiBcIl4yLjAuMFwiLFxuICAgIFwiQGxpdC9yZWFjdGl2ZS1lbGVtZW50XCI6IFwiXjEuMC4wXCIsXG4gICAgXCJsaXQtZWxlbWVudFwiOiBcIl4zLjAuMFwiLFxuICAgIFwibGl0LWh0bWxcIjogXCJeMi4wLjBcIlxuICB9XG59IiwiaGlkZGVuIjp0cnVlfV0

Workaround

type="link"
href="javascript:void(0);"

Is this a regression?

Yes. This used to work, but now it doesn't.

Affected versions

1.0.0

Browser/OS/Node environment

Chrome 117.0.5938.92
Windows 11
@asyncLiz asyncLiz self-assigned this Oct 2, 2023
copybara-service bot pushed a commit that referenced this issue Oct 3, 2023
@asyncLiz
fix(list): show pointer cursor for button list items
a3f7084 
Fixes #5045

PiperOrigin-RevId: 570479624
@copybara-service copybara-service bot mentioned this issue Oct 3, 2023
Merged
copybara-service bot pushed a commit that referenced this issue Oct 3, 2023
@asyncLiz
fix(list): show pointer cursor for button list items
401f21c 
Fixes #5045

PiperOrigin-RevId: 570479624
@release-please release-please bot mentioned this issue Oct 3, 2023
Merged
@X-oss-byte X-oss-byte mentioned this issue May 24, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@asyncLiz asyncLiz

Labels
None yet
Projects
None yet
Milestone
No milestone
2 participants
@asyncLiz @kkachniarz220