New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

New kind of display for the albums list #241

Closed
lazka opened this Issue Mar 14, 2015 · 18 comments

Comments

Projects
None yet
7 participants
@lazka
Member

lazka commented Mar 14, 2015

Original issue 241 created by brunobergot on 2009-06-24T17:40:51.000Z:

Hi and sorry for my poor english,

I really like the look and feel of the albums list. This list can be very
long when you have a lot of albums, and generally i like to search an album
only by scanning the covers "au hasard".

I think it would be great to allow users to modifiy some settings about the
display of this list :

  • show textuals informations or not
  • show this information when the mouse hover a cover
  • size of the covers

Some years ago i was using a plugin called album list for winamp. I would
love to have this type of display in the album list of QL.

Thanks for reading

@lazka

This comment has been minimized.

Member

lazka commented Mar 14, 2015

Comment #1 originally posted by steven.strobe.cc on 2009-06-25T16:25:11.000Z:

Wow, somebody actually likes Cover Flow? :)

I think improving startup performance is important, and for technical reasons this
feature would interfere with that goal. (Fixed-size thumbnails of covers will be
cached in a future release, making scaling to an arbitrary size more difficult.) Your
feature will be considered after those startup optimizations are in place.

@lazka

This comment has been minimized.

Member

lazka commented Mar 14, 2015

Comment #2 originally posted by reiter.christoph on 2009-06-25T16:42:55.000Z:

/browsers/albumicons.py is something like that, but in an early stage it seems:

http://666kb.com/i/ba3woceskk4mjci0x.png

Imho if that can be included into the album view and if there are mouseover hints
like in the treeview this could be quite useful..

@lazka

This comment has been minimized.

Member

lazka commented Mar 14, 2015

Comment #3 originally posted by brunobergot on 2009-06-26T14:02:35.000Z:

wow this style of display is great ! that's exactly the kind o thing i'm looking for.
I'll give it a try very soon. Thanks for pointing me this one :D

@brunob

This comment has been minimized.

Contributor

brunob commented Mar 16, 2015

Work is on in my fork here : https://bitbucket.org/b_b_/quodlibet

@brunob

This comment has been minimized.

Contributor

brunob commented Apr 18, 2015

@brunob

This comment has been minimized.

Contributor

brunob commented Apr 18, 2015

See also #1489 (in order to centralize issues about this feature).

@Lalopx

This comment has been minimized.

Lalopx commented Aug 7, 2015

Is there a way to change the album view, or add the option to the right clivk menu?? i would love to help with some coding. Also, hello to evryone 👍

@darthoctopus

This comment has been minimized.

Collaborator

darthoctopus commented Oct 26, 2016

@brunob's fork seems to have died. I've taken the liberty of getting covergrid to work with 3.7.1, but only after brutally commenting out a bunch of calls to GtkIconView.set_cursor. Would you be open to a pull request for covergrid as-is?

@brunob

This comment has been minimized.

Contributor

brunob commented Oct 26, 2016

@darthoctopus i haven't touched to my fork since a while cause i never knowed if it would be merge one day... I think it would be better to ask to @lazka to have a look at this, maybe a pull request of your fork would suit to him.

@nodiscc

This comment has been minimized.

nodiscc commented Oct 26, 2016

Duplicate of #1101?
Related #1665.

@brunob

This comment has been minimized.

Contributor

brunob commented Oct 26, 2016

@nodiscc #1101 is the duplicate of the one present here ;)

@nodiscc

This comment has been minimized.

nodiscc commented Oct 26, 2016

I've just drafted what an album art based view should look at in my humble opinion. Feedback welcome.

ql-albugrid
ql-hover

Clicking anywhere on an album art selects it and display details in the left pane.
Selecting multiple albums displays 'N albums selected in the left pane, along with play all/enqueue all buttons.

@nodiscc

This comment has been minimized.

nodiscc commented Oct 26, 2016

About #1665, a variant of this view could be used

  • Display a top-level artists list/images grid instead of directly an album grid
  • Upon clicking on the artist, the album grid for this artist is displayed, along with a Back button to clear the selected artist.
  • Selecting an album works as described above.
@darthoctopus

This comment has been minimized.

Collaborator

darthoctopus commented Oct 26, 2016

well @brunob already did the heavy lifting to implement this (with less fancy), I've been using it on a regular basis for a while now and thought it would be a shame not to merge this to master, is all. Right now we show the songlist in an RVPaned display (i.e. horizontal instead of vertical split) but I guess that can be left to some config variable (not sure how to implement though).

That said I'd rather get @brunob's code merged in first before looking to implement more features, I find it fairly usable (and also impressive) as it stands and would hate to see it go to waste.

@brunob

This comment has been minimized.

Contributor

brunob commented Oct 26, 2016

@darthoctopus i also like to see this one merged, since i asked for it in 2099 and passed a lot of time on this last year (this was clearly not easy for me since it was the first time i made ptyhon + gtk code ^^).

If i remember correctly, @lazka had some doubts about the performances of this display, but QL core surely have evolved since my last commits, and all this may need a big update.

@urielz

This comment has been minimized.

Contributor

urielz commented Oct 26, 2016

just writing to express that as a QL user this is one of the missing features that I would like to see implemented as well. so +1 to get it merged.

@declension

This comment has been minimized.

Member

declension commented Nov 24, 2016

Can we close this ticket too now?

@brunob

This comment has been minimized.

Contributor

brunob commented Nov 24, 2016

I think so.

@declension declension closed this Nov 24, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment