Lyrics box should be improved

Bug #1021970 reported by Devid Antonio Filoni
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
BeatBox
Fix Released
Low
Scott Ringwelski

Bug Description

ATM lyrics box is ugly, you should add few pixels of padding. Also I should be able to copy a lyric so I can share it in my friends and I should be able to correct bad lyrics :)

Revision history for this message
Victor Martinez (victored) wrote :

Confirmed under Ambiance. We need to discuss how we want to implement that editable lyrics field.

Changed in beat-box:
status: New → Confirmed
importance: Undecided → Low
milestone: none → 0.7
Revision history for this message
Kathen Brewster (booiscute) wrote :

I think there should be a faster way to access the lyrics, like a "Show Lyrics" menu option when you right click on a song.

Revision history for this message
Scott Ringwelski (sgringwe) wrote :

I can confirm that it needs padding, but I would say no to making the "Now playing view" lyrics box editable. It would be better IMO to add a button to edit the currently playing song.

Revision history for this message
Devid Antonio Filoni (d.filoni) wrote :

I wrote an HTML "concept" to show you how I would adjust info view, it's too confusing ATM. Please note that I wrote this in 20 minutes so sorry for the applied style and for the code quality. It also needs discussion as I think several things can be improved.

Revision history for this message
Scott Ringwelski (sgringwe) wrote :

Hey, not bad. A few thoughts:

You've grouped information by song, album and artist. When i made the now playing view, I had similar thoughts to do that, but in the end I decided to 'flatten' out the data as much as possible. I did this because navigating through hierarchy is annoying, especially for a view that is meant for purely entertainment purposes. It's much easier and takes less effort just to scroll and lets the user have a nice general overview of everything. Afterall, there isn't THAT much data.

Another weak point with this page is if the user has no internet connection, which leaves you with only the song metadata, album art, and album songs. In this case, you have the album songs list in an awkward 1-page notebook.

Overall though, pretty good.

Revision history for this message
Devid Antonio Filoni (d.filoni) wrote :

About information: I got two ideas, the first one was about a page like wikipedia with the fixed navigation box (so you can move fast between section), the other one is the one I wrote. I prefer the second one because I'm pretty sure people will look for artists and albums tabs few times than songs tab (which contains lyrics) and has you have to scroll less the switch between tabs is intuitive and fast :)

About lyrics: I think it's a good idea to have text in colums so you don't have to scroll when singing your favourite song :)

About missing content: you only have to hide tabs or sections :)

About artists photos: I forgot to add this one but it can be done as it is ATM (after artist albums) with a box and the horizontal scrollbar

About artists albums: my first idea was to create something such as http://www.jcoverflip.com/demo (I love its style) but I didn't add that to my HTML page because I don't know if it can be done easily in GTK+ (although it should be easy than Apple coverflow)

Changed in beat-box:
status: Confirmed → Fix Committed
assignee: nobody → Scott Ringwelski (sgringwe)
Changed in beat-box:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Bug attachments

Remote bug watches

Bug watches keep track of this bug in other bug trackers.