Skip to content

Commit

Permalink
Make TextInput correctly handle multibyte chars.
Browse files Browse the repository at this point in the history
  • Loading branch information
ema-fox committed Dec 5, 2014
1 parent c6aadc5 commit 0316100
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion components/script/textinput.rs
Expand Up @@ -149,7 +149,7 @@ impl TextInput {

/// Return the length of the current line under the editing point.
fn current_line_length(&self) -> uint {
self.lines[self.edit_point.line].len()
self.lines[self.edit_point.line].char_len()
}

/// Adjust the editing point position by a given of lines. The resulting column is
Expand Down

5 comments on commit 0316100

@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 jdm
at ema-fox@0316100

@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 ema-fox/servo/textinput = 0316100 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.

ema-fox/servo/textinput = 0316100 merged ok, testing candidate = 58f9b4a

@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 = 58f9b4a

Please sign in to comment.