Mirrors/servo
1
0
Fork 0
mirror of https://github.com/servo/servo.git synced 2025-06-23 16:44:33 +01:00
Code Issues Projects Releases Packages Wiki Activity

Fix Font::measure_text() to actually compute a bounding box width.

Browse source
This commit is contained in:
Brian J. Burg 2012-10-16 16:43:22 -07:00
parent 22af9770b8
commit c545dd901b
1 changed files with 2 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

5
src/servo/text/font.rs
View file

@ -55,12 +55,11 @@ pub impl Font : FontMethods {
// TODO: alter advance direction for RTL
// TODO(Issue #98): using inter-char and inter-word spacing settings when measuring text
let mut advance = au(0);
let mut bounds = Rect(Point2D(au(0), -self.metrics.ascent),
Size2D(au(0), self.metrics.ascent + self.metrics.descent));
do run.glyphs.iter_glyphs_for_range(offset, length) |_i, glyph| {
advance += glyph.advance();
bounds = bounds.translate(&Point2D(glyph.advance(), au(0)));
}
let mut bounds = Rect(Point2D(au(0), -self.metrics.ascent),
Size2D(advance, self.metrics.ascent + self.metrics.descent));
// TODO(Issue #125): support loose and tight bounding boxes; using the
// ascent+descent and advance is sometimes too generous and