[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