Skip to content

Conversation

@bsastregx
Copy link
Collaborator

@bsastregx bsastregx commented Sep 20, 2024

Changes in this PR:

  • disabled property is now reflected on the host element to improve the disabled styling.

@bsastregx bsastregx added the feature Feature implementation pull request label Sep 20, 2024
@bsastregx bsastregx changed the title [ch-edit] reflect disabled property on ch-edit [ch-edit] reflect disabled property Sep 20, 2024
@netlify
Copy link

netlify bot commented Sep 20, 2024

Deploy Preview for gx-chameleon ready!

Name Link
🔨 Latest commit e3a7a69
🔍 Latest deploy log https://app.netlify.com/sites/gx-chameleon/deploys/66f0b13f357f2e0008d7300c
😎 Deploy Preview https://deploy-preview-422--gx-chameleon.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.

@bsastregx bsastregx requested a review from ncamera September 20, 2024 22:12
@ncamera ncamera changed the title [ch-edit] reflect disabled property [ch-edit] Reflect disabled property to improve disabled styling Sep 23, 2024
@ncamera
Copy link
Collaborator

ncamera commented Sep 23, 2024

@bsastregx
Thanks!

I've added 198 unit tests to test this feature. Take a look at the latest commit to understand the importance of unit testing.

@ncamera ncamera added the target: minor This PR is targeted for the next minor release label Sep 23, 2024
@bsastregx bsastregx merged commit 4d8b41a into main Sep 23, 2024
@bsastregx bsastregx deleted the feat/reflect-disabled-ch-edit branch September 23, 2024 12:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature Feature implementation pull request target: minor This PR is targeted for the next minor release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants