Skip to content

Commit b7332ee

Browse files
committed
using string for accessibilityHours
1 parent 2bedebc commit b7332ee

File tree

4 files changed

+14
-12
lines changed

4 files changed

+14
-12
lines changed

ReactCommon/react/renderer/attributedstring/TextAttributes.cpp

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -100,9 +100,9 @@ void TextAttributes::apply(TextAttributes textAttributes) {
100100
: accessibilityRole;
101101
accessibilityUnit = !textAttributes.accessibilityUnit.empty() ? textAttributes.accessibilityUnit
102102
: accessibilityUnit;
103-
accessibilityHours = !(textAttributes.accessibilityHours == 0) ? textAttributes.accessibilityHours
103+
accessibilityHours = !textAttributes.accessibilityHours.empty() ? textAttributes.accessibilityHours
104104
: accessibilityHours;
105-
accessibilityMinutes = !(textAttributes.accessibilityMinutes == 0) ? textAttributes.accessibilityMinutes
105+
accessibilityMinutes = !textAttributes.accessibilityMinutes.empty() ? textAttributes.accessibilityMinutes
106106
: accessibilityMinutes;
107107
}
108108

@@ -222,6 +222,8 @@ SharedDebugStringConvertibleList TextAttributes::getDebugProps() const {
222222
debugStringConvertibleItem("layoutDirection", layoutDirection),
223223
debugStringConvertibleItem("accessibilityRole", accessibilityRole),
224224
debugStringConvertibleItem("accessibilityUnit", accessibilityUnit),
225+
debugStringConvertibleItem("accessibilityHours", accessibilityHours),
226+
debugStringConvertibleItem("accessibilityMinutes", accessibilityMinutes),
225227
};
226228
}
227229
#endif

ReactCommon/react/renderer/attributedstring/TextAttributes.h

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -80,8 +80,8 @@ class TextAttributes : public DebugStringConvertible {
8080
std::optional<LayoutDirection> layoutDirection{};
8181
std::optional<AccessibilityRole> accessibilityRole{};
8282
std::string accessibilityUnit{};
83-
int accessibilityHours{};
84-
int accessibilityMinutes{};
83+
std::string accessibilityHours{};
84+
std::string accessibilityMinutes{};
8585

8686
#pragma mark - Operations
8787

ReactCommon/react/renderer/attributedstring/conversions.h

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1013,11 +1013,11 @@ inline folly::dynamic toDynamic(const TextAttributes &textAttributes) {
10131013
_textAttributes(
10141014
"accessibilityUnit", textAttributes.accessibilityUnit);
10151015
}
1016-
if (!(textAttributes.accessibilityHours == 0)) {
1016+
if (!textAttributes.accessibilityHours.empty()) {
10171017
_textAttributes(
10181018
"accessibilityHours", textAttributes.accessibilityHours);
10191019
}
1020-
if (!(textAttributes.accessibilityMinutes == 0)) {
1020+
if (!textAttributes.accessibilityMinutes.empty()) {
10211021
_textAttributes(
10221022
"accessibilityMinutes", textAttributes.accessibilityMinutes);
10231023
}
@@ -1250,14 +1250,14 @@ inline MapBuffer toMapBuffer(const TextAttributes &textAttributes) {
12501250
builder.putString(
12511251
TA_KEY_ACCESSIBILITY_UNIT, textAttributes.accessibilityUnit);
12521252
}
1253-
if (!(textAttributes.accessibilityHours == 0)) {
1253+
if (!textAttributes.accessibilityHours.empty()) {
12541254

1255-
builder.putInt(
1255+
builder.putString(
12561256
TA_KEY_ACCESSIBILITY_HOURS, textAttributes.accessibilityHours);
12571257
}
1258-
if (!(textAttributes.accessibilityMinutes == 0)) {
1258+
if (!textAttributes.accessibilityMinutes.empty()) {
12591259

1260-
builder.putInt(
1260+
builder.putString(
12611261
TA_KEY_ACCESSIBILITY_MINUTES, textAttributes.accessibilityMinutes);
12621262
}
12631263
return builder.build();

ReactCommon/react/renderer/components/view/AccessibilityProps.h

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -40,8 +40,8 @@ class AccessibilityProps {
4040
bool accessible{false};
4141
AccessibilityState accessibilityState;
4242
AccessibilityUnit accessibilityUnit;
43-
int accessibilityHours;
44-
int accessibilityMinutes;
43+
AccessibilityUnit accessibilityHours;
44+
AccessibilityUnit accessibilityMinutes;
4545
std::string accessibilityLabel{""};
4646
AccessibilityLabelledBy accessibilityLabelledBy{};
4747
AccessibilityLiveRegion accessibilityLiveRegion{

0 commit comments

Comments
 (0)