Skip to content

Conversation

@saschaszott
Copy link
Contributor

Description

This PR removes redundant nested span elements in various template files. These span elements contained identical content with respect to their parent elements.

@saschaszott saschaszott changed the title Remove nested span elements with duplicate content of parent element Remove nested span elements with duplicate content of parent elements Dec 6, 2024
@tdonohue tdonohue added bug 1 APPROVAL pull request only requires a single approval to merge port to dspace-7_x This PR needs to be ported to `dspace-7_x` branch for next bug-fix release port to dspace-8_x This PR needs to be ported to `dspace-8_x` branch for next bug-fix release labels Dec 6, 2024
@tdonohue tdonohue added this to the 9.0 milestone Dec 6, 2024
@artlowel artlowel self-requested a review December 9, 2024 09:10
Copy link
Member

@artlowel artlowel left a comment

Choose a reason for hiding this comment

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

Thanks @saschaszott!

While this works, and can be merged as is, would you mind also removing those text-muted classes inside the tooltips? That would be another a11y improvement

@tdonohue tdonohue merged commit f24e53f into DSpace:main Jan 7, 2025
12 checks passed
@dspace-bot
Copy link
Contributor

Successfully created backport PR for dspace-7_x:

@dspace-bot
Copy link
Contributor

Successfully created backport PR for dspace-8_x:

@tdonohue tdonohue removed port to dspace-7_x This PR needs to be ported to `dspace-7_x` branch for next bug-fix release port to dspace-8_x This PR needs to be ported to `dspace-8_x` branch for next bug-fix release labels Jan 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

1 APPROVAL pull request only requires a single approval to merge bug

Projects

No open projects
Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

4 participants