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