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

Allow to customize the capitalization of messages #408

Merged
merged 1 commit into from
Apr 7, 2017

Conversation

lbalmaceda
Copy link
Contributor

@lbalmaceda lbalmaceda commented Apr 4, 2017

This PR removes the All Caps flag from the styles and instead, capitalizes the text directly from the resources.
Note that the only case where the "error_description" received in a failed request it's directly used as input for the error banner on top of the Lock's header, it's when there's a Rule error. The rest of the cases are covered in this PR and are already capitalized.

@lbalmaceda lbalmaceda added this to the v2-Next milestone Apr 4, 2017
@lbalmaceda lbalmaceda requested a review from hzalaz April 4, 2017 20:17
@lbalmaceda lbalmaceda force-pushed the fix-do-not-force-uppercase branch from f24cea1 to ae6e97c Compare April 6, 2017 17:17
@lbalmaceda lbalmaceda merged commit 80d6237 into master Apr 7, 2017
@lbalmaceda lbalmaceda deleted the fix-do-not-force-uppercase branch April 7, 2017 19:25
@lbalmaceda lbalmaceda modified the milestones: v2-Next, 2.5.0 Apr 26, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants