Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[ActivityIndicator] Update accessibility label and value #3261

Merged
merged 1 commit into from
Apr 9, 2018
Merged

[ActivityIndicator] Update accessibility label and value #3261

merged 1 commit into from
Apr 9, 2018

Conversation

jmdetloff
Copy link
Contributor

@jverkoey
Copy link
Contributor

jverkoey commented Apr 5, 2018

Were you able to auto-generate these changes in some way, or was this a manual change?


/* Accessibility label for activity indicator progress halted */
"MaterialActivityIndicatorProgressHaltedAccessibilityLabel" = "Progress Halted";
/* Accessibility value for indeterminate activity indicator in progress. */
Copy link
Contributor

Choose a reason for hiding this comment

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

"determinate"?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yep! Thanks, updated

Copy link
Contributor

@jverkoey jverkoey left a comment

Choose a reason for hiding this comment

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

lgtm, but please update the commit description with the method you used to update the strings (whether manual or automated).

@codecov-io
Copy link

Codecov Report

Merging #3261 into develop will decrease coverage by 0.01%.
The diff coverage is 0%.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop    #3261      +/-   ##
===========================================
- Coverage    51.76%   51.75%   -0.02%     
===========================================
  Files          178      178              
  Lines        25009    25014       +5     
  Branches      1196     1196              
===========================================
  Hits         12946    12946              
- Misses       11989    11994       +5     
  Partials        74       74
Impacted Files Coverage Δ
...nents/ActivityIndicator/src/MDCActivityIndicator.m 25.2% <0%> (-0.21%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 246ab9e...9b8bd07. Read the comment docs.

@jverkoey jverkoey changed the title Update Activity Indicator accessibility label and value [ActivityIndicator] Update accessibility label and value Apr 7, 2018
@jmdetloff
Copy link
Contributor Author

This ended up being a partially manual process. Generate the strings internally, manually create a PR externally.

The generated files were created with generate_string_tables.py
internally, then exported manually to the external
material-components-ios repo
@jmdetloff jmdetloff merged commit 7b38f3f into material-components:develop Apr 9, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
4 participants