Unverified Commit 88f75712 authored by Tomasz Gucio's avatar Tomasz Gucio Committed by GitHub

Remove irrelevant comment in TextPainter (#144308)

This PR removes an irrelevant comment in `TextPainter` for `_computePaintOffsetFraction`. Also some typos are corrected and missing spaces/newlines added.

test-exempt: no functional change
parent d00bfe82
...@@ -331,10 +331,10 @@ class _TextLayout { ...@@ -331,10 +331,10 @@ class _TextLayout {
/// (text.length, downstream), unless maxLines is set to a non-null value, in /// (text.length, downstream), unless maxLines is set to a non-null value, in
/// which case the caret is placed at the visual end of the last visible line. /// which case the caret is placed at the visual end of the last visible line.
/// ///
/// This should not be called when the paragraph is emtpy as the implementation /// This should not be called when the paragraph is empty as the implementation
/// relies on line metrics. /// relies on line metrics.
/// ///
/// When the last bidi level run in the paragraph and the parargraph's bidi /// When the last bidi level run in the paragraph and the paragraph's bidi
/// levels have opposite parities (which implies opposite writing directions), /// levels have opposite parities (which implies opposite writing directions),
/// this makes sure the caret is placed at the same "end" of the line as if the /// this makes sure the caret is placed at the same "end" of the line as if the
/// line ended with a line feed. /// line ended with a line feed.
...@@ -1045,6 +1045,7 @@ class TextPainter { ...@@ -1045,6 +1045,7 @@ class TextPainter {
} }
ui.Paragraph _getOrCreateLayoutTemplate() => _layoutTemplate ??= _createLayoutTemplate(); ui.Paragraph _getOrCreateLayoutTemplate() => _layoutTemplate ??= _createLayoutTemplate();
/// The height of a space in [text] in logical pixels. /// The height of a space in [text] in logical pixels.
/// ///
/// Not every line of text in [text] will have this height, but this height /// Not every line of text in [text] will have this height, but this height
...@@ -1305,8 +1306,6 @@ class TextPainter { ...@@ -1305,8 +1306,6 @@ class TextPainter {
return isLowSurrogate(prevCodeUnit) ? offset - 2 : offset - 1; return isLowSurrogate(prevCodeUnit) ? offset - 2 : offset - 1;
} }
// Get the caret metrics (in logical pixels) based off the trailing edge of the
// character upstream from the given string offset.
static double _computePaintOffsetFraction(TextAlign textAlign, TextDirection textDirection) { static double _computePaintOffsetFraction(TextAlign textAlign, TextDirection textDirection) {
return switch ((textAlign, textDirection)) { return switch ((textAlign, textDirection)) {
(TextAlign.left, _) => 0.0, (TextAlign.left, _) => 0.0,
...@@ -1357,8 +1356,10 @@ class TextPainter { ...@@ -1357,8 +1356,10 @@ class TextPainter {
final TextBox textBox = _getOrCreateLayoutTemplate().getBoxesForRange(0, 1, boxHeightStyle: ui.BoxHeightStyle.strut).single; final TextBox textBox = _getOrCreateLayoutTemplate().getBoxesForRange(0, 1, boxHeightStyle: ui.BoxHeightStyle.strut).single;
return textBox.toRect().height; return textBox.toRect().height;
} }
bool _isNewlineAtOffset(int offset) => 0 <= offset && offset < plainText.length bool _isNewlineAtOffset(int offset) => 0 <= offset && offset < plainText.length
&& WordBoundary._isNewline(plainText.codeUnitAt(offset)); && WordBoundary._isNewline(plainText.codeUnitAt(offset));
// Cached caret metrics. This allows multiple invokes of [getOffsetForCaret] and // Cached caret metrics. This allows multiple invokes of [getOffsetForCaret] and
// [getFullHeightForCaret] in a row without performing redundant and expensive // [getFullHeightForCaret] in a row without performing redundant and expensive
// get rect calls to the paragraph. // get rect calls to the paragraph.
...@@ -1464,7 +1465,7 @@ class TextPainter { ...@@ -1464,7 +1465,7 @@ class TextPainter {
.getBoxesForRange(graphemeRange.start, graphemeRange.end, boxHeightStyle: ui.BoxHeightStyle.strut); .getBoxesForRange(graphemeRange.start, graphemeRange.end, boxHeightStyle: ui.BoxHeightStyle.strut);
if (boxes.isNotEmpty) { if (boxes.isNotEmpty) {
final TextBox box = boxes.single; final TextBox box = boxes.single;
metrics =_LineCaretMetrics( metrics = _LineCaretMetrics(
offset: Offset(anchorToLeadingEdge ? box.start : box.end, box.top), offset: Offset(anchorToLeadingEdge ? box.start : box.end, box.top),
writingDirection: box.direction, writingDirection: box.direction,
); );
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment