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

DIG-4807, DIG-5571 #789

Merged
merged 29 commits into from
Jan 9, 2025
Merged

DIG-4807, DIG-5571 #789

merged 29 commits into from
Jan 9, 2025

Conversation

subaha-cob
Copy link
Contributor

DIG-4807 - Add underline to links in CK Editor
DIG-5571 - Styling error in hero image of article page

subaha-cob and others added 29 commits December 9, 2024 09:00
DIG-4807 Content text underline for accessibility
DIG-4807 Content text underline for accessibility
DIG-4807 Content text underline for accessibility
DIG-4807 Content text underline for accessibility
DIG-4807 Content text underline for accessibility
DIG-5470 Added new social media icon, bluesky
DIG-5470 Adding new svg image
DIG-4807 CKEditor text links underline
DIG-4807 Adding underline to CKEditor
DIG-4807 Add underline to links CK Editor
DIG-4807 Add underline to links in CK Editor
DIG-4807 Adding underllinee to linnks in CK editor
@subaha-cob subaha-cob added the bug label Jan 9, 2025
Copy link
Contributor

@davidrkupton davidrkupton left a comment

Choose a reason for hiding this comment

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

Approved, but it looks like you are setting the anchor to be underlined by default for everything, and then turning off where it is not needed? Would it not be easier to maintain this going forwards to just add the underlines where we do want them (paragraphs and node with formatted text fields)?

@subaha-cob subaha-cob merged commit 64abcfe into production Jan 9, 2025
1 check passed
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.

2 participants