Hi there,

It would be nice to add a context to the label "Order" for translation. This appears when we expose the sort option.
Right now this string is translated (in Spanish) in the context of electronic commerce.

Thanks

CommentFileSizeAuthor
#1 order-context-2425355.patch578 bytesmnico

Comments

mnico’s picture

mnico commented
Status: Active » Needs review
StatusFileSize
new order-context-2425355.patch578 bytes

Adding a patch.

chris matthews’s picture

chris matthews commented

The 4 year old patch to views_plugin_exposed_form.inc applied cleanly to the latest views 7.x-3.x-dev and if still relevant needs to be reviewed.

mustanggb’s picture

mustanggb commented
Category: Support request » Feature request
renatog’s picture

renatog commented
Status: Needs review » Reviewed & tested by the community

Yeah, I applied the patch, tested it makes sense totally.
In that case, we were using the word "Order" without context. For us (in English) that's okay because the word "Order" is used on "Ordination" like "sort order". But for commerce "Order" can be used for purchase. So use context on that really makes sense. Moving that to RTBC.

Thank you so much @mnico, good catch

  • RenatoG committed 0e15d69 on 7.x-3.x authored by mnico 
    Issue #2425355 by mnico, Chris Matthews, MustangGB, RenatoG: Add context...
renatog’s picture

renatog commented
Status: Reviewed & tested by the community » Fixed
Parent issue: » #3118500: Plan for Views 7.x-3.25

Moved to the dev branch

Thank you people for your contribution

Status: Fixed » Closed (fixed)

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