Closed
Bug 763833
Opened 14 years ago
Closed 12 years ago
"Featured" banner on add-on pages has poor text rendering quality (bad kerning, inconsistent baselines)
Categories
(addons.mozilla.org Graveyard :: Public Pages, defect, P5)
addons.mozilla.org Graveyard
Public Pages
Tracking
(Not tracked)
RESOLVED
WONTFIX
2014-06
People
(Reporter: jaws, Unassigned)
References
(Depends on 1 open bug, )
Details
Attachments
(1 file)
|
9.33 KB,
image/png
|
Details |
The "Featured" banner on add-on pages has poor text rendering quality. Some of the letters have bad kerning and some letters have lower baselines than others.
I've filed bug 763832 to fix our platform so we can render this text in higher quality, but in the meantime we should find another solution that can show localization-friendly text at a rotated angle.
One possible solution is to do the following:
- Using canvas, draw the localized text in to a canvas.
- Then get the data URI of the generated canvas image and set it as the src of an img.
- Set the "title" attribute (and "alt" too) on the img.
- Continue to use the -moz-transform: rotate(45deg)
This new solution should give us better text rendering since the rotation will only be rotating pixels instead of rotating the baseline that text needs to be drawn upon.
| Reporter | ||
Comment 1•14 years ago
|
||
| Reporter | ||
Comment 2•14 years ago
|
||
This is an example of the solution that I have suggested:
> data:text/html,<style>canvas { width: -moz-fit-content; margin-top: 200px; -moz-transform: rotate(45deg) }</style><canvas/><script>d=document.querySelector('canvas').getContext('2d');d.font='bold 15px "Helvetica Neue"';d.fillText('ALL CAPS Camel Case no caps',50,50)</script>
| Reporter | ||
Comment 3•14 years ago
|
||
(In reply to Jared Wein [:jaws] from comment #2)
> This is an example of the solution that I have suggested:
> > data:text/html,<style>canvas { width: -moz-fit-content; margin-top: 200px; -moz-transform: rotate(45deg) }</style><canvas/><script>d=document.querySelector('canvas').getContext('2d');d.font='bold 15px "Helvetica Neue"';d.fillText('ALL CAPS Camel Case no caps',50,50)</script>
This solution was supplied by Frank Yan.
| Reporter | ||
Updated•14 years ago
|
Updated•12 years ago
|
Target Milestone: --- → 2014-04
Updated•12 years ago
|
Target Milestone: 2014-04 → 2014-05
Updated•12 years ago
|
Target Milestone: 2014-05 → 2014-06
Comment 4•12 years ago
|
||
Thanks for the suggestion. I'm going to wontfix this as the bug has been open for 2 years, is very minor and I'd rather not have to maintain the complex workaround. The real fix is to resolve bug 534064.
Severity: normal → trivial
Status: NEW → RESOLVED
Closed: 12 years ago
Priority: -- → P5
Resolution: --- → WONTFIX
| Assignee | ||
Updated•10 years ago
|
Product: addons.mozilla.org → addons.mozilla.org Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•