Closed Bug 894908 Opened 11 years ago Closed 11 years ago

Minor profile cleanup tasks in preparation for launch

Categories

(Participation Infrastructure :: Phonebook, defect)

defect
Not set
normal

Tracking

(Not tracked)

VERIFIED FIXED
2013-07-25

People

(Reporter: hoosteeno, Assigned: julia)

Details

(Whiteboard: [kb=1047049])

Attachments

(2 files)

A handful of things we'd like to fix before launching:

* on edit, make "when did you get involved" go from 1998-current_year
* on edit, add borders around input boxes on profile edit
* on edit, get rid of day on "when did you get involved"
* on edit, _try_ to keep font from being clipped top/bottom in input boxes
* on edit, remove "preferred language" element
* on edit, make "language" block look like skills/groups blocks
* on edit, change "other languages you speak" to "languages you speak"
* on edit, somehow make profile delete button a little less easy/fun to click
* on edit, the "visible to" label on groups/skills is a different font style from the others
* on view, get rid of the space before the comma when there are multiple groups
Whiteboard: [kb=1047049]
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Attached image qa - reopened
Reopening:

Each requirement in comment 0 looks great (has been met) except for one.

* on view, get rid of the space before the comma when there are multiple groups
- see attached screenshot

Note these look great:
* on edit, make "when did you get involved" go from 1998-current_year
* on edit, add borders around input boxes on profile edit
* on edit, get rid of day on "when did you get involved"
* on edit, _try_ to keep font from being clipped top/bottom in input boxes
* on edit, remove "preferred language" element
* on edit, make "language" block look like skills/groups blocks
* on edit, change "other languages you speak" to "languages you speak"
* on edit, somehow make profile delete button a little less easy/fun to click
* on edit, the "visible to" label on groups/skills is a different font style from the others
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Commits pushed to master at https://github.com/mozilla/mozillians

https://github.com/mozilla/mozillians/commit/cd530880cbb0c1a87b21bd3b346c301bb882f2ab
[fix bug 894908] Get rid of the space before the comma when there are multiple groups.

https://github.com/mozilla/mozillians/commit/6904dbc739b97e9ada551fedd45ae935515c3488
Merge pull request #564 from glogiotatidis/mozilla/master

[fix bug 894908] Get rid of the space before the comma when there are multiple groups.
Status: REOPENED → RESOLVED
Closed: 11 years ago11 years ago
Resolution: --- → FIXED
Attached image qa - verified on stage
QA verified on stage - woot ya. Thanks Giorgos, spaces before commas removed.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: