[mythtv-commits] Re: Ticket #667: Fix OSD font width and height for non-4:3 display, letterbox modes

[MythTV]

#667: Fix OSD font width and height for non-4:3 display, letterbox modes
------------------------------+---------------------------------------------
 Reporter:  gtgj at pacbell.net  |        Owner:  danielk
     Type:  patch             |       Status:  new    
 Priority:  minor             |    Milestone:  0.20   
Component:  mythtv            |      Version:  head   
 Severity:  medium            |   Resolution:         
------------------------------+---------------------------------------------
Comment (by gtgj at pacbell.net):

 Thanks for the feedback.

 * I haven't looked much into EIA-708 but I was thinking it would be
 different enough that it would likely use a different (new) OSD container
 (maybe more like what is going to be used for DVB/IVTV teletext?).  If the
 current container will be extended to support EIA-708, I figured we could
 change its boundaries at that time.

 * I don't think I've changed how the OSD dimensions are currently
 calculated, did I?  I was only trying to change the way the fonts were
 displayed and how the text dimensions were calculated.  I added an
 "aspect" ratio parameter that is supposed to specify the target OSD aspect
 ratio so that the fonts can be correctly scaled.  This aspect ratio is
 normally the video aspect ratio but not in certain letterbox modes.  I
 thought XJ_aspect was a better representation but I have to admit I'm not
 sure how certain letterbox modes are supposed to look like.  Is there a
 better aspect ratio to use?

 Also, I thought the displaywidth/height in the OSD class represented the
 OSD dimensions, not the display window size.  I don't think it's even
 aware of the display window size.

 The calculation of "wscale" is supposed to account for non-square pixels
 but I've only tested non-square source video pixels and not non-square
 display pixels.  Is there a good way of testing non-square display pixels?

-- 
Ticket URL: <http://cvs.mythtv.org/trac/ticket/667>
MythTV <http://www.mythtv.org/>
MythTV