Closed
Bug 923499
Opened 12 years ago
Closed 12 years ago
Add word-wrap:break-word; to #profile-details
Categories
(Participation Infrastructure :: Phonebook, defect)
Participation Infrastructure
Phonebook
Tracking
(Not tracked)
VERIFIED
FIXED
2013-10-10
People
(Reporter: hoosteeno, Assigned: tbharath)
Details
(Whiteboard: [kb=1142466] [good first bug][mentor=hoosteeno])
Attachments
(1 file)
102.22 KB,
image/png
|
Details |
We need to keep people from breaking the layout with long strings in their profile fields, esp. bio.
This is an easy fix: Put word-wrap: break word; in the CSS of the #profile-details element.
Screenshot of breakage attached.
Assignee | ||
Comment 1•12 years ago
|
||
Hi hoosteeno, can you assign this to me?
Reporter | ||
Updated•12 years ago
|
Assignee: nobody → bharath_ves
Status: NEW → ASSIGNED
Reporter | ||
Comment 2•12 years ago
|
||
:tbharath, let me know if I can help point the way on this. I'm in #commtools during daylight PDT.
Assignee | ||
Comment 3•12 years ago
|
||
As I already have a pull request sent to mozillians for Bug 922729, I am not sending PR yet, but here is my change for this bug https://github.com/bharaththiruveedula/mozillians/commit/01b7be1498952171052e1c90cfd5481b181f309c
Updated•12 years ago
|
Whiteboard: [good first bug][mentor=hoosteeno] → [kb=1142466] [good first bug][mentor=hoosteeno]
Comment 4•12 years ago
|
||
Commits pushed to master at https://github.com/mozilla/mozillians
https://github.com/mozilla/mozillians/commit/8a7bb2181a57a0b9306ffc29a952269ad6f9bf18
[Fix Bug 923499] Added word-wrap property for #profile-details
https://github.com/mozilla/mozillians/commit/3287aa06890e5228aa59dd689b63eef2584d8f0b
Merge pull request #654 from bharaththiruveedula/Bug923499
[Fix Bug 923499] Added word-wrap property for #profile-details
Updated•12 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Comment 5•12 years ago
|
||
Verified on stage. Profiles look much better and neater thanks to word-wrap goodness. Many thanks, tbharath!
Status: RESOLVED → VERIFIED
Reporter | ||
Updated•12 years ago
|
Target Milestone: --- → 2013-10-10
You need to log in
before you can comment on or make changes to this bug.
Description
•