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

TD-1300 Label Chip in Detail Card #803

Conversation

jegasriskantha
Copy link
Contributor

@jegasriskantha jegasriskantha commented Jan 5, 2024

contributes to: https://sce.myjetbrains.com/youtrack/issue/TD-1300/Clicking-the-label-chips-does-not-do-anything

raised in PR here: https://github.com/IPG-Automotive-UK/virto/pull/434#pullrequestreview-1805702144

Changes

  • updated label chip in the detail card to not be clickable.

UI/UX

Confirmed with @Sowbhagya-ipg that the label should not be clickable in the detail card : https://github.com/IPG-Automotive-UK/virto/pull/434#issuecomment-1878623486

Testing notes

  • check that the label chips are no longer clickable in the detail card component.

Author checklist before assigning a reviewer

  • Reviewed my own code-diff.
  • [ ] Branch has been run in docker.
  • PR assigned to me or an appropriate delegate.
  • Relevant labels added to the PR.
  • [ ] Appropriate tests have been added.
  • Lint and test workflows pass.

@jegasriskantha jegasriskantha added the bug Something isn't working label Jan 5, 2024
@jegasriskantha jegasriskantha self-assigned this Jan 5, 2024
@jegasriskantha jegasriskantha requested a review from evoinea January 5, 2024 13:14
Copy link
Contributor

@evoinea evoinea left a comment

Choose a reason for hiding this comment

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

LGTM

@jegasriskantha jegasriskantha merged commit 21bc801 into main Jan 8, 2024
3 checks passed
@jegasriskantha jegasriskantha deleted the bug/TD-1300-prevent-label-chip-clickable-in-detail-card-view branch January 8, 2024 12:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants