[vlc-devel] (no subject)

Ron Wright logiconcepts819 at gmail.com
Mon Mar 11 00:26:16 CET 2013


Unfortunately, the "spaces" are not removed when rendered.  Here are URLs
of screenshot images of the equalizer panel that demonstrate the rationale
behind my fixes:

Before (as it is right now in trunk):  http://tinypic.com/r/2u7nn7b/6
After (with posted patch):  http://tinypic.com/r/2qa7rfd/6
After (with just the alignment issue fixed):  http://tinypic.com/r/ix87xs/6


On Sun, Mar 10, 2013 at 11:58 AM, Francois Cartegnie <fcvlcdev at free.fr>wrote:

> Le 10/03/2013 17:54, Ron Wright a écrit :
> > What I meant to say was that the bands in the actual equalizer module do
> > not need to be changed, but in the GUI, the strings corresponding to the
> > two lowest ISO frequencies are simply too long and need to be shortened
> in
> > some fashion.  Otherwise, I could submit an alternative patch that fixes
> > only the alignment issue (caused by the extra spaces in the 1 kHz and 2
> kHz
> > labels) because this issue is much more important.
>
> We don't need "spaces" or to change text at all.
>
> foreach( slider )
>         computemaxwidth();
> foreach( slider )
>         setMinimumSize( maxwidth, h );
>
> Francois
> _______________________________________________
> vlc-devel mailing list
> To unsubscribe or modify your subscription options:
> http://mailman.videolan.org/listinfo/vlc-devel
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20130310/72c5715d/attachment.html>


More information about the vlc-devel mailing list