Bug 359835

Summary:	[patch] add snap-to-grid for the dvd menu wizard
Product:	[Applications] kdenlive	Reporter:	Michael Vogt <michael.vogt>
Component:	User Interface & Miscellaneous	Assignee:	Jean-Baptiste Mardelle <jb>
Status:	RESOLVED FIXED
Severity:	wishlist
Priority:	NOR
Version First Reported In:	unspecified
Target Milestone:	---
Platform:	Other
OS:	Linux
Latest Commit:		Version Fixed/Implemented In:
Sentry Crash Report:
Attachments:	snap-to-grid with a grid size of 10px Add UI in the dvd wizard to enable/disable the grid option

Description Michael Vogt 2016-02-26 18:23:00 UTC

I really like the dvd creation wizard and the fact that I can add "buttons" there to jump to chapters.

However I find that positioning this aligned is a bit of a challenge. E.g. when I build a list like

chapter1
chapter2

aligning them exactly is tricky. Therefore I implemented a snap-to-grid feature so that it easier to align them.

Reproducible: Always

Comment 1 Michael Vogt 2016-02-26 18:24:56 UTC

Created attachment 97563 [details]
snap-to-grid with a grid size of 10px

Initial version of the snap-to-grid. If you like it I would look into overlaying the grid points and (maybe?) into a toggle to turn the grid on-and-off.

Comment 2 Michael Vogt 2016-02-26 20:13:22 UTC

Created attachment 97567 [details]
Add UI in the dvd wizard to enable/disable the grid option

This adds the missing UI from my previous patch.

Fwiw, I only tested on 15.08.1, git master segfauled for me for what appears to be unreleated reasons. But I'm not faimilar enough with the kdenlive development to know if its a real issue or just something strange with my local system :)

Please let me know what you think, happy to adjust patches as needed for coding conventions etc.

Comment 3 Michael Vogt 2016-02-26 20:22:45 UTC

The patches are all against kdenlive git. There is some churn in the ".ui" file. I think that was designer-qt4 doing some stuff I did not ask it to do. I just added the QCheckBox "use_grid".

Comment 4 Jean-Baptiste Mardelle 2016-02-26 20:50:59 UTC

Thanks for your patch. It has been merged in git master, where I also just fixed a crash.
https://quickgit.kde.org/?p=kdenlive.git&a=commit&h=f7bb11459b7c6729b46917c3987c26351a0d445a