#3445 closed defect (fixed)
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: | |
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 (3)
Change History (14)
comment:1 Changed 11 years ago by manuq
- Cc manuq added
- Milestone changed from Unspecified by Release Team to 0.96
Changed 11 years ago by humitos
Changed 11 years ago by humitos
comment:2 Changed 11 years ago by humitos
- Cc humitos added
- Keywords patch added
comment:3 Changed 11 years ago by humitos
comment:4 Changed 11 years ago by humitos
comment:5 follow-up: ↓ 6 Changed 11 years 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).
comment:6 in reply to: ↑ 5 Changed 11 years 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.
comment:7 Changed 11 years ago by HoboPrimate
Oh, sorry.
comment:8 Changed 11 years ago by humitos
Here you are a test case:
- Go to "My settings"
- Click on "Language"
- Change it to some one different that you have (it could be Portuguese for example)
- Save this option
- Open Browse
- 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 11 years ago by humitos
comment:9 Changed 11 years ago by manuq
- Keywords 12.1.0 olpc-test-pending added
- Resolution set to fixed
- Status changed from new to closed
Pushed as ebb013347fcf777401c589a557c50fae8b7eb7e7 .
comment:10 Changed 11 years 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.
The comment was "stolen" from the Epiphany source code: