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

feat(tag): colour token updates #310

Merged
merged 18 commits into from
Jan 9, 2025
Merged

feat(tag): colour token updates #310

merged 18 commits into from
Jan 9, 2025

Conversation

jaideep08
Copy link
Contributor

@jaideep08 jaideep08 commented Dec 20, 2024

Summary

Tag component

  • Color token updates.
  • New clickable variant.

ADO Story or GitHub Issue Link

TASK 2003572

Figma Link

Tag component

Checklist

  • Used Conventional Commit messages as outlined in the contributing guide.
    • Noted breaking changes (if any).
  • Documented/updated all props, events, slots, parts, etc with JSDoc.
    • Ran the analyze command to update Storybook docs.
  • Added/updated Stories with controls to cover all variants.
  • Ran test locally to address any failures.
  • Added/updated the Figma link for the Story's Design tab.
  • Added any new component exports to the src/index.ts file

Copy link

netlify bot commented Dec 20, 2024

Deploy Preview for shidoka-applications ready!

Name Link
🔨 Latest commit bf2a57c
🔍 Latest deploy log https://app.netlify.com/sites/shidoka-applications/deploys/677faa69ad1d470008fe88fb
😎 Deploy Preview https://deploy-preview-310--shidoka-applications.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.

@jaideep08 jaideep08 requested review from a team January 7, 2025 04:59
Copy link
Contributor

@kumarivijeta kumarivijeta left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  1. Show all/show Less color is not matching with Figma
  2. Can we remove shade props? I don't think it is required anymore.

@jaideep08 jaideep08 requested review from kumarivijeta and a team January 7, 2025 10:11
@srpriyankashetty
Copy link

Waiting for response from design team regarding the color when Tag is in pressed state.
Currently seeing hover color when pressed in UI.

* Shade `'light'` (default) and `'dark'` for tag.
*/
@property({ type: String })
shade = 'light';
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We're removing shade variants? Might want to note that as a soft breaking change.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes, we thought of removing it since it is changing already as part of the light and dark theme. Will make a note of it.

src/components/reusable/tag/tag.ts Show resolved Hide resolved
Copy link

@srpriyankashetty srpriyankashetty left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Verified the Tag colors in dark mode
Verified the Tag group in dark mode
Verified the Clickable Tags
Verified Disable Tag color and looks fine

@jaideep08 , observed that close button boarder is bit thicker than Figma.! you can proceed if it's fine from backend.

@jaideep08
Copy link
Contributor Author

Verified the Tag colors in dark mode Verified the Tag group in dark mode Verified the Clickable Tags Verified Disable Tag color and looks fine

@jaideep08 , observed that close button boarder is bit thicker than Figma.! you can proceed if it's fine from backend.

@srpriyankashetty, fixed it.

@jaideep08 jaideep08 merged commit 1c4bf17 into next Jan 9, 2025
8 of 9 checks passed
@jaideep08 jaideep08 deleted the feat/themes3 branch January 9, 2025 10:57
@brian-patrick-3
Copy link
Contributor

🎉 This PR is included in version 2.0.0-next.40 🎉

The release is available on:

Your semantic-release bot 📦🚀

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.

4 participants