Skip to content

Commit 581d643

Browse files
soutuameta-codesync[bot]
authored andcommitted
fix(ios): Fix rounding error occurring randomly in iOS text line height calculation (#54260)
Summary: Fixes issue #53450 There are some discussion and explanation about the issue here: #53450 (comment) In short, there seems to be a rounding error occurring with the new architecture's implementation when the text paragraph line heights are calculated on iOS, that happens randomly and depends on what text content the view has, and on the device screen. The fix is basically the same that had been implemented in the React Native's [old architecture 8 years ago](a534672), but hasn't been brought into the new architecture implementation. The fix adds a fraction to the text line dimensions to mitigate the rounding error. ## Changelog: [IOS] [FIXED] - Fix rounding error occurring randomly in iOS text line height calculation Pull Request resolved: #54260 Test Plan: Note that the reproduction of the issue is also dependant on the used device/simulator, this was tested using the iPhone 17 simulator. - Go to Text Layout Example in the RNTester app - Scroll down the text until you see this text paragraph: Without the fix: <img width="400" alt="without-the-fix" src="https://github.com/user-attachments/assets/9a1fe071-be39-4f99-a7e6-25c2fb431fa4" /> With the fix: <img width="400" alt="with-the-fix" src="https://github.com/user-attachments/assets/eb798afc-1d39-4464-a21a-07d95db8ce62" /> Reviewed By: javache Differential Revision: D85438723 Pulled By: cipolleschi fbshipit-source-id: f1b2890ce5341fda790f5689ab1291b69cf7595b
1 parent ee7575f commit 581d643

3 files changed

Lines changed: 333 additions & 2 deletions

File tree

packages/react-native/ReactCommon/react/renderer/textlayoutmanager/platform/ios/react/renderer/textlayoutmanager/RCTTextLayoutManager.mm

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -388,8 +388,9 @@ - (TextMeasurement)_measureTextStorage:(NSTextStorage *)textStorage
388388
size.height = enumeratedLinesHeight;
389389
}
390390

391-
size = (CGSize){ceil(size.width * layoutContext.pointScaleFactor) / layoutContext.pointScaleFactor,
392-
ceil(size.height * layoutContext.pointScaleFactor) / layoutContext.pointScaleFactor};
391+
CGFloat epsilon = 0.001;
392+
size = (CGSize){ceil((size.width + epsilon) * layoutContext.pointScaleFactor) / layoutContext.pointScaleFactor,
393+
ceil((size.height + epsilon) * layoutContext.pointScaleFactor) / layoutContext.pointScaleFactor};
393394

394395
__block auto attachments = TextMeasurement::Attachments{};
395396

Lines changed: 326 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,326 @@
1+
/**
2+
* Copyright (c) Meta Platforms, Inc. and affiliates.
3+
*
4+
* This source code is licensed under the MIT license found in the
5+
* LICENSE file in the root directory of this source tree.
6+
*
7+
* @flow
8+
* @format
9+
*/
10+
11+
import type {RNTesterModuleExample} from '../../types/RNTesterTypes';
12+
import type {ListRenderItemInfo} from 'react-native';
13+
14+
import {FlatList, StyleSheet, Text, View} from 'react-native';
15+
16+
type ContentItem = {
17+
type: 'heading' | 'body',
18+
text: string,
19+
};
20+
21+
const content: Array<ContentItem> = [
22+
{
23+
type: 'heading',
24+
text: 'Lorem Ipsum',
25+
},
26+
{
27+
type: 'heading',
28+
text: 'Lorem Ipsum Dolor Sit Amet',
29+
},
30+
{
31+
type: 'body',
32+
text: 'Lorem ipsum dolor sit amet, consectetur adipiscing elit, sed do eiusmod tempor incididunt ut labore et dolore magna aliqua. Ut enim ad minim veniam, quis nostrud exercitation ullamco laboris nisi ut aliquip ex ea commodo consequat. Duis aute irure dolor in reprehenderit in voluptate velit esse cillum dolore eu fugiat nulla pariatur.',
33+
},
34+
{
35+
type: 'body',
36+
text: 'Sed ut perspiciatis unde omnis iste natus error sit voluptatem accusantium doloremque laudantium, totam rem aperiam, eaque ipsa quae ab illo inventore veritatis et quasi architecto beatae vitae dicta sunt explicabo.',
37+
},
38+
{
39+
type: 'body',
40+
text: 'Excepteur sint occaecat cupidatat non proident, sunt in culpa qui officia deserunt mollit anim id est laborum. Nemo enim ipsam voluptatem quia voluptas sit aspernatur aut odit aut fugit, sed quia consequuntur magni dolores eos qui ratione voluptatem sequi nesciunt.',
41+
},
42+
{
43+
type: 'body',
44+
text: 'Neque porro quisquam est, qui dolorem ipsum quia dolor sit amet, consectetur, adipisci velit, sed quia non numquam eius modi tempora incidunt ut labore et dolore magnam aliquam quaerat voluptatem.',
45+
},
46+
{
47+
type: 'heading',
48+
text: 'Ut Enim Ad Minim Veniam',
49+
},
50+
{
51+
type: 'body',
52+
text: 'At vero eos et accusamus et iusto odio dignissimos ducimus qui blanditiis praesentium voluptatum deleniti atque corrupti quos dolores et quas molestias excepturi sint occaecati cupiditate non provident, similique sunt in culpa qui officia deserunt mollitia animi.',
53+
},
54+
{
55+
type: 'body',
56+
text: 'Et harum quidem rerum facilis est et expedita distinctio. Nam libero tempore, cum soluta nobis est eligendi optio cumque nihil impedit quo minus id quod maxime placeat facere possimus, omnis voluptas assumenda est, omnis dolor repellendus.',
57+
},
58+
{
59+
type: 'body',
60+
text: 'Temporibus autem quibusdam et aut officiis debitis aut rerum necessitatibus saepe eveniet ut et voluptates repudiandae sint et molestiae non recusandae. Itaque earum rerum hic tenetur a sapiente delectus, ut aut reiciendis voluptatibus maiores alias consequatur aut perferendis doloribus asperiores repellat.',
61+
},
62+
{
63+
type: 'body',
64+
text: 'Lorem ipsum dolor sit amet, consectetur adipiscing elit, sed do eiusmod tempor incididunt ut labore et dolore magna aliqua. Ut enim ad minim veniam, quis nostrud exercitation ullamco laboris nisi ut aliquip ex ea commodo consequat.',
65+
},
66+
{
67+
type: 'body',
68+
text: 'Duis aute irure dolor in reprehenderit in voluptate velit esse cillum dolore eu fugiat nulla pariatur. Excepteur sint occaecat cupidatat non proident, sunt in culpa qui officia deserunt mollit anim id est laborum.',
69+
},
70+
{
71+
type: 'heading',
72+
text: 'Consectetur Adipiscing Elit',
73+
},
74+
{
75+
type: 'body',
76+
text: 'Sed ut perspiciatis unde omnis iste natus error sit voluptatem accusantium doloremque laudantium, totam rem aperiam, eaque ipsa quae ab illo inventore veritatis et quasi architecto beatae vitae dicta sunt explicabo.',
77+
},
78+
{
79+
type: 'body',
80+
text: 'Nemo enim ipsam voluptatem quia voluptas sit aspernatur aut odit aut fugit, sed quia consequuntur magni dolores eos qui ratione voluptatem sequi nesciunt. Neque porro quisquam est, qui dolorem ipsum quia dolor sit amet.',
81+
},
82+
{
83+
type: 'body',
84+
text: 'Consectetur, adipisci velit, sed quia non numquam eius modi tempora incidunt ut labore et dolore magnam aliquam quaerat voluptatem. Ut enim ad minima veniam, quis nostrum exercitationem ullam corporis suscipit. This sentence is cut off.',
85+
},
86+
{
87+
type: 'body',
88+
text: 'Nisi ut aliquid ex ea commodi consequatur? Quis autem vel eum iure reprehenderit qui in ea voluptate velit esse quam nihil molestiae consequatur, vel illum qui dolorem eum fugiat quo voluptas nulla pariatur?',
89+
},
90+
{
91+
type: 'heading',
92+
text: 'Tempor Incididunt Ut Labore',
93+
},
94+
{
95+
type: 'body',
96+
text: 'At vero eos et accusamus et iusto odio dignissimos ducimus qui blanditiis praesentium voluptatum deleniti atque corrupti quos dolores et quas molestias excepturi sint occaecati cupiditate non provident.',
97+
},
98+
{
99+
type: 'body',
100+
text: 'Similique sunt in culpa qui officia deserunt mollitia animi, id est laborum et dolorum fuga. Et harum quidem rerum facilis est et expedita distinctio. Nam libero tempore, cum soluta nobis est eligendi optio.',
101+
},
102+
{
103+
type: 'body',
104+
text: 'Cumque nihil impedit quo minus id quod maxime placeat facere possimus, omnis voluptas assumenda est, omnis dolor repellendus. Temporibus autem quibusdam et aut officiis debitis aut rerum necessitatibus saepe eveniet.',
105+
},
106+
{
107+
type: 'body',
108+
text: 'Ut et voluptates repudiandae sint et molestiae non recusandae. Itaque earum rerum hic tenetur a sapiente delectus, ut aut reiciendis voluptatibus maiores alias consequatur aut perferendis doloribus asperiores repellat.',
109+
},
110+
{
111+
type: 'heading',
112+
text: 'Dolore Magna Aliqua',
113+
},
114+
{
115+
type: 'body',
116+
text: 'Lorem ipsum dolor sit amet, consectetur adipiscing elit, sed do eiusmod tempor incididunt ut labore et dolore magna aliqua. Ut enim ad minim veniam, quis nostrud exercitation ullamco laboris nisi ut aliquip ex ea commodo consequat.',
117+
},
118+
{
119+
type: 'body',
120+
text: 'Duis aute irure dolor in reprehenderit in voluptate velit esse cillum dolore eu fugiat nulla pariatur. Excepteur sint occaecat cupidatat non proident, sunt in culpa qui officia deserunt mollit anim id est laborum.',
121+
},
122+
{
123+
type: 'body',
124+
text: 'Sed ut perspiciatis unde omnis iste natus error sit voluptatem accusantium doloremque laudantium, totam rem aperiam, eaque ipsa quae ab illo inventore veritatis et quasi architecto beatae vitae dicta sunt explicabo.',
125+
},
126+
{
127+
type: 'body',
128+
text: 'Nemo enim ipsam voluptatem quia voluptas sit aspernatur aut odit aut fugit, sed quia consequuntur magni dolores eos qui ratione voluptatem sequi nesciunt. Neque porro quisquam est, qui dolorem ipsum quia dolor sit amet, consectetur, adipisci velit.',
129+
},
130+
{
131+
type: 'heading',
132+
text: 'Quis Nostrud Exercitation',
133+
},
134+
{
135+
type: 'body',
136+
text: 'Sed quia non numquam eius modi tempora incidunt ut labore et dolore magnam aliquam quaerat voluptatem. Ut enim ad minima veniam, quis nostrum exercitationem ullam corporis suscipit laboriosam, nisi ut aliquid ex ea commodi consequatur?',
137+
},
138+
{
139+
type: 'body',
140+
text: 'Quis autem vel eum iure reprehenderit qui in ea voluptate velit esse quam nihil molestiae consequatur, vel illum qui dolorem eum fugiat quo voluptas nulla pariatur? At vero eos et accusamus et iusto odio dignissimos ducimus.',
141+
},
142+
{
143+
type: 'body',
144+
text: 'Qui blanditiis praesentium voluptatum deleniti atque corrupti quos dolores et quas molestias excepturi sint occaecati cupiditate non provident, similique sunt in culpa qui officia deserunt mollitia animi, id est laborum et dolorum fuga.',
145+
},
146+
{
147+
type: 'body',
148+
text: 'Et harum quidem rerum facilis est et expedita distinctio. Nam libero tempore, cum soluta nobis est eligendi optio cumque nihil impedit quo minus id quod maxime placeat facere possimus, omnis voluptas assumenda est, omnis dolor repellendus.',
149+
},
150+
{
151+
type: 'heading',
152+
text: 'Voluptatem Accusantium Doloremque',
153+
},
154+
{
155+
type: 'body',
156+
text: 'Temporibus autem quibusdam et aut officiis debitis aut rerum necessitatibus saepe eveniet ut et voluptates repudiandae sint et molestiae non recusandae. Itaque earum rerum hic tenetur a sapiente delectus.',
157+
},
158+
{
159+
type: 'body',
160+
text: 'Ut aut reiciendis voluptatibus maiores alias consequatur aut perferendis doloribus asperiores repellat. Lorem ipsum dolor sit amet, consectetur adipiscing elit, sed do eiusmod tempor incididunt ut labore et dolore magna aliqua.',
161+
},
162+
{
163+
type: 'body',
164+
text: 'Ut enim ad minim veniam, quis nostrud exercitation ullamco laboris nisi ut aliquip ex ea commodo consequat. Duis aute irure dolor in reprehenderit in voluptate velit esse cillum dolore eu fugiat nulla pariatur.',
165+
},
166+
{
167+
type: 'body',
168+
text: 'Excepteur sint occaecat cupidatat non proident, sunt in culpa qui officia deserunt mollit anim id est laborum. Sed ut perspiciatis unde omnis iste natus error sit voluptatem accusantium doloremque laudantium.',
169+
},
170+
{
171+
type: 'heading',
172+
text: 'Beatae Vitae Dicta Sunt',
173+
},
174+
{
175+
type: 'body',
176+
text: 'Totam rem aperiam, eaque ipsa quae ab illo inventore veritatis et quasi architecto beatae vitae dicta sunt explicabo. Nemo enim ipsam voluptatem quia voluptas sit aspernatur aut odit aut fugit.',
177+
},
178+
{
179+
type: 'body',
180+
text: 'Sed quia consequuntur magni dolores eos qui ratione voluptatem sequi nesciunt. Neque porro quisquam est, qui dolorem ipsum quia dolor sit amet, consectetur, adipisci velit, sed quia non numquam eius modi tempora incidunt.',
181+
},
182+
{
183+
type: 'body',
184+
text: 'Ut labore et dolore magnam aliquam quaerat voluptatem. Ut enim ad minima veniam, quis nostrum exercitationem ullam corporis suscipit laboriosam, nisi ut aliquid ex ea commodi consequatur?',
185+
},
186+
{
187+
type: 'body',
188+
text: 'Quis autem vel eum iure reprehenderit qui in ea voluptate velit esse quam nihil molestiae consequatur, vel illum qui dolorem eum fugiat quo voluptas nulla pariatur? At vero eos et accusamus.',
189+
},
190+
{
191+
type: 'heading',
192+
text: 'Praesentium Voluptatum Deleniti',
193+
},
194+
{
195+
type: 'body',
196+
text: 'Et iusto odio dignissimos ducimus qui blanditiis praesentium voluptatum deleniti atque corrupti quos dolores et quas molestias excepturi sint occaecati cupiditate non provident, similique sunt in culpa.',
197+
},
198+
{
199+
type: 'body',
200+
text: 'Qui officia deserunt mollitia animi, id est laborum et dolorum fuga. Et harum quidem rerum facilis est et expedita distinctio. Nam libero tempore, cum soluta nobis est eligendi optio cumque nihil impedit.',
201+
},
202+
{
203+
type: 'body',
204+
text: 'Quo minus id quod maxime placeat facere possimus, omnis voluptas assumenda est, omnis dolor repellendus. Temporibus autem quibusdam et aut officiis debitis aut rerum necessitatibus saepe eveniet.',
205+
},
206+
{
207+
type: 'body',
208+
text: 'Ut et voluptates repudiandae sint et molestiae non recusandae. Itaque earum rerum hic tenetur a sapiente delectus, ut aut reiciendis voluptatibus maiores alias consequatur aut perferendis doloribus asperiores repellat.',
209+
},
210+
{
211+
type: 'heading',
212+
text: 'Mollitia Animi Id Est',
213+
},
214+
{
215+
type: 'body',
216+
text: 'Lorem ipsum dolor sit amet, consectetur adipiscing elit, sed do eiusmod tempor incididunt ut labore et dolore magna aliqua. Ut enim ad minim veniam, quis nostrud exercitation ullamco laboris.',
217+
},
218+
{
219+
type: 'body',
220+
text: 'Nisi ut aliquip ex ea commodo consequat. Duis aute irure dolor in reprehenderit in voluptate velit esse cillum dolore eu fugiat nulla pariatur. Excepteur sint occaecat cupidatat non proident.',
221+
},
222+
{
223+
type: 'body',
224+
text: 'Sunt in culpa qui officia deserunt mollit anim id est laborum. Sed ut perspiciatis unde omnis iste natus error sit voluptatem accusantium doloremque laudantium, totam rem aperiam.',
225+
},
226+
{
227+
type: 'body',
228+
text: 'Eaque ipsa quae ab illo inventore veritatis et quasi architecto beatae vitae dicta sunt explicabo. Nemo enim ipsam voluptatem quia voluptas sit aspernatur aut odit aut fugit, sed quia consequuntur magni dolores.',
229+
},
230+
{
231+
type: 'heading',
232+
text: 'Exercitationem Ullam Corporis',
233+
},
234+
{
235+
type: 'body',
236+
text: 'Eos qui ratione voluptatem sequi nesciunt. Neque porro quisquam est, qui dolorem ipsum quia dolor sit amet, consectetur, adipisci velit, sed quia non numquam eius modi tempora incidunt ut labore.',
237+
},
238+
{
239+
type: 'body',
240+
text: 'Et dolore magnam aliquam quaerat voluptatem. Ut enim ad minima veniam, quis nostrum exercitationem ullam corporis suscipit laboriosam, nisi ut aliquid ex ea commodi consequatur?',
241+
},
242+
{
243+
type: 'body',
244+
text: 'Quis autem vel eum iure reprehenderit qui in ea voluptate velit esse quam nihil molestiae consequatur, vel illum qui dolorem eum fugiat quo voluptas nulla pariatur? At vero eos et accusamus et iusto odio.',
245+
},
246+
{
247+
type: 'body',
248+
text: 'Dignissimos ducimus qui blanditiis praesentium voluptatum deleniti atque corrupti quos dolores et quas molestias excepturi sint occaecati cupiditate non provident, similique sunt in culpa qui officia deserunt.',
249+
},
250+
{
251+
type: 'heading',
252+
text: 'Facilis Est Et Expedita',
253+
},
254+
{
255+
type: 'body',
256+
text: 'Mollitia animi, id est laborum et dolorum fuga. Et harum quidem rerum facilis est et expedita distinctio. Nam libero tempore, cum soluta nobis est eligendi optio cumque nihil impedit quo minus.',
257+
},
258+
{
259+
type: 'body',
260+
text: 'Id quod maxime placeat facere possimus, omnis voluptas assumenda est, omnis dolor repellendus. Temporibus autem quibusdam et aut officiis debitis aut rerum necessitatibus saepe eveniet ut et voluptates.',
261+
},
262+
{
263+
type: 'body',
264+
text: 'Repudiandae sint et molestiae non recusandae. Itaque earum rerum hic tenetur a sapiente delectus, ut aut reiciendis voluptatibus maiores alias consequatur aut perferendis doloribus asperiores repellat.',
265+
},
266+
{
267+
type: 'body',
268+
text: 'Lorem ipsum dolor sit amet, consectetur adipiscing elit, sed do eiusmod tempor incididunt ut labore et dolore magna aliqua. Ut enim ad minim veniam, quis nostrud exercitation ullamco laboris nisi ut aliquip ex ea commodo consequat.',
269+
},
270+
];
271+
272+
const TextLayoutExample = (): React.Node => {
273+
const renderItem = ({item}: ListRenderItemInfo<ContentItem>): React.Node => {
274+
return (
275+
<View style={styles.textContainer}>
276+
<Text
277+
style={item.type === 'heading' ? styles.headingText : styles.text}>
278+
{item.text}
279+
</Text>
280+
</View>
281+
);
282+
};
283+
284+
return (
285+
<View style={styles.container}>
286+
<FlatList
287+
data={content}
288+
renderItem={renderItem}
289+
showsVerticalScrollIndicator={true}
290+
/>
291+
</View>
292+
);
293+
};
294+
295+
const styles = StyleSheet.create({
296+
container: {
297+
backgroundColor: '#fff',
298+
},
299+
textContainer: {
300+
marginBottom: 16,
301+
marginHorizontal: 16,
302+
},
303+
headingText: {
304+
fontWeight: 'bold',
305+
fontSize: 28,
306+
},
307+
text: {
308+
fontSize: 17,
309+
lineHeight: 24,
310+
},
311+
});
312+
313+
const examples: Array<RNTesterModuleExample> = [
314+
{
315+
title: 'Multiple Paragraph Text Layout Example',
316+
render(): React.Node {
317+
return <TextLayoutExample />;
318+
},
319+
},
320+
];
321+
322+
module.exports = {
323+
title: 'Text Layout',
324+
description: 'Example demonstrating text layout of multiple paragraphs.',
325+
examples,
326+
};

packages/rn-tester/js/utils/RNTesterList.ios.js

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -125,6 +125,10 @@ const Components: Array<RNTesterModuleInfo> = [
125125
module: require('../examples/TextInput/TextInputExample'),
126126
category: 'Basic',
127127
},
128+
{
129+
key: 'TextLayoutExample',
130+
module: require('../examples/TextLayout/TextLayoutExample'),
131+
},
128132
{
129133
key: 'TouchableExample',
130134
module: require('../examples/Touchable/TouchableExample'),

0 commit comments

Comments
 (0)