Closed Bug 668776 Opened 10 years ago Closed 10 years ago
[csb] Firefox protocol handler setup for Kashubian
Protocol handlers allow links like mailto:firstname.lastname@example.org to be handled by a webmail application. In order to do that, the site needs to support a special API described on http://developer.mozilla.org/en/docs/Web-based_protocol_handlers. For en-US, we're currently shipping with GMail and Yahoo! Mail for mailto:, 30boxes for webcal: and Mibbit for irc: urls, and we're on a ongoing evangelism effort to add more. For Kashubian, we'd like to do that, too. First, the localization team and Milos (Milos Dinic) will look at the market for the language and come up with a good candidates. Jurk, we'll need your input on this, the guidelines for making recommendations are on <https://wiki.mozilla.org/L10n:Firefox/Productization>. Following that is a reach out by the evangelism team to try to get the vendor to support protocol handlers. Once that happens, Milos will take over in reviewing the actual implementation, which will need to change the gecko.handlerService.* entries in browser/chrome/browser-region/region.properties. Please don't make changes to that file without getting a positive review by Milos or somebody else appointed by Milos/Stas on a patch for that change upfront. The Kashubian team will be driving this bug to completion.
Milos, Please, use the following Web mail handlers for the Kashubian FF (mostly Polish): Poczta Interia.pl (http://poczta.interia.pl/mh/?mailto=%s) OnetPoczta (http://poczta.onet.pl/napisz.html?uri=%s) Poczta WP (http://poczta.wp.pl/napisz.html?url=%s) GMail Yahoo! The other settings should mimic those of en-US.
I assume we (the Kashubian Team) have addressed the bug.
Milos, I'm attaching a patch containing Kashubian Firefox Protocol Handlers. Let me know how this looks to you.
Component: Other → csb / Kashubian
QA Contact: csb
Comment on attachment 553214 [details] [diff] [review] Protocol Handles for Kashubian Firefox Hi Jurk, you can request a review explicitly by going to the flags in the attachment details and set the review flag to "?", and set the requestee to :Milos. I've talking to Milos about this patch on irc, and that get's an r- because you added browser/chrome/browser-region/region.properties.orig, which shouldn't end up in the repository. Mishaps like this are among the reasons why we're doing the reviews on the more critical files. You probably want to `hg rm browser/chrome/browser-region/region.properties.orig` again, and use `hg addremove` with much caution. Following the settings from the Polish localization is cool, though. Please attach an updated patch to this bug and request review from Milos again?
Axel, Thanks for the explanation. I have included the .orig in the other patches too -- will review and resubmit.
Milos, I have to learn how to properly make the patches. Before I go on, can please let me know if this is done correctly? I have re-cloned my local repository, so there is no garbage. Thanks
Comment on attachment 553809 [details] [diff] [review] Csb.regional.properties Review of attachment 553809 [details] [diff] [review]: ----------------------------------------------------------------- Looks OK to me. Please commit. Thanks!
Attachment #553809 - Flags: review?(milos) → review+
The patch has been pushed out to the Mozilla repository.
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.