Closed
Bug 1380492
Opened 7 years ago
Closed 7 years ago
Kerning defect in Gmail compose window
Categories
(Core :: CSS Parsing and Computation, defect)
Core
CSS Parsing and Computation
Tracking
()
RESOLVED
FIXED
mozilla56
| Tracking | Status | |
|---|---|---|
| firefox56 | --- | fixed |
People
(Reporter: ekr, Assigned: emilio)
References
(Blocks 1 open bug)
Details
Attachments
(3 files)
Without Stylo
STR:
1. Go to gmail
2. Start composing
3. Type the first letter of the address
Defect:
Insufficient space between the To and the name.
|
Reporter | |
Comment 1•7 years ago
|
||
|
|
Reporter | |
Updated•7 years ago
|
Blocks: stylo-site-issues
|
||
Comment 2•7 years ago
|
||
Nice catch :-)
|
Assignee | |
Comment 4•7 years ago
|
||
Test-case:
<style>
div {
display: none;
padding: -3px;
}
</style>
<div id="target"></div>
<script>
alert(getComputedStyle(target).paddingTop);
</script>
Stylo alerts -3px, incorrectly.
| Comment hidden (mozreview-request) |
|
|
||
Comment 6•7 years ago
|
||
| mozreview-review | ||
Comment on attachment 8885959 [details]
Bug 1380492: Test.
https://reviewboard.mozilla.org/r/156746/#review161862
Nice.
Attachment #8885959 -
Flags: review?(bobbyholley) → review+
|
||
Comment 7•7 years ago
|
||
What would happen for `padding-left: calc(-3px)`? Would it be clamped properly during computation?
| Comment hidden (mozreview-request) |
|
|
Assignee | |
Comment 9•7 years ago
|
||
(In reply to Xidorn Quan [:xidorn] UTC+10 from comment #7)
> What would happen for `padding-left: calc(-3px)`? Would it be clamped
> properly during computation?
Yes, as far as I can tell... I'll test it though.
|
||
Comment 10•7 years ago
|
||
Pushed by ecoal95@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/aa8e8b3f4ce1
Test. r=bholley
|
|
Assignee | |
Comment 11•7 years ago
|
||
(In reply to Xidorn Quan [:xidorn] UTC+10 from comment #7)
> What would happen for `padding-left: calc(-3px)`? Would it be clamped
> properly during computation?
The following properly alerts 0px:
<!doctype html>
<style>
div {
padding: 3px;
padding: calc(-3px);
display: none;
}
</style>
<div></div>
<script>
alert(getComputedStyle(document.querySelector('div')).paddingTop);
</script>
|
||
Comment 12•7 years ago
|
||
| bugherder | ||
Status: NEW → RESOLVED
Closed: 7 years ago
status-firefox56:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla56
You need to log in
before you can comment on or make changes to this bug.
Description
•