[Bugs] #3714 Jukebox UNSP: Jukebox-26 design review

[Sugar Labs Bugs]

#3714: Jukebox-26 design review
------------------------------------------+---------------------------------
    Reporter:  garycmartin                |          Owner:  godiard                    
        Type:  enhancement                |         Status:  assigned                   
    Priority:  Unspecified by Maintainer  |      Milestone:  Unspecified by Release Team
   Component:  Jukebox                    |        Version:  Unspecified                
    Severity:  Unspecified                |       Keywords:  design                     
Distribution:  Unspecified                |   Status_field:  Unconfirmed                
------------------------------------------+---------------------------------

Comment(by manuq):

 Replying to [ticket:3714 garycmartin]:
 > Jukebox-26 design review:
 >  * missing help hint for Add to playlist toolbar button

 Yes, needs tooltip.

 {{{
 self.open_button.set_tooltip(_('Add track'))
 }}}

 >  * add to playlist and Remove from playlist should be a matching pair of
 icons -- proposal, use the (+) icon for Add to match the current (-) icon

 Yes, it should be:

 {{{
 self.open_button = ToolButton('list-add')
 }}}

 >  * place a separator after add/remove, before the
 Previous/Play/Next/Scrub slider to help separate the functionality and put
 more space between the destructive remove button and the rewind button

 +1

 >  * missing help hint for Play

 Yes, this icon changes graphic and behaviour, play - pause.  Should the
 hint (tooltip) can be just "Play/Pause" all the time?  Or change
 accordingly? there are methods set_button_play() and set_button_pause()
 that change the graphic, can change the tooltip too.

 >  * "remove selected track from the playlist" not yet translated to
 Spanish & overly complex string -- propose using "Add track" & "Remove
 track" for the strings

 +1

 >  * slider next to the small volume toolbar icon is sub-optimal as the
 (otherwise unlabled) slider looks like it might control volume, where as
 the volume slider is actually hidden in a palette made by the small
 speaker icon (this is an old design flaw).

 As Gonzalo said, maybe we can just remove the volume control, which is
 hidden as you said.  It just changes the global sound anyway, that can be
 changed at the frame.  And maybe we can display the time in numbers at the
 right of the slider.

 >  * place track play head time text with the slider to make its function
 more obvious, something like M:SS -----o-----------------M:SS where the
 first M:SS is the current playhead position in minutes and seconds, and
 the last M:SS is the total track length

 Oh, that's what I was proposing above, great.

 >  * in the View secondary toolbar, swap the fullscreen and Show Playlist
 icons around to put the playlist icon nearer the real view

 +1

 >  * Use a standard Volume icon just after the View toolbar icon, place a
 volume slider in its secondary toolbar, include base and treble sliders if
 possible? Or if base/treble not something we can do, we should consider
 dropping the volume feature altogether, users can already control the
 volume with the keyboard and/or the Speaker device in the Frame, so
 control in Jukebox is only useful if it provides something more.

 I think we can just drop it.

-- 
Ticket URL: <http://bugs.sugarlabs.org/ticket/3714#comment:2>
Sugar Labs <http://sugarlabs.org/>
Sugar Labs bug tracking system