Closed Bug 910886 Opened 11 years ago Closed 11 years ago

style award details page

Categories

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

defect

Tracking

(Not tracked)

RESOLVED FIXED
2013Q3

People

(Reporter: willkg, Assigned: rehandalal+mozilla)

References

Details

(Whiteboard: u=contributor c=badges p=2 s=2013.18 [needsverify])

Attachments

(1 file, 1 obsolete file)

Bug #896137 adds badge infrastructure to Kitsune.

This bug covers restyling the award details page which shows the details for a Badge that has been awarded to a user.

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

That template needs styling.
Also, this needs a mobile template.
Blocks: 897057
Priority: -- → P2
Target Milestone: --- → 2013Q3
We can hold off on the mobile template if it adds a lot of time to this.

Estimating at ~2pts as well although that is probably at the high end. In for next sprint.
Whiteboard: u=contributor c=badges p= s=2013.backlog → u=contributor c=badges p=2 s=2013.18
Assignee: nobody → rdalal
Kadir requested screenshots before landing.
Whiteboard: u=contributor c=badges p=2 s=2013.18 → u=contributor c=badges p=2 s=2013.18 [needsverify]
Attached image award_details.png (obsolete) —
See screenshot attached. Is that okay?
Flags: needinfo?(a.topal)
Attached image award_details.png
should fix the screenshot
Attachment #807608 - Attachment is obsolete: true
I think this looks lovely!
This looks great. Just one question: Are we storing who awarded a badge? Would be great to put it on this page if we do, but it was not a requirement, so it's not a blocker. Also, one nit: We should either capitalize all headings or not, here we have "Awarded Badge" and "Badge detail".
Flags: needinfo?(rtanglao)
Flags: needinfo?(rardila)
Flags: needinfo?(mana)
Flags: needinfo?(a.topal)
I'll fix the capitalization in the headers.

As mentioned on IRC we do store the info (optional) on who awarded the badge and when stored it will be displayed.
Landed:
https://github.com/mozilla/kitsune/commit/671265e53d05a9ad25f0dc9e4b1a543efe3a8cae

Pushed to prod by me, right now.
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Flags: needinfo?(mana)
Flags: needinfo?(rtanglao)
Flags: needinfo?(rardila)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: