Re: [Rhythmbox-devel] UI tweeks with mock up



On Tue, 2005-09-20 at 02:31 +0200, Martin Jeppesen wrote:
> I have just submitted a suggestion with mock up for an UI tweek in
> this bug
> http://bugzilla.gnome.org/show_bug.cgi?id=316748

* Moved Search Entry below Track Browser.
> When something is written it effects only the Track Browser, and the
> Search Entry should therefore be close to where the changes occur.

If we get rid of the the Show/Hide Browser widget then this could work,
but I don't really have a strong opinion either way. If we don't remove
the Show/Hide Browser widget then this will cause more vertical space to
get taken up, which I personally would prefer not to do.


* Removed Hide Browser.
> This is an UI change, and fits better in the menubar->View together
> with the other UI changes.

I don't know about other people, but I use the Show/Hide Browser widget
a fair bit - because it is more convenient than using the menu item. If
some other ideas that have been talked about are implemented then the
widget will be used for other sources - e.g. an "inline playlist editor",
basically having the automatic playlist dialog where the library has it's
browsers.

The space that this widget occupied will still be there, but empty, unless
the search box is moved down, the info text is moved out of the status bar
and they are placed adjacent to each other. I doubt that we'd move the text
out of the status bar (see below), and I think that it's worth keeping the
widget if the space will be there anyway.


* Removed Horizontalbar.

This makes sense if both the above changes are made, otherwise it's useful
as a divider between the header (which is visible in small mode) and the
rest.


* Removed Repeat and Shuffle.
> Once the user have setteled for their state, they are not/seldom
> changed again. These are also available in the menubar->Controls.

I agree that they don't belong in the status bar, however I don't
know if we should keep them somewhere in the main UI. There is a patch
in bugzilla[0] which moves then to the toolbar that the patch adds.

One thing I will bring up again, is that we really need a better way of
choosing the play order. The Shuffle/Repeat method only gives you four
choices, of which "shuffle off, repeat off" probably isn't used much, and
not many people know the different between "shuffle on, repeat off" and
"shuffle on, repeat on".

Rhythmbox actually has 7 play orders currently defined, but there is no
UI to choose some of them. A menu would be the obvious method to use, but
thinking up good short descriptions is hard.


* Removed Volume Control.
> As Rhythmbox is a Gnome only program it would be safe to assume, that
> the user have the Volume Control in the Gnome Panel.

Multimedia programs generally have a volume control because people may
want them to be quieter than other programs. Totem also has the same
volume control.


* Removed Size in MB that was left to the Progressbar.
> This is the only place in where file sizes are written, so this
> information does not fit.

Although Rhythmbox doesn't currently burn tracks to normal (data) CDs,
this is handy when you are making a playlist for that - because you
need to ensure that it will fit on the CD. I would probably be a good
idea to show the file size in the track information dialog.


* Moved Progress Slider to the right.
* Moved Text and Progressbar up.
> This effects only the Track Browser, and should therefore be close to
> where the changes occur

CVS has a proper GTK status bar which these are in and the progress bar
is on the right. I don't think moving them to get a tiny bit more
vertical space for the sidebar is really worth it.


* Added Resize Handle to the bottom right corner.

CVS has this - and it only shows up when the window isn't maximised.



my 2c,

James "Doc" Livingston

[0] http://bugzilla.gnome.org/show_bug.cgi?id=316238

--
+-------------------------------------------------------------------+
|-- SELF-ASSEMBLY MOEBIUS-STRIP - SEE OTHER SIDE FOR INSTRUCTIONS --|
+-------------------------------------------------------------------+

Attachment: signature.asc
Description: This is a digitally signed message part



[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]