I need to translate add new in the Popups Reference module.

Attached is a patch file

CommentFileSizeAuthor
diff.patch936 bytesali.khairat
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

ali.khairat’s picture

ali.khairat CreditAttribution: ali.khairat commented
Title: Mising t() function for some text » Missing t() function for some text
owahab’s picture

owahab CreditAttribution: owahab commented
Status: Active » Needs review
bibo’s picture

bibo CreditAttribution: bibo commented
Version: 6.x-1.x-dev » 6.x-2.0-alpha1
Status: Needs review » Reviewed & tested by the community

I was just about to post an identical patch exactly for this. Luckily I saw this post first so I didn't create a duplicate.
But less luckily, seems that this patch has been waiting for months.. Is this going to be added to the next version or what?

These small translation details can get really annoying. I mean this module even uses t() for most UI strings, but these 2 lines
are basically the only important ones (because normal users see them). Imo the admin settings can stay untranslated, if just normal users see the localized version (not the issue here, but imo "normal users" are a priority in UI-related things).

Please fix this.

anrikun’s picture

anrikun CreditAttribution: anrikun commented

Thank you for the patch!
This is a so basic thing to fix that it should really be committed.

But I think that the test itself is not really easy to understand for end users:
"Add New: Add Type"

Shouldn't it be:
"Add new Type"?

It's simpler to understand than the original text, don't you think?