910884 - style the recent awards list page

Status: RESOLVED FIXED

(support.mozilla.org :: General, defect, P2)

Description

[Will Kahn-Greene [:willkg] ET needinfo? me]

Bug #896137 adds badge infrastructure to Kitsune.

That bug adds a view at /awards/ which lists the recently awarded badges.

The template for that view is at kitsune/kbadge/templates/badges/awards_list.html.

That template needs styling and pagination.

Comment 1

[Will Kahn-Greene [:willkg] ET needinfo? me]

Sticking it in the backlog.

One of the things we need to do is figure out whether it's sufficient as it is behavior-wise or whether we want that view to do other things, too. After that, we need to figure out what it should look like. Then this bug is ready to be worked on.

Comment 2

[Will Kahn-Greene [:willkg] ET needinfo? me]

Also, this needs a mobile template.

Comment 3

[Ricky Rosario [:rrosario, :r1cky]]

(In reply to Will Kahn-Greene [:willkg] from comment #2)
> Also, this needs a mobile template.

This is contributor facing, we can do mobile templates later as part of the improving contributor facing pages for mobile "project".

Comment 4

[Ricky Rosario [:rrosario, :r1cky]]

I think this is ~2pts. Putting in next sprint.

Comment 5

[Ricky Rosario [:rrosario, :r1cky]]

Kadir requested screenshots before landing.

Comment 6

[Rehan Dalal (Inactive)]

Attachment: awards_list.png

Comment 7

[Rehan Dalal (Inactive)]

Does this look okay?

Comment 8

[Rehan Dalal (Inactive)]

Attachment: awards_list.png

Comment 9

[Kadir Topal [:atopal]]

Thanks, Rehan, this is awesome. One thing I'd add: The date the badge was awarded on. Maybe below the name? Just the data though, not the time. Since this is the recent awards page, it would be nice to see at a glance how many badges were awarded in like the last day week or so. Just having the dates displayed on this page would be enough.

Comment 10

[Rehan Dalal (Inactive)]

I will try to figure out the best way to display the date on each of those.

FYI: You can't see this in the screen shot but when you mouse over the badge image it displays the name of the badge on top of it.

Comment 11

[Rehan Dalal (Inactive)]

Landed:
https://github.com/mozilla/kitsune/commit/28570ca8eefdb9d4cad36b7e4e7c7e289f504d80

Pushed to prod by me, right now.

Comment 12

[Roland Tanglao needinfo please :rolandtanglao, :mohnkuchen, :adobo, :sinigang, :roland]

since we released this clearing needinfo