Ticket #3445 (closed defect: fixed)

Opened 14 months ago

Last modified 12 months ago

Add comment to .PO file on getting localized Google search, also please release with new translations

Reported by: HoboPrimate Owned by: erikos
Priority: Unspecified by Maintainer Milestone: 0.96
Component: Browse Version: Unspecified
Severity: Unspecified Keywords: 12.1.0, patch, olpc-test-passed
Cc: manuq, humitos Distribution/OS: Unspecified
Bug Status: Unconfirmed

Description

Hi,
I've updated the translation for Browse, finally figured out how to make google search page be in portuguese: I added "hl=pt&" in the middle of the url, right after " http://www.google.com/search?". Could you add a Comment on the .PO file about this place to add the language?
And also, could you release a new version of Browse with the updated translation?
Thanks

Attachments

Change History

  Changed 13 months ago by manuq

  • cc manuq added
  • milestone changed from Unspecified by Release Team to 0.96

Changed 13 months ago by humitos

  Changed 13 months ago by humitos

  • cc humitos added
  • keywords patch added

  Changed 13 months ago by humitos

The comment was "stolen" from the Epiphany source code:

follow-up: ↓ 6   Changed 13 months ago by HoboPrimate

I would prefer the use of hl=LANG, the language a person is using with sugar won't necessarily mean they are living in the language's native country, and there may be other cases, like what happens with European Portuguese, where other countries have it as one of their official languages but have their own country-specific google page (São Tomé e Príncipe = google.st).

in reply to: ↑ 5   Changed 13 months ago by manuq

Replying to HoboPrimate:

I would prefer the use of hl=LANG, the language a person is using with sugar won't necessarily mean they are living in the language's native country, and there may be other cases, like what happens with European Portuguese, where other countries have it as one of their official languages but have their own country-specific google page (São Tomé e Príncipe = google.st).

Yes, and that's what latest humitos patch does.

  Changed 13 months ago by HoboPrimate

Oh, sorry.

  Changed 13 months ago by humitos

Here you are a test case:

  1. Go to "My settings"
  2. Click on "Language"
  3. Change it to some one different that you have (it could be Portuguese for example)
  4. Save this option
  5. Open Browse
  6. Write something in the url like "I'm doing a search in google :)"

Expected behavior (after applying this patch)
The search results should be displayed in Portuguese

Changed 13 months ago by humitos

  Changed 13 months ago by manuq

  • keywords 12.1.0, patch, olpc-test-pending added; patch removed
  • status changed from new to closed
  • resolution set to fixed

Pushed as ebb013347fcf777401c589a557c50fae8b7eb7e7 .

  Changed 12 months ago by greenfeld

  • keywords olpc-test-passed added; olpc-test-pending removed

The Google search matches the current set language in Browse-137 in 12.1.0 os10 for searches done in the title bar only (instead of entering URLs).

The OLPC custom homepage's Google search form still searches only in English.

Note: See TracTickets for help on using tickets.