Re: [jokosher-devel] [PATCH] InstrumentViewer volume slider patch (SVN Rev. 1519)

["Jeff Ratliff" <jefrat gmail com>]

Sorry, don't think I sent this to the list:

The volume slider in this position makes the think of a pan control,
not a volume control. This comes from my experience with physical
sound boards though, and a novice may not have this problem.

You could do a volume icon that pops up a volume slider, a la
rhythmbox and others. If you were really clever, you could find a way
to integrate it with the mute button and take up no additional space.
This is less discoverable though, and would really rely on a good
descriptive icon, or a little mini popup arrow thingy (excuse my
advanced technical jargon). :)

On 22/06/2008, Laszlo Pandy <laszlok2 gmail com> wrote:
> Thanks for the patch Tom. This is really well done, and I think it serves a
> good purpose because currently the volume control is not as discoverable as
> it could be.
>
> However right now the volume slider is underneath the buttons, which looks a
> little weird. Previously the buttons were at the bottom and they formed a
> nice straight line along the bottom of the instrument header. Also it makes
> the instrument lanes a little bigger; bigger than the events are. This is
> not as big a problem, we can certainly resize the height of the events if we
> need to.
>
> I want everyone to take a look at the screenshot:
> http://laszlopandy.com/files/volume-slider.png
>
> Any suggestions for where else we could put it?
>
> (I think there was previously a bug for having a recording monitor
> vertically against the right side of the header, but a volume slider
> requires more width and height than that).
>
> Laszlo
>
>
> Tom Halligan wrote:
> >
> > Coded against SVN Revision 1519
> >
> > Updated version of the instrument volume slider patch. The volume slider
> is now part of the ControlsBox. When creating the controls box, you should
> add a 'includeVolumeSlider=True' to the parameters. For example, the code
> used in InstrumentViewer.py is:
> >
> >    self.controlsBox =
> >
> ControlsBox.ControlsBox(project,mainview,instrument,includeEffects=True,includeVolumeSlider=True)
> >
> >
> > This means the volume slider is an optional control (for example, the
> mixer strip omits the volume slider for obvious reasons).
> >
> > Peace,
> > Tom
> >
> >
> >
> ------------------------------------------------------------------------
> >
> > _______________________________________________
> > jokosher-devel-list mailing list
> > jokosher-devel-list gnome org
> >
> http://mail.gnome.org/mailman/listinfo/jokosher-devel-list
> >
>
> _______________________________________________
> jokosher-devel-list mailing list
> jokosher-devel-list gnome org
> http://mail.gnome.org/mailman/listinfo/jokosher-devel-list
>


-- 
Jeff Ratliff (Gomer_X)