Context for message "Offset:"

Bug #1269943 reported by Kris on 2014-01-16
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Inkscape
Low
Kris

Bug Description

Translators should be able to add different translations for the message "Offset:".

#: ../src/extension/internal/bitmap/adaptiveThreshold.cpp:43
#: ../src/widgets/gradient-toolbar.cpp:1174
#: ../src/widgets/gradient-vector.cpp:926
#: ../share/extensions/printing_marks.inx.h:12

Kris (kris-degussem) wrote :

I propose something similar to the attachment, but I do not know how to handle the params tags for the first and last file mentioned above.

jazzynico (jazzynico) on 2014-01-17
tags: added: i18n translation
jazzynico (jazzynico) wrote :

Attributes can't be contextualized in the external extensions (but elements can, in the trunk only).
As for the internal extension, I'd test with NC_("AdaptativeThreshold", "Offset:"). But since none of the internal extensions uses translation context, there's no certainty it's supported.

Changed in inkscape:
assignee: nobody → Kris (kris-degussem)
milestone: none → 0.91
status: New → In Progress
jazzynico (jazzynico) wrote :

> there's no certainty it's supported.

I confirm it's not. As for the external extensions, attributes contextualization doesn't work.

Kris (kris-degussem) wrote :

Thanks a lot for checking.
I wish there was a possibility to separate the context of the extensions (it would be better), but it seems I'll need to carefully choose a term that works for both extensions.
So committing patch in comment 1 and marking as fix committed?

jazzynico (jazzynico) wrote :

Patch tested successfully on Crunchbang Waldorf and committed revision 12952.
Thanks Kris!

Changed in inkscape:
status: In Progress → Fix Committed
Bryce Harrington (bryce) on 2015-02-21
Changed in inkscape:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers