Skip to content

Commit

Permalink
Fix typos made in PR #764
Browse files Browse the repository at this point in the history
  • Loading branch information
june0cho committed Aug 28, 2013
1 parent b75a396 commit 823bbdd
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions src/components/main/layout/inline.rs
Expand Up @@ -713,7 +713,7 @@ impl InlineFlowData {
let line_height = scanner.calculate_line_height(cur_box, em_size);

// Find the top and bottom of the content area.
// Those are used in text-top and text-bottom value of 'vertex-align'
// Those are used in text-top and text-bottom value of 'vertical-align'
let text_ascent = text_box.run.font.metrics.ascent;

// Offset from the top of the box is 1/2 of the leading + ascent
Expand All @@ -734,7 +734,7 @@ impl InlineFlowData {
let mut top_from_base = top_from_base;
let mut bottom_from_base = bottom_from_base;

// To calculate text-top and text-bottom value of 'vertex-align',
// To calculate text-top and text-bottom value of 'vertical-align',
// we should find the top and bottom of the content area of parent box.
// The content area is defined in "http://www.w3.org/TR/CSS2/visudet.html#inline-non-replaced".
// TODO: We should extract em-box info from font size of parent
Expand Down

5 comments on commit 823bbdd

@bors-servo
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

saw approval from metajack
at june0cho@823bbdd

@bors-servo
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

merging june0cho/servo/master = 823bbdd into auto

@bors-servo
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

june0cho/servo/master = 823bbdd merged ok, testing candidate = 4aaaf60

@bors-servo
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@bors-servo
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fast-forwarding master to auto = 4aaaf60

Please sign in to comment.