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

Isaac Richards ijr at po.cwru.edu
Sun Jun 22 15:12:41 EDT 2003


On Sunday 22 June 2003 01:46 pm, Jim Paris wrote:
> > > 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.

Adding the subtitle helps, yeah. (it makes the added height exactly the amount 
of padding, intead of a bit more).  How the popups work internally has 
changed in CVS, as it no longer adds in the padding automatically.

Isaac


More information about the mythtv-dev mailing list