Add labels to gallery thumbnail size controls#1775
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
@whitewooood is attempting to deploy a commit to the Universal Viewer Team on Vercel. A member of the Team first needs to authorize it. |
demiankatz
left a comment
There was a problem hiding this comment.
Thanks for your contribution, @whitewooood -- I took a quick look and have a question and suggestion below.
| this._$sizeDownButton = $( | ||
| '<input class="btn btn-default size-down" type="button" value="-" />' | ||
| ); | ||
| ).attr("aria-label", this.options.data.content.decreaseSize); |
There was a problem hiding this comment.
Is there a particular reason why you chose to shorten the string names when including them in content? I think it might be better to use decreaseThumbnailSize or decreaseThumbSize instead of just decreaseSize to reduce potential ambiguity if we add more strings in the future relating to different size-related functionality. (This is also true of increaseSize and especially size).
| "$shareLink": "Dolen i'r dudalen hon", | ||
| "$shareInstructions": "Er mwyn rhannu'r eitem hon, cop\u00efwch y URL isod.", | ||
| "$size": "Maint:", | ||
| "$decreaseThumbnailSize": "Lleihau maint bawdluniau", |
There was a problem hiding this comment.
Where/how did you source your non-English translations? Do we need to have our translation team review them for accuracy?
|
Thanks for the review. I pushed an update using more specific content keys: For the non-English strings, I added direct equivalents so the new labels resolve in the existing locale files. They have not gone through a separate translation-team review, so adjustments from the translation workflow would be welcome. |
Summary
Fixes #1488.
Validation