[mythtv] [PATCH] Fix delete program popup size in 0.9.1

Jim Paris jim at jtan.com
Sun Jun 22 14:46:44 EDT 2003


> > It won't apply to CVS due to popup changes; also, it looks like you
> > "fixed" the problem by adding an arbitrary constant, which should
> > probably be replaced by subtitle->height().  I'm not running CVS right
> > now so I can't test that.
> 
> It's not quite arbitrary -- it's roughly the amount of padding between the 
> elements in the popup.

For 0.9.1, adding up the ->height() of every element and using
SetMinimumSize instead of SetMaximumSize makes the popup size
perfectly.  Are you sure any additional padding is necessary?
In CVS, you're neglecting to add the ->height() of the subtitle.

-jim


More information about the mythtv-dev mailing list