Skip to content

Add text-underline Utility class, closing #23880#23881

Closed
PascaleBeier wants to merge 10 commits intotwbs:v4-devfrom
PascaleBeier:feature/text-underline
Closed

Add text-underline Utility class, closing #23880#23881
PascaleBeier wants to merge 10 commits intotwbs:v4-devfrom
PascaleBeier:feature/text-underline

Conversation

@PascaleBeier
Copy link
Copy Markdown

Adding a Decoration block for utilites and adding .text-underline

Can be used to add the other text-decoration utilties.

@andresgalante
Copy link
Copy Markdown
Collaborator

sorry @PascaleBeier but I wouldn't like to see this merged. We need to take really good care about not turning utility classes a 1:1 reflection of every CSS property.

@PascaleBeier
Copy link
Copy Markdown
Author

@andresgalante I guess one had to decide where to draw the line right? Interesting to see.

@Johann-S
Copy link
Copy Markdown
Member

Johann-S commented Sep 8, 2017

Agreed with @andresgalante but it's @mdo who will have the final word

@XhmikosR XhmikosR requested a review from mdo September 17, 2017 21:54
@mdo
Copy link
Copy Markdown
Member

mdo commented Oct 19, 2017

I don't see us adding this one, but I would consider a .text-decoration-none or .no-underline utility added for that use case. Far more common than adding an underline via class.

@mdo mdo closed this Oct 19, 2017
swrobel added a commit to swrobel/bootstrap that referenced this pull request Mar 24, 2018
mdo pushed a commit that referenced this pull request Jul 25, 2018
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.

4 participants