2002971 - zaburon.com - Textarea line breaks are poorly managed

Status: RESOLVED FIXED

(Web Compatibility :: Site Reports, defect)

Description

[antonellodaguanno2]

User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:145.0) Gecko/20100101 Firefox/145.0

Steps to reproduce:

go to this website:
https://zaburon.com/pagina1.php

Write the following text inside the text area (the text is just an example):
L'Inter perde l'imbattibilità. E subisce la prima sconfitta in Champions. Dopo aver tenuto testa e impensierito i padroni di casa dell'Atletico Madrid, i nerazzurri si sono dovuti piegare al 92esimo: 2-1.

Gli spagnoli sono andati in vantaggio del primo tempo con Julian Alvaraz. Il pareggio è arrivato nel secondo con Zielinski. Poi la beffa al 93esimo con Gimenez

submit the page using "invia" button and then go back to the page using the "torna alla pagina precedente button"

Actual results:

The text became something like:
L'Inter perde l'imbattibilità. E subisce la prima

sconfitta in Champions. Dopo aver tenuto testa e

impensierito i padroni di casa dell'Atletico

Madrid, i nerazzurri si sono dovuti piegare al

92esimo: 2-1.

Gli spagnoli sono andati in

vantaggio del primo tempo con Julian Alvaraz. Il

pareggio è arrivato nel secondo con Zielinski.
Poi
la beffa al 93esimo con Gimenez

Expected results:

The text should stay unchanged:
L'Inter perde l'imbattibilità. E subisce la prima sconfitta in Champions. Dopo aver tenuto testa e impensierito i padroni di casa dell'Atletico Madrid, i nerazzurri si sono dovuti piegare al 92esimo: 2-1.

Gli spagnoli sono andati in vantaggio del primo tempo con Julian Alvaraz. Il pareggio è arrivato nel secondo con Zielinski. Poi la beffa al 93esimo con Gimenez

Comment 1

[antonellodaguanno2]

The same problem appears on Firefox for windows

Comment 2

[Alice0775 White]

This seems a half regression.

The former build: Word wraps are ignored in "Hai inserito:" section after clicking "invia" button.
The later build: "Hai inserito:" section appears properly. But this bug appears.

https://hg-edge.mozilla.org/integration/autoland/pushloghtml?fromchange=c76d70d64667325472186ec7eb540dae1bdd34b8&tochange=06a08ba182a152aa63ef32bf5a2e12f3f21377b0

Comment 3

[BugBot [:suhaib / :marco]]

:emilio, since you are the author of the regressor, bug 1568313, could you take a look? Also, could you set the severity field?

For more information, please visit BugBot documentation.

Comment 4

[Emilio Cobos Álvarez [:emilio]]

I'm confused, at least the form submission seems to be working as intended, right? It's getting wrapped at 50 chars.

What does the "Torna alla pagina precedente" button do? Can you attach the source code of those two pages here using the "Attach New File" button?

Comment 5

[antonellodaguanno2]

Attachment: pagina1.php

Comment 6

[antonellodaguanno2]

Attachment: pagina2.php

Comment 7

[antonellodaguanno2]

Pagina1.php is the first page with the "invia" button
Pagina2.php is the second page with the "Torna alla pagina precedente" button

The "Torna alla pagina precedente" is a submit button that send back the same text received in the POST variable from the textarea to the previous page.

The text is not changed inside the pages (or at list it should be left unchanged).

Comment 8

[Alice0775 White]

paste the following texts into textarea of Pagina1.php

0.2.4.6.8.0.2.4.6.8.0.2.4.6.8.
0.2.4.6.8.0.2.4.6.8.0.2.4.6.8.
0.2.4.6.8.0.2.4.6.8.0.2.4.6.8.

then "invia"
Hai inserito:

0.2.4.6.8.0.2.4.6.8.0.2.4.6.8.

0.2.4.6.8.0.2.4.6.8.0.2.4.6.8.

0.2.4.6.8.0.2.4.6.8.0.2.4.6.8.

is this as expected? I think something wrong. Extra line breaks are added at the end of each line after bug 1568313.

Chrome returns
Hai inserito:

0.2.4.6.8.0.2.4.6.8.0.2.4.6.8.
0.2.4.6.8.0.2.4.6.8.0.2.4.6.8.
0.2.4.6.8.0.2.4.6.8.0.2.4.6.8.

I filed a Bug 2003039.

Comment 9

[antonellodaguanno2]

Please try with some real text and click on invia and then on the other button, repeat this a couple of times and you will see the text break lines totally messed up.
For Example start with this text:
Lorem Ipsum is simply dummy text of the printing and typesetting industry. Lorem Ipsum has been the industry's standard dummy text ever since the 1500s, when an unknown printer took a galley of type and scrambled it to make a type specimen book. It has survived not only five centuries, but also the leap into electronic

Click on Invia and you get the right text but if you click on "Torna alla pagiona precedente" and you get:

`Lorem Ipsum is simply dummy text of the printing

and typesetting industry. Lorem Ipsum has been
the
industry's standard dummy text ever since the

1500s, when an unknown printer took a galley of

type and scrambled it to make a type specimen

book. It has survived not only five centuries,
but
also the leap into electronic`

Please note that some <br/> appears in the code but also some lines are broken in a weird position (for example the: Lorem Ipsum has been
the <<<<-------- See this line break here
industry's standard dummy text ever since the).

Comment 10

[Emilio Cobos Álvarez [:emilio]]

Ah, so it's likely newline normalization getting messed up and causing us to wrap... Let me see...

Comment 11

[Emilio Cobos Álvarez [:emilio]]

Attachment: Standalone test-case

Yeah I agree that wrapping what's already correctly wrapped shouldn't add extra lines.

Comment 12

[Emilio Cobos Álvarez [:emilio]]

Attachment: Bug 2002971 - Remove trailing newlines when outputting a hard-wrapped line. r=smaug

Comment 13

[BugBot [:suhaib / :marco]]

Set release status flags based on info from the regressing bug 1568313

Comment 14

[Pulsebot]

Pushed by [e-mail]:
https://github.com/mozilla-firefox/firefox/commit/eb9ba190413e
https://hg.mozilla.org/integration/autoland/rev/20f34e644e1c
Remove trailing newlines when outputting a hard-wrapped line. r=smaug

Comment 15

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/56345 for changes under testing/web-platform/tests

Comment 17

[Norisz Fay [:noriszfay]]

https://hg.mozilla.org/mozilla-central/rev/20f34e644e1c

Comment 18

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Upstream PR merged by moz-wptsync-bot

Comment 19

[BugBot [:suhaib / :marco]]

The patch landed in nightly and beta is affected.
:emilio, is this bug important enough to require an uplift?

• If yes, please nominate the patch for beta approval.
  • See https://wiki.mozilla.org/Release_Management/Requesting_an_Uplift for documentation on how to request an uplift.
• If no, please set status-firefox146 to wontfix.

For more information, please visit BugBot documentation.