Request: Please use c-format

Bug #980518 reported by Masato HASHIMOTO on 2012-04-13
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Inkscape
Low
jazzynico

Bug Description

For the sake of translation, please use c-format in the following message:

* src/ui/dialog/ocaldialogs.cpp:1081
    Glib::ustring markup = Glib::ustring::compose(
        "<span size=\"large\">%1 <b>%2</b> %3</span>\n<span color=\"%4\">%5</span>",
        _("No clipart named"), keywords, _("was found."), grey.to_string(),
        _("Please make sure all keywords are spelled correctly, or try again with different keywords."));

version: trunk-r11233

jazzynico (jazzynico) on 2012-04-13
tags: added: translation
Changed in inkscape:
assignee: nobody → JazzyNico (jazzynico)
importance: Undecided → Low
milestone: none → 0.49
status: New → Triaged
Jan Pulmann (jan-pulmann) wrote :

Hi, I wrote little fix that tranforms the string into format I figured out using other tranlated examples, 1 string divided to three ".." blocks. Is this the kind of format you was asking for? Thanks :)
Jan Pulmann

Masato HASHIMOTO (hashimo) wrote :

@Jan:
Yes, your patch resolves my request.
Strings sequence the '_("No clipart named"), keywords, _("was found.")' restricts its grammar and it makes difficulty in Japanese translation.

Thanks

jazzynico (jazzynico) wrote :

Thanks for your patch Jan!

The correction committed in the trunk (revision 11251) is a bit different. The patch you did was a good start, but it was cleaner to split the two sentences and remove XML elements (including the color attribute value that must not be translated) from the translatable string.

Changed in inkscape:
status: Triaged → Fix Committed
milestone: 0.49 → none
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