1966955 - Ultimate guitar tabs are copied with extra newlines

Status: RESOLVED FIXED

(Web Compatibility :: Site Reports, defect, P2)

Description

[Greg Tatum [:gregtatum]]

Steps to reproduce:

• Go to a tab: https://tabs.ultimate-guitar.com/tab/misc-computer-games/the-legend-of-zelda-ocarina-of-time-fairy-fountain-tabs-1508906
• Highlight a section with a text selection.
• Copy the text via cmd+c
• Paste into a plain text editor

Results in Chrome:

e|--------------------------------------------------------------------|
B|--10-------8-------7-------8-------8-------6-------5-------6--------|
G|-------7-------7-------7-------7-------5-------5-------5-------5----|
D|-----8---8---8---8---8---8---8---8---7---7---7---7---7---7---7---7--|
A|--------------------------------------------------------------------|
E|--------------------------------------------------------------------|

Results in Safari:

e|--------------------------------------------------------------------|
B|--10-------8-------7-------8-------8-------6-------5-------6--------|
G|-------7-------7-------7-------7-------5-------5-------5-------5----|
D|-----8---8---8---8---8---8---8---8---7---7---7---7---7---7---7---7--|
A|--------------------------------------------------------------------|
E|--------------------------------------------------------------------|

Results in Firefox

e|--------------------------------------------------------------------|

B|--10-------8-------7-------8-------8-------6-------5-------6--------|

G|-------7-------7-------7-------7-------5-------5-------5-------5----|

D|-----8---8---8---8---8---8---8---8---7---7---7---7---7---7---7---7--|

A|--------------------------------------------------------------------|

E|--------------------------------------------------------------------|

Expected results: The copied guitar tabs should only have one space between the lines, and not two, similar to Chrome and Safari.

Comment 1

[Sean Feng [:sefeng211]]

Looks like there's no trivial fix at the moment.

Comment 2

[Kagami Rosylight [:saschanaz] (they/them)]

This is because there's \r\n in <pre>, I think there's an existing bug...

Edit: Oh, comment #1.

Comment 3

[Greg Tatum [:gregtatum]]

Ah interesting. In Chrome if I look in DevTools at $0.innerText it contains the \r\n in the text. When I run copy($0.innerText) in Chrome the \r values are stripped out. So it appears to be normalizing the text while copying. Could this actually be trivially fixed by handling this normalization step in the copy handler code?

Comment 4

[Edgar Chen [:edgar]]

Bug 1650720 has been fixed and is included in today's Nightly build (2025-11-11).
I could not reproduce this anymore on Nightly. Hi Greg, would you mind helping to confirm as well? Thanks!

Comment 5

[Hsin-Yi Tsai (she/her)[:hsinyi]]

(In reply to Edgar Chen [:edgar] from comment #4)

Bug 1650720 has been fixed and is included in today's Nightly build (2025-11-11).
I could not reproduce this anymore on Nightly. Hi Greg, would you mind helping to confirm as well? Thanks!

I can also confirm this is fixed.

Comment 6

[Greg Tatum [:gregtatum]]

Thanks for the fix! I can confirm it's working as well. Now to just get good enough at guitar to play Zelda songs to my heart's content.