Modify
Warning Please read the Ticket HowTo before creating or commenting on a ticket. Failure to do so may cause your ticket to be rejected or result in a slower response.

Opened 16 months ago

Closed 16 months ago

Last modified 15 months ago

#11291 closed Patch - Bug Fix (fixed)

Use the right highlight area in DVD menus when 'wide' is not the first in the list.

Reported by: peper03@… Owned by: stuartm
Priority: minor Milestone: 0.27
Component: MythTV - DVD Playback Version: 0.26-fixes
Severity: medium Keywords:
Cc: Ticket locked: no

Description

Menu sub-pictures (highlights) are never scaled and so up to three can be supplied
with each menu. These are 'wide', 'letterbox' and 'pan&scan'. 'Wide' is used when a 16:9 display is used, 'letterbox' when a 4:3 display should show a 16:9 image and 'pan&scan' when the image should be cropped to fit a 4:3 display full-screen.

Because the co-ordinates of the menu hotspots change depending on which display
method is (potentially) used, these are also given for each method.

Myth already uses the 'wide' sub-picture but the libdvdnav library provides no direct way to retrieve the current highlight area for a given display method. It always returns the first set of co-ordinates found. Most DVDs seem to store the 'wide' co-ordinates first so the highlight area return by libdvdnav is usually correct. As far as I can tell, this is not a requirement. If the order is different, it can be at best ugly (e.g. part of the highlight is cropped) and at worst impossible to visibly navigate a DVD menu (e.g. the highlight is cropped completely).

The attached patch adds functionality to libdvdnav to return the highlight area for a given display method. This functionality could be placed in DVDRingBuffer if preferred as it only requires access to the PCI structure.

The required behaviour was determined (more or less) from these sources:

http://dvd.sourceforge.net/nav_notes
libmythdvdnav/dvdread/nav_types.h

See also the sample 'ButtonGroup?.iso' in stuartm's sample repository.

Attachments (1)

0001-Use-the-right-highlight-area-in-DVD-menus-when-wide-.patch (5.2 KB) - added by peper03@… 16 months ago.

Download all attachments as: .zip

Change History (5)

Changed 16 months ago by peper03@…

comment:1 Changed 16 months ago by Richard <peper03@…>

  • Owner set to Richard <peper03@…>
  • Resolution set to fixed
  • Status changed from new to closed

In 1eaecea6dd38db0c20b94f8ecb549b4c29491297/mythtv:

Use the right highlight area in DVD menus when 'wide' is not the first in the list.

Signed-off-by: Stuart Morgan <smorgan@…>

Fixes #11291

comment:2 Changed 16 months ago by kenni

  • Owner changed from Richard <peper03@…> to stuartm

comment:3 Changed 16 months ago by kenni

  • Milestone changed from unknown to 0.27

comment:4 Changed 15 months ago by Richard <peper03@…>

In 012ebe9de7e20a0bae1920cfc55199ad63937636/mythtv:

Use the right highlight area in DVD menus when 'wide' is not the first in the list.

Signed-off-by: Stuart Morgan <smorgan@…>

Fixes #11291
(cherry picked from commit 1eaecea6dd38db0c20b94f8ecb549b4c29491297)

Add Comment

Modify Ticket

Action
as closed .
The resolution will be deleted. Next status will be 'new'.
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.