[mythtv] [PATCH] Fix delete program popup size in 0.9.1
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.
More information about the mythtv-dev