Nice patch, just few comments :
Today there is :
Code:
#define ACTION_VSHIFT_UP 227 // shift up video image in DVDPlayer
#define ACTION_VSHIFT_DOWN 228 // shift down video image in DVDPlayer
And recently I've seen this new actions :
Code:
#define ACTION_SUBTITLE_VSHIFT_UP 230 // shift up subtitles in DVDPlayer
#define ACTION_SUBTITLE_VSHIFT_DOWN 231 // shift down subtitles in DVDPlayer
#define ACTION_SUBTITLE_ALIGN
I've read that has already been discussed, but it would be great to have a kind of ACTION_ALIGN with top/bottom/center similar to what ACTION_SUBTITLE_ALIGN does.
Another thing, those _UP / _DOWN actions has very small increments, depending on device you use to control, like a IR remote with some latency, that can be a real pain.
I would 80% of the time put the video at top with subs below if enough space...
And I'm "dreaming" to set such settings on a folder or TV Show basis... or just some default ones (like in advancedsettings.xml)