Skip to content

refactor: replace title-attribute for removable tag with DBTooltip #4218

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

Merged
merged 26 commits into from
May 26, 2025

Conversation

nmerget
Copy link
Collaborator

@nmerget nmerget commented May 16, 2025

Proposed changes

closes #4203

That for introduced variant="description"|"label" property for the DBTooltip component, that controls whether tooltip content is either connected as a complementary description or as a label (overwriting the inner, visible content).

Types of changes

  • Bugfix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Refactoring (fix on existing components or architectural decisions)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation Update (if none of the other choices apply)

Further comments

@nmerget nmerget requested a review from mfranzke as a code owner May 16, 2025 08:34
Copy link
Contributor

🔭🐙🐈 Test this branch here: https://db-ux-design-system.github.io/core-web/review/fix-remove-tag-title

@mfranzke mfranzke self-assigned this May 16, 2025
@mfranzke mfranzke added the 👩‍👧‍👦communityFeedback Feedback from community label May 16, 2025
@mfranzke mfranzke moved this to 👀 In review in UX Engineering Team Backlog May 16, 2025
@mfranzke mfranzke moved this from 👀 In review to ready for release in UX Engineering Team Backlog May 16, 2025
@mfranzke mfranzke added this to the 2.1.0 milestone May 16, 2025
mfranzke
mfranzke previously approved these changes May 16, 2025
@mfranzke mfranzke added this to Roadmap May 16, 2025
@nmerget nmerget enabled auto-merge (squash) May 16, 2025 12:36
@mfranzke
Copy link
Collaborator

@nmerget this one is "ready for release", let's not activate "auto-merge".

@mfranzke mfranzke disabled auto-merge May 16, 2025 12:38
@mfranzke mfranzke changed the title refactor: replace title for remove tag with tooltip refactor: replace title-attribute for remove tag with DBTooltip May 22, 2025
@mfranzke mfranzke changed the title refactor: replace title-attribute for remove tag with DBTooltip refactor: replace title-attribute for removable tag with DBTooltip May 22, 2025
@mfranzke mfranzke enabled auto-merge (squash) May 22, 2025 20:09
@mfranzke
Copy link
Collaborator

mfranzke commented May 23, 2025

@nmerget one pipeline step is still failing for this issue.

@nmerget nmerget requested a review from mfranzke May 26, 2025 05:42
mfranzke
mfranzke previously approved these changes May 26, 2025
# Conflicts:
#	packages/components/src/components/tag/tag.lite.tsx
@nmerget nmerget requested a review from mfranzke May 26, 2025 08:22
@nmerget nmerget disabled auto-merge May 26, 2025 08:35
@nmerget nmerget enabled auto-merge (squash) May 26, 2025 08:35
@nmerget nmerget merged commit 64081ef into main May 26, 2025
73 checks passed
@nmerget nmerget deleted the fix-remove-tag-title branch May 26, 2025 15:18
@github-project-automation github-project-automation bot moved this from ready for release to ✅ Done in UX Engineering Team Backlog May 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: No status
Status: ✅ Done
Development

Successfully merging this pull request may close these issues.

Title 'Remove tag' in DBTag component
2 participants