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

Comments

yoroy’s picture

Status: Needs review » Needs work

Spreadsheet apps call these "decimal places" or "number of decimals". The current patch has too much overlap with the description.

droplet’s picture

Version: 7.x-dev » 8.x-dev
sutharsan’s picture

Issue summary: View changes
Status: Needs work » Needs review
StatusFileSize
new1.64 KB

This patch uses 'decimal places' as context. Number module has now two occurrences of "Scale", both receive this context.

Status: Needs review » Needs work

The last submitted patch, 3: drupal-number-add-scale-context-1078618-3.patch, failed testing.

sutharsan’s picture

Status: Needs work » Needs review
droplet’s picture

Status: Needs review » Reviewed & tested by the community

grepped the sources, only 2 occurrences. Patch still apply.

webchick’s picture

Status: Reviewed & tested by the community » Fixed

Committed and pushed to 8.x. Thanks!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.

swentel’s picture

Status: Closed (fixed) » Needs review
StatusFileSize
new1.67 KB

@Sutharsan isn't argument missing the 'context' key ? See attached patch.

sutharsan’s picture

Status: Needs review » Reviewed & tested by the community

@swentel You couldn't be more right.

Status: Reviewed & tested by the community » Needs work

The last submitted patch, 9: 1078618-9.patch, failed testing.

swentel’s picture

Status: Needs work » Reviewed & tested by the community

Bot hickup

alexpott’s picture

Status: Reviewed & tested by the community » Fixed

I think this deserves a new issue to keep issue credit etc sane - also this was closed 2 years ago.

joelpittet’s picture

@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

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.