Skip to content
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

Proposal for improving the presentation of "who likes this dist" #1184

Closed
neilb opened this issue May 16, 2014 · 6 comments
Closed

Proposal for improving the presentation of "who likes this dist" #1184

neilb opened this issue May 16, 2014 · 6 comments

Comments

@neilb
Copy link
Contributor

neilb commented May 16, 2014

Firstly, I like seeing who the likers are.

When a dist has a lot of likes, the top of the page becomes a bit messy, and visually it makes the list of mugshots the thing that draws your attention.

Here's a proposal for making the information less distracting, but still easily accessible:

proposed-metacpan

The basic idea is:

  • All the 'likage' information is off to the right. It's still "headline information", but separate from the author / dist / module block.
  • A maximum number of likers are shown by default.
  • Clicking on the "show all" would drop down a div containing the remaining mugshots, rolling over the existing content rather than pushing it down

Cheers,
Neil

@ranguard
Copy link
Member

Like it - no pun intended :)

I can't remember if there is a mobile UI reason for having the ++ on the left - that would need to be checked

@oalders
Copy link
Member

oalders commented May 19, 2014

Works for me. The "show all" could even bounce over to another page which shows more author details and with pagination etc. Could also be laid out like the contributors page. Eventually the big lists are going to become even bigger, so it may just not be practical for Moose, Perl, etc to be shown on a Pod page.

@majensen
Copy link

Just one voice here, but the likers list drives me crazy. I generally want to see as much distro info on one page as possible. The suggested compromise is a good one IMO. [One question-- what likers get to be displayed first? Could be a random selection each refresh...]

@ranguard
Copy link
Member

That's now live - I prefer @neilbowers design but this will do until someone can submit a pull request

@tobyink
Copy link
Contributor

tobyink commented May 27, 2014

@neilbowers' suggestion included a "show all" link.

The current implementation is great if you want to see which dists ABRAXXA and ADAMJS like, but less good if you're interested in ZOFFIX's preferences.

@ranguard
Copy link
Member

Yes - hence the 'I prefer @neilbowers design' comment :)

Please send a pull request and we'll put it live

oiami pushed a commit to oiami/metacpan-web that referenced this issue Jun 9, 2014
oiami pushed a commit to oiami/metacpan-web that referenced this issue Jun 9, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants