Skip to content

Conversation

@gcolpart
Copy link

@gcolpart gcolpart commented Nov 5, 2024

Fix #9677

@pabzm
Copy link
Member

pabzm commented Nov 7, 2024

Thank you! While trying to write a test I came up with a test message body, that isn't fixed by your change:

EsseevenietharumattotaminciduntesseQuiitaquenullarepellatRepellendusidvoluptatemcorruptiassumendaofficiaaetDolorembeataelaborumasperioresratione.

OptiodoloremtotamfugiatblanditiisexercitationemcorruptinecessitatibusoccaecatiNemodelenitiquiafugitutrepellendusFacilisteneturautidfacereRepellendusatquesitminimaminimaculpateneturidreiciendisVoluptasdignissimosdignissimosquodvoluptatibusexplicaboSedaliasasperioresmaioresoccaecatietnemo.

AliasquisquaminsedVeniamvelitconsecteturquitemporeanimiutautsapienteFugitteneturvoluptasquiaetfacere.

(It's just Lopem Ipsum without blanks.)

For these lines to be printed in full length, I have to use @media print{body{overflow-wrap: anywhere}}.

Could you please show which kind of text is printed properly with your suggested change?

And could you check if my CSS suggestion also fixes your example text?

@github-actions
Copy link

@pabzm, @alecpl
🛎️ This PR has had no activity in two weeks.

@pabzm
Copy link
Member

pabzm commented Nov 22, 2024

@gcolpart Are you still interested in this?

@gcolpart
Copy link
Author

Hello,

Thank you! While trying to write a test I came up with a test message body, that isn't fixed by your change:
[…]
(It's just Lopem Ipsum without blanks.)
For these lines to be printed in full length, I have to use @media print{body{overflow-wrap: anywhere}}.
Could you please show which kind of text is printed properly with your suggested change?

My patch is to fix issue #9677 for very long lines with spaces when I select zoom 200% in print settings., like :

Optiodolor emtotamfugiat blanditiisexercitat ionemcorruptinecessita tibusoccaecatiNemo delenitiquiafu gitutrepellendusFac ilisteneturautidf acereRepellendu satquesitminimamin imaculpatenetu ridreiciendisVolup tasdignissim osdignissi mosquodvoluptatib usexplicaboSedaliasas perioresma ioresoccae catietnemo.

It doesn't fix for very long lines without blanks .

And could you check if my CSS suggestion also fixes your example text?

I test your patch, it fixes a bug when printing with zoom 100% with very long lines without blanks

BUT :

@github-actions
Copy link

@pabzm, @alecpl
🛎️ This PR has had no activity in two weeks.

@alecpl
Copy link
Member

alecpl commented Apr 21, 2025

This does not fix the issue at all. And it does not make sense. I don't see how min-width of 240px would be related to anything like that.

@alecpl alecpl closed this Apr 21, 2025
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.

Elastic: "print" view truncates lines

3 participants