Comments

Gauravmahlawat created an issue. See original summary.

gauravvvv’s picture

StatusFileSize
new256 bytes

I have provided the patch please verify if it is working or not.

gauravvvv’s picture

Status: Active » Needs review
gauravvvv’s picture

StatusFileSize
new436.35 KB

I am adding a after patch screenshot for reference.

meghasharma’s picture

Assigned: Unassigned » meghasharma
meghasharma’s picture

Assigned: meghasharma » Unassigned
StatusFileSize
new289 bytes
new223 bytes

i thing, we should add both properties,
word-wrap: break-word;
overflow-wrap: break-word;
because: The “partial” support indicated above is due to some browsers supporting word-wrap and some browsers supporting overflow-wrap. Using both can ensure backwards compatibility.
https://css-tricks.com/almanac/properties/o/overflow-wrap/

gauravvvv’s picture

Patch is working on different cross browser, RTBC +1

ambuj_gupta’s picture

Assigned: Unassigned » ambuj_gupta
ambuj_gupta’s picture

StatusFileSize
new128.86 KB
new267.1 KB

Testing Steps:
1. Install Drupal 8.9
2. Install theme from https://www.drupal.org/node/3023165/git-instructions/8.x-1.x/nonmaintainer
3. Go to Article Contents type. And add the long tag
4. Save the Article and check the view page
5. Verify the page in cross-browsers.

Testing Results:
- Tested and verified by applying the patch updated_css_file-3172151-6.patch. And its working as expected.
- Verified in Safari, Chrome, Firefox and Edge.

ambuj_gupta’s picture

Assigned: ambuj_gupta » Unassigned
Status: Needs review » Reviewed & tested by the community