Bug 364556

Summary:	Title editor moves titles around when reopening titles
Product:	[Applications] kdenlive	Reporter:	Stefan Naumann <me>
Component:	User Interface & Miscellaneous	Assignee:	Jean-Baptiste Mardelle <jb>
Status:	RESOLVED FIXED
Severity:	normal	CC:	wegwerf-1-2-3
Priority:	NOR
Version First Reported In:	16.04.0
Target Milestone:	---
Platform:	Arch Linux
OS:	Linux
Latest Commit:		Version Fixed/Implemented In:
Sentry Crash Report:
Attachments:	a project for title-(re)-opening test

Description Stefan Naumann 2016-06-20 18:21:38 UTC

I use very specific formats for titles with several lines and use very mild changes in these titles throughout the video (therefore creating loads of title clips with nearly the same content or with exactly the same format). When duplicating and reopening titles the texts moved some pixels. I inverted the x-axis so it showed me the distance to the right edge of the screen, not the left, maybe that causes the error?

Reproducible: Always

Steps to Reproduce:
1. Prepare titles with several lines of text (one per title-string)
2. Duplicate the title-clip
3. Reopen the title clip

Actual Results:  
The title-strings moved some pixels

Expected Results:  
No movement of the titles is expected

Probably a rounding-error?

Comment 1 Jean-Baptiste Mardelle 2016-06-20 18:34:32 UTC

I cannot duplicate. Could you attach here a simple project file that contains both the original title clip and the duplicate with moved text ?

Comment 2 Stefan Naumann 2016-06-20 19:08:56 UTC

Created attachment 99634 [details]
a project for title-(re)-opening test

Huge project with loads of titles (and other stuff), change on title around, e.g. change the numbers on the bottom of one title and reopen it. Some text should've moved now by some pixels.

Comment 3 Jean-Baptiste Mardelle 2016-06-20 19:16:18 UTC

Thanks, I was able to reproduce the problem with your project file:
selecting a text line moves it... I will investigate.

Comment 4 Jean-Baptiste Mardelle 2016-06-20 21:42:40 UTC

Git commit 33d2a51956bbbca3090a3ca67ec5b0f57064b32d by Jean-Baptiste Mardelle.
Committed on 20/06/2016 at 21:42.
Pushed by mardelle into branch 'master'.

Fix 3 pixel offset when editing text that had inverted x or y axis

M  +3    -3    src/effectstack/graphicsscenerectmove.cpp

http://commits.kde.org/kdenlive/33d2a51956bbbca3090a3ca67ec5b0f57064b32d

Comment 5 Jean-Baptiste Mardelle 2016-06-20 21:46:27 UTC

Ouch, took me a few hours to understand that one... Should be fixed in git master now. As a workaround, can you try the following:

in your title clip, select a text item, enable shadow and set shadow offset to 0x0. After that disable shadow and click ok to save your title. After that, the text should not move anymore when edited/reopening.