Created attachment 78842 [details] [review]
ruler patch

nice one :) Due to xmas and all I won't be able to review/commit before tuesday. In fact, if you're up to it, you could add the actual values for every 'second' tick, so you know which position it represents.
Something like hh:mm:ss would be nice.

Thanks a lot, and merry xmas :)

hi edward! hmm... I can't figure exactly how that would fit, but perhaps it will show itself if one experiments.

btw, I tried to use the mailinglist, but sourceforge seems a bit broken as usual and my mail didn't seem to get delivered. perhaps move the mailinglist to something more usable?

Created attachment 78915 [details] [review]
New patch

New patch that shows the time also. Lot's of if-statements to get that right...

Cleaned up and commited the patch.

It might still need some more loving though. Like maybe using some bold fonts for some key-points (like the minutes or the hours), in the same way the tics are bigger for minutes or 10s markers.



2006-12-28  Ernst Persson  <ernstp@gmail.com>

	reviewed by: Edward Hervey  <edward@fluendo.com>

	* pitivi/ui/ruler.py:
	Make the ruler nicer. Add time text position.