Skip to content

change whitespace from normal to pre-line#22

Open
christopherpickering wants to merge 1 commit into
CreativeBulma:masterfrom
christopherpickering:patch-1
Open

change whitespace from normal to pre-line#22
christopherpickering wants to merge 1 commit into
CreativeBulma:masterfrom
christopherpickering:patch-1

Conversation

@christopherpickering
Copy link
Copy Markdown

For multiline tooltips it would be nice to accept a multi line input and preserve line breaks.
pre-line will keep the source line breaks.

For multiline tooltips it would be nice to accept a multi line input and preserve line breaks.
pre-line will keep the source line breaks.
@christopherpickering
Copy link
Copy Markdown
Author

@wikiki any change on merging?

@suterma
Copy link
Copy Markdown

suterma commented Mar 16, 2024

While this works well as intended, I would not personally like this to be the default. In my Vue app, the linter might choose to break text lines (or I do for readability) and I am happy to have the default as is.

That said, I would suggest to make this either optional with a class like 'has-tooltip-multiline-pre-line' or better yet, with a new scss variable like $tooltip-whitespace

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.

2 participants