Closed Bug 3116 Opened 26 years ago Closed 25 years ago

ex and em units are way off

Categories

(Core :: CSS Parsing and Computation, defect, P2)

x86
Windows 95
defect

Tracking

()

VERIFIED FIXED

People

(Reporter: dbaron, Assigned: peterl-retired)

References

()

Details

The ex and em units don't seem to be resizing themselves to the font size of
the parent on these pages.  All the images show up the same height.
Status: NEW → ASSIGNED
Assignee: troy → peterl
Status: ASSIGNED → NEW
Component: Layout → Style System
Peter, this looks like a style system problem. Just looking at this much smaller
case, when I get the style info for the image I'm told its height is only 170
twips. That's why we end up with such a small image

<html lang="en-US">
<head>
<base href="http://www.fas.harvard.edu/~dbaron/csstest/">
<style type="text/css">
IMG { height: 1em; vertical-align: bottom; }
SPAN.hl { line-height: 1em; background-color: yellow; vertical-align: bottom; }
.one { font-size: 50px; font-family: Verdana, Arial, sans-serif; }
</style>
</head>
<body>
<p class="one">
  <span class="hl">M</span><img src="sq_small.jpg" alt="[Squirrel Image]"><span
class="hl">M</span>
</p>
</body></html>
The problem has to do with inheritance of the font-size property.  If I stick
an explicit "font-size: inherit" on the image, things are OK.

Once the inheritance is fixed, you should probably reassign the bug to someone
else, because the sizing is still a little bit off with inherit put in
(although it could be line-height that has the size error).
The cause of the inheritance problem is the font-size: 8.5pt in ua.css, which
you use to set the size of the ALT text.

ALT text should probably be the same size as regular text.  In fact, it should
not be constrained to the image size.  See bug 1994.

If you insist on doing ALT text the way you do now, you probably need an
internal pseudo-element for it.

This bug prevents the use of em/ex units on images without font-size: inherit,
which is a serious problem.  It's something that you probably will find on the
web already.
Status: NEW → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
Changed alt text sizing rule in ua.css (now only applies font size to text
within images, not the image itself). The images are now the right size, but
they still don't line up with the text backgrounds. That's a seperate issue.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.