Skip to content

Use icon-only dismiss button for repeatable delete#26030

Closed
AmoghParmar wants to merge 1 commit intojenkinsci:masterfrom
AmoghParmar:fix-18321-repeatable-dismiss
Closed

Use icon-only dismiss button for repeatable delete#26030
AmoghParmar wants to merge 1 commit intojenkinsci:masterfrom
AmoghParmar:fix-18321-repeatable-dismiss

Conversation

@AmoghParmar
Copy link
Contributor

This change updates the repeatable delete button to use an icon-only design,
removing the visible “Delete/Dismiss” text.

The goal is to align repeatable delete actions with existing Jenkins UI patterns
(e.g. dismiss buttons used in system notifications), while preserving
accessibility.

Changes included:

  • Remove visible text from the repeatable delete button
  • Keep tooltip text for discoverability
  • Add aria-label to ensure screen reader accessibility

No functional behavior is changed; clicking the button continues to remove the
repeated block as before.

Note: When hovering over the icon-only button, the button width briefly collapses
due to existing CSS assumptions that previously relied on the presence of text.
This PR focuses only on removing visible text while maintaining accessibility.
Any CSS refinements related to hover width can be addressed separately.

Fixes #18321

@github-actions
Copy link
Contributor

github-actions bot commented Jan 2, 2026

Missing required label for changelog. Requires at least 1 of: bug, developer, dependencies, internal, localization, major-bug, major-rfe, rfe, regression-fix, removed, skip-changelog. Found: .

You can add the required label by adding a comment with the following text: /label <category>

@AmoghParmar
Copy link
Contributor Author

AmoghParmar commented Jan 2, 2026

/label rfe

@NotMyFault
Copy link
Member

Duplicate of #25964

Please check for existing PRs before creating a new one.

@NotMyFault NotMyFault closed this Jan 2, 2026
@daniel-beck
Copy link
Member

Fixes #18321

It doesn't.

Also please do not remove the PR template.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[JENKINS-69787] Dismiss buttons in Admin monitors should use an (x) for dismissing

3 participants