Updated: Comment #3
Problem/Motivation
There is the string 'Scale' used in two different context in core;
- Used as title of the number of digits in number.module.
- Used as label of an image effect in image.module.
In Japanese, and other languages, different translations are required for each.
Proposed resolution
Add a context to the string.
Remaining tasks
none
User interface changes
Interface translation will change.
API changes
none
| Comment | File | Size | Author |
|---|---|---|---|
| #9 | 1078618-9.patch | 1.67 KB | swentel |
| #3 | drupal-number-add-scale-context-1078618-3.patch | 1.64 KB | sutharsan |
| number.module.1.patch | 664 bytes | Takafumi | |
| used_in.png | 33.85 KB | Takafumi |
Comments
Comment #1
yoroy commentedSpreadsheet apps call these "decimal places" or "number of decimals". The current patch has too much overlap with the description.
Comment #2
droplet commentedComment #3
sutharsan commentedThis patch uses 'decimal places' as context. Number module has now two occurrences of "Scale", both receive this context.
Comment #5
sutharsan commented3: drupal-number-add-scale-context-1078618-3.patch queued for re-testing.
Comment #6
droplet commentedgrepped the sources, only 2 occurrences. Patch still apply.
Comment #7
webchickCommitted and pushed to 8.x. Thanks!
Comment #9
swentel commented@Sutharsan isn't argument missing the 'context' key ? See attached patch.
Comment #10
sutharsan commented@swentel You couldn't be more right.
Comment #12
swentel commentedBot hickup
Comment #13
alexpottI think this deserves a new issue to keep issue credit etc sane - also this was closed 2 years ago.
Comment #14
joelpittet@swentel made you a follow-up, can you re-upload to this follow-up issue for credit? #2656442: Argument missing the 'context' key for 'decimal places' context