Skip to content

[css] Fix breaking layout breaking error in code block#28328

Merged
XhmikosR merged 5 commits into
twbs:masterfrom
wackerm:patch-1
Jul 16, 2019
Merged

[css] Fix breaking layout breaking error in code block#28328
XhmikosR merged 5 commits into
twbs:masterfrom
wackerm:patch-1

Conversation

@wackerm

@wackerm wackerm commented Feb 20, 2019

Copy link
Copy Markdown
Contributor

Problem: Code block does not break long lines

see miminal example

Was before:
word-break: break-word;
Should be (in my Opinion):
word-wrap: break-word;

@wackerm wackerm requested a review from a team as a code owner February 20, 2019 16:59
@wackerm

wackerm commented Feb 20, 2019

Copy link
Copy Markdown
Contributor Author

This change could/should also be done with current master...

The break-word option for word-break ist not supported by all browsers (for e.g. Firefox), see , the break-word option with word-wrap is supported by all major browsers see and does fix the issue.

@XhmikosR XhmikosR changed the base branch from v4-dev to master February 20, 2019 17:44
@XhmikosR XhmikosR changed the base branch from master to v4-dev February 20, 2019 17:44
@XhmikosR

Copy link
Copy Markdown
Member

Please retarget against master.

@XhmikosR XhmikosR added the css label Feb 20, 2019
@wackerm wackerm changed the base branch from v4-dev to master February 20, 2019 18:22
Was before:
word-break: break-word;
Should be (in my Opinion):
word-wrap: break-word;
@wackerm

wackerm commented Feb 20, 2019

Copy link
Copy Markdown
Contributor Author

Done: retarget to master branch

@XhmikosR

Copy link
Copy Markdown
Member

Friendly ping @twbs/css-review

@XhmikosR

Copy link
Copy Markdown
Member

Should this be backported to v4?

@XhmikosR XhmikosR added the v5 label Jul 16, 2019
@wackerm

wackerm commented Jul 16, 2019

Copy link
Copy Markdown
Contributor Author

I think that would be great since not everyone will upgrade existing v4 projects...
Shall i submit this patch as new PR against v4?

@XhmikosR

Copy link
Copy Markdown
Member

Nah, if we are going to backport it, I'll cherrypick it.

@MartijnCuppens

Copy link
Copy Markdown
Member

Should this be backported to v4?

Yup, it's a bug

@XhmikosR XhmikosR merged commit 78c9c8b into twbs:master Jul 16, 2019
XhmikosR pushed a commit that referenced this pull request Jul 16, 2019
Change `word-break: break-word;` to `word-wrap: break-word;`
@XhmikosR XhmikosR added the v4 label Jul 16, 2019
@wackerm wackerm deleted the patch-1 branch July 16, 2019 11:14
XhmikosR pushed a commit that referenced this pull request Jul 17, 2019
Change `word-break: break-word;` to `word-wrap: break-word;`
XhmikosR pushed a commit that referenced this pull request Jul 17, 2019
Change `word-break: break-word;` to `word-wrap: break-word;`
@mdo mdo mentioned this pull request Jul 22, 2019
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.

3 participants