Closed Bug 450444 Opened 13 years ago Closed 13 years ago
[ru] Add Mail
.ru mailto: protocol handler
Mail.ru has informed me that they has implemented mailto: protocol handler for Firefox users and I'd love to include it in Russian build of Firefox. Mail.ru is the largest free e-mail service in Russia (see http://en.wikipedia.org/wiki/Mail.ru for more info) and it would be large benefit for Russian users to have Mail.ru protocol web-handler in Russian build of Firefox. Patch is attached (regarding http vs. https issue - Mail.ru has informed me, that they are not ready to provide https web-mail right now)
We need their permission in Mic's files first.
alexander, can you get the mail.ru guys to email me asap with permission or comment in this bug. if you have tested the feature and it works we can add it to the current list of 3 mailto for ru: rambler, yandex and google
It's ok to use mailto link http://win.mail.ru/cgi-bin/sentmsg?mailto=% in firefox Alexander Bykov, Mail.Ru Team
Hi Alexander, thanks. Does that url pass the testcases on http://people.mozilla.org/~ctalbert/test-protocol-links.html ?
Test 5 failed (url works) Test 6 failed Test 7 failed Test 8 failed (unfixable due to lack of utf-8 support) Test 10 failed thanks for link we will fix it soon, except utf-8 support (Test 8)
all fixed, except: Test 5 - we trying to parse incorrect url (common practice) Test 8 - we support only windows-1251 charset on the web so utf-8 chars not in windows-1251 converted to '?' its acceptable ?
Test 8 should work as long as the UTF-8 content has an equivalent in windows-1251. Say, mailto:email@example.comfirstname.lastname@example.org&subject=Фото should work. Re Test 5, what's the outcome of the given url?
I forgot to bump gecko.handlerService.defaultHandlersVersion, fixing it.
to Axel Hecht: Test 8 - it's OK Test 5 - CC field value will be: "email@example.com?body=foo" because we spliting params by '&'
Comment on attachment 333787 [details] [diff] [review] Add Mail.ru mailto: protocol handler v.2 r=me, code is OK, tests are OK, permission is in comment 3. There are obviously more advanced ways for error recovery in URLs as the example shows, but that's not a real issue.
Attachment #333787 - Flags: review?(l10n) → review+
Comment on attachment 333787 [details] [diff] [review] Add Mail.ru mailto: protocol handler v.2 Please land this on hg, and on cvs trunk with my 184.108.40.206 approval. Please use check-in comments referencing this bug and my review/approval, and use the fixed220.127.116.11 and verified18.104.22.168 keywords to track landing and testing on fx 3.0.x
Attachment #333787 - Flags: approval22.214.171.124+
Pushed to l10n-central: http://hg.mozilla.org/l10n-central/index.cgi/ru/rev/c6ced56bc047
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Fixed on CVS trunk: Checking in chrome/browser-region/region.properties; /l10n/l10n/ru/browser/chrome/browser-region/region.properties,v <-- region.properties new revision: 1.23; previous revision: 1.22 done
You need to log in before you can comment on or make changes to this bug.