[kdenlive] [Bug 457878] Kdenlive ignores more than one space in the subtitles.
https://bugs.kde.org/show_bug.cgi?id=457878 Jean-Baptiste Mardelle changed: What|Removed |Added Latest Commit|https://invent.kde.org/mult |https://invent.kde.org/mult |imedia/kdenlive/commit/61c7 |imedia/kdenlive/commit/6e6d |35cd77b191440533a967511bb14 |06dd2d010ea88160783bc89f751 |749e16017 |c25a868cd --- Comment #6 from Jean-Baptiste Mardelle --- Git commit 6e6d06dd2d010ea88160783bc89f751c25a868cd by Jean-Baptiste Mardelle, on behalf of Eric Jiang. Committed on 05/09/2022 at 06:37. Pushed by mardelle into branch 'release/22.08'. Don't remove consecutive spaces in SRT subtitles The Kdenlive UI allows you to add multiple spaces between words in subtitles. This displays correctly in the UI and is preserved when saving the SRT file, but the spaces are removed when opening the SRT file. Changing line.simplified() to line.trimmed() seems to fix the round-trip issue. M +1-1src/bin/model/subtitlemodel.cpp A +4-0tests/dataset/multiple-spaces.srt https://invent.kde.org/multimedia/kdenlive/commit/6e6d06dd2d010ea88160783bc89f751c25a868cd -- You are receiving this mail because: You are watching all bug changes.
[kdenlive] [Bug 457878] Kdenlive ignores more than one space in the subtitles.
https://bugs.kde.org/show_bug.cgi?id=457878 Eric Jiang changed: What|Removed |Added Latest Commit||https://invent.kde.org/mult ||imedia/kdenlive/commit/61c7 ||35cd77b191440533a967511bb14 ||749e16017 Status|ASSIGNED|RESOLVED Resolution|--- |FIXED --- Comment #5 from Eric Jiang --- Git commit 61c735cd77b191440533a967511bb14749e16017 by Eric Jiang. Committed on 23/08/2022 at 05:50. Pushed by mardelle into branch 'master'. Don't remove consecutive spaces in SRT subtitles The Kdenlive UI allows you to add multiple spaces between words in subtitles. This displays correctly in the UI and is preserved when saving the SRT file, but the spaces are removed when opening the SRT file. Changing line.simplified() to line.trimmed() seems to fix the round-trip issue. M +1-1src/bin/model/subtitlemodel.cpp A +4-0tests/dataset/multiple-spaces.srt M +8-0tests/subtitlestest.cpp https://invent.kde.org/multimedia/kdenlive/commit/61c735cd77b191440533a967511bb14749e16017 -- You are receiving this mail because: You are watching all bug changes.
[kdenlive] [Bug 457878] Kdenlive ignores more than one space in the subtitles.
https://bugs.kde.org/show_bug.cgi?id=457878 --- Comment #4 from Eric Jiang --- The merge request linked above has not been merged yet. If/when it has been merged, you would be able to test it in a nightly build or another later build that contains it. -- You are receiving this mail because: You are watching all bug changes.
[kdenlive] [Bug 457878] Kdenlive ignores more than one space in the subtitles.
https://bugs.kde.org/show_bug.cgi?id=457878 --- Comment #3 from q...@vp.pl --- kdenlive: ver. 22.08.0 ver. 22.11.70 (rev. 448dc5b73) Still many spaces is ignored!. -- You are receiving this mail because: You are watching all bug changes.
[kdenlive] [Bug 457878] Kdenlive ignores more than one space in the subtitles.
https://bugs.kde.org/show_bug.cgi?id=457878 Eric Jiang changed: What|Removed |Added Assignee|vpi...@kde.org |erji...@alumni.iu.edu -- You are receiving this mail because: You are watching all bug changes.
[kdenlive] [Bug 457878] Kdenlive ignores more than one space in the subtitles.
https://bugs.kde.org/show_bug.cgi?id=457878 Bug Janitor Service changed: What|Removed |Added Status|CONFIRMED |ASSIGNED --- Comment #2 from Bug Janitor Service --- A possibly relevant merge request was started @ https://invent.kde.org/multimedia/kdenlive/-/merge_requests/340 -- You are receiving this mail because: You are watching all bug changes.
[kdenlive] [Bug 457878] Kdenlive ignores more than one space in the subtitles.
https://bugs.kde.org/show_bug.cgi?id=457878 Eric Jiang changed: What|Removed |Added CC||erji...@alumni.iu.edu Ever confirmed|0 |1 Status|REPORTED|CONFIRMED --- Comment #1 from Eric Jiang --- Confirmed on latest master (74ef83e0): when adding subs with many spaces, kdenlive saves all of the spaces into the srt file. When opening the kdenlive project again, the spaces are lost and the srt file is overwritten, losing the spaces. -- You are receiving this mail because: You are watching all bug changes.
