refactor: Simplify cursor-pointer mixin for inputs#6269
refactor: Simplify cursor-pointer mixin for inputs#6269
cursor-pointer mixin for inputs#6269Conversation
|
| cursor: initial; | ||
| } | ||
|
|
||
| &:is(input[type="checkbox"]), |
There was a problem hiding this comment.
@nmerget within https://github.com/db-ux-design-system/core-web/blob/refactor-simplify-cursor-pointer-mixin-for-inputs/packages/components/src/components/tag/tag.scss this mixin isn't used, that property is being set directly within that file https://github.com/db-ux-design-system/core-web/blob/refactor-simplify-cursor-pointer-mixin-for-inputs/packages/components/src/components/tag/tag.scss#L146
Nevertheless do we need these lines of code at all, as we're even already setting cursor: var(--db-overwrite-cursor, pointer) via line 16 ?
There was a problem hiding this comment.
So we need some other logic for DBTag with a checkbox included. The checkbox gets cursor:auto on hover and active
Proposed changes
:isselector by merging two declarationscursor: pointer, as this is even already the default.Types of changes
Further comments
🔭🐙🐈 Test this branch here: https://design-system.deutschebahn.com/core-web/review/refactor-simplify-cursor-pointer-mixin-for-inputs