Skip to content

fix(tile): prevent padding change on hover when disabled#14196

Merged
kodiakhq[bot] merged 2 commits into
carbon-design-system:mainfrom
janhassel:14191
Jul 13, 2023
Merged

fix(tile): prevent padding change on hover when disabled#14196
kodiakhq[bot] merged 2 commits into
carbon-design-system:mainfrom
janhassel:14191

Conversation

@janhassel

Copy link
Copy Markdown
Member

Closes #14191

The component reset of in the link styles are causing an overwrite of padding: 0 for disabled clickable tiles.

Changelog

Changed

  • Added the appropriate padding definition to the .#{$prefix}--tile--clickable:hover.#{$prefix}--link--disabled selector in _tile.scss

Testing / Reviewing

see issue for reproduction steps

@janhassel janhassel requested a review from a team as a code owner July 12, 2023 08:47
@netlify

netlify Bot commented Jul 12, 2023

Copy link
Copy Markdown

Deploy Preview for carbon-components-react ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit 4a63ccc
🔍 Latest deploy log https://app.netlify.com/sites/carbon-components-react/deploys/64afac3419e045000896a943
😎 Deploy Preview https://deploy-preview-14196--carbon-components-react.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@netlify

netlify Bot commented Jul 12, 2023

Copy link
Copy Markdown

Deploy Preview for carbon-elements ready!

Name Link
🔨 Latest commit 4a63ccc
🔍 Latest deploy log https://app.netlify.com/sites/carbon-elements/deploys/64afac34c1a8f40008294e24
😎 Deploy Preview https://deploy-preview-14196--carbon-elements.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@kodiakhq kodiakhq Bot merged commit f20a3a4 into carbon-design-system:main Jul 13, 2023
@janhassel janhassel deleted the 14191 branch July 14, 2023 08:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: Disabled Tile padding dissapears on hover

3 participants