Open Bug 1311472 Opened 8 years ago Updated 2 years ago

[tracking] mozillaonline addons/firefox CN distribution

Categories

(WebExtensions :: General, defect, P3)

49 Branch
defect

Tracking

(Not tracked)

People

(Reporter: mixedpuppy, Unassigned)

References

(Depends on 12 open bugs, Blocks 1 open bug)

Details

(Keywords: meta, Whiteboard: triaged)

Issues encountered during conversion of easyscreenshot [I'll have to see if solutions or bugs exist for any of these] 1. We lose the ability to style the navbar button to make it consistent with browser; Bug 1207597 ? 2. We can't use the same button id for it to stay where it was (navbar, menu panel, customization palette); 3. We lose the ability to communicate with other extensions, e.g. cpmanager, our main extension; 4. We now have to use the scroll and capture approach for the full page screenshot, which takes longer than the "old" drawWindow call and involves handling fixed positioned element in the page; Bug 1310318 5. I have not been able to figure out how to copy the edited screenshot into the clipboard in an extension page, the selection/execCommand combo doesn't seem to work for any thing but text; 6. It's not possible to let user customize the WebExtensions defined shortcuts; 7. No visual clue when a badged navbar button (all WebExtensions ones) is disabled. Bug 1204609
Depends on: 1204609, 1310318, 1207597
Depends on: 1311500
(In reply to Shane Caraveo (:mixedpuppy) from comment #1) > > 1. We lose the ability to style the navbar button to make it consistent > with browser; Bug 1207597 ? Yes, we were trying to match all the variants of Fx toolbar styles[1]: inverted, hi-dpi, different versions of macOS or Windows. Bug 1207597 sounds like helpful. > 3. We lose the ability to communicate with other extensions, e.g. > cpmanager, our main extension; We have an unified on/off switch inside cpmanager to control tracking pings we send from other extensions. This used to be a `try { Cu.import(...) ... } catch(ex) { ... }`, maybe bug 1258360 would be helpful here. > 6. It's not possible to let user customize the WebExtensions defined > shortcuts; See [2], "If a key combination is already used by the browser (for example, "Ctrl+Shift+R"), or by an existing add-on, then you can't override it. You will be allowed to define it, but your event handler will not be called when the user enters it." We used to make the shortcuts customizable for this, but it's no longer possible with the `commands` JavaScript API[3]. [1]: https://dxr.mozilla.org/mozilla-central/search?q=path%3Abrowser%2Fthemes+path%3AToolbar+-path%3Aplaces+ext%3Apng [2]: https://developer.mozilla.org/en-US/Add-ons/WebExtensions/manifest.json/commands#Key_combinations [3]: https://developer.mozilla.org/en-US/Add-ons/WebExtensions/API/commands
Whiteboard: triaged
Recently, we at Beijing office did a preliminary classification of current features of our extensions. I'll file bugs for the potential API gaps we identified blocking this one. For each of the features, we might need a new API created (or better, one already planned or exists but I'm not aware of it) or maybe it's better to upstream the work into the vanilla Firefox.
Depends on: 1329853
Depends on: 1322304
Depends on: 1329860
Depends on: 1329868
Depends on: 1329874
(In reply to Shane Caraveo (:mixedpuppy) from comment #1) > 5. I have not been able to figure out how to copy the edited screenshot > into the clipboard in an extension page, the selection/execCommand combo > doesn't seem to work for any thing but text; Bug 1325759
Depends on: 1325759
Blocks: 1303905
TCPSocket will be necessary for our Email notifier extension which has a very minimal implementation of IMAP.
Depends on: libdweb-udp
(In reply to Shane Caraveo (:mixedpuppy) from comment #0) > share_all_cn@mozillaonline.com Possible API for this one.
Depends on: 1312098
(In reply to Shane Caraveo (:mixedpuppy) from comment #0) > tabtweak@mozillaonline.com Double click a tab to close it. Open bookmarks/search results in new tab when the current tab is not blank
Depends on: 1246706, 1270763
(In reply to Shane Caraveo (:mixedpuppy) from comment #0) > cehomepage@mozillaonline.com Alternative new tab page, and its access to thumbnailing service Load a webpage into a browser inside about:cehome (may not be necessary per bug 1322304 comment 6) and use it as homepage Updates to partner bookmarks, based on the predefined keywords in distribution.ini
A few APIs we're interested in but not specific to one extension.
Depends on: 1253419, 1268020, 1315558
Depends on: 1331557
Depends on: 1333403
No longer depends on: 1333403
Depends on: 1331559
Depends on: 1356543
Depends on: 1376793
Depends on: 1376802
Depends on: 1379615
Depends on: 1379618
(In reply to Hector Zhao [:hectorz] from comment #9) > A few APIs we're interested in but not specific to one extension. Bug 1253419 is necessary for the restore recently closed tabs button: we need the main button to restore the most recent one and the secondary dropdown to list more.
Depends on: 1379623
Necessary for opening a link with drag-n-drop.
Depends on: 1238314
(In reply to Hector Zhao [:hectorz] from comment #11) > Necessary for opening a link with drag-n-drop. Really? browser.tabs.create with index. FYI: https://github.com/harytfw/GlitterDrag
(In reply to YF (Yang) from comment #12) > > Really? browser.tabs.create with index. FYI: > https://github.com/harytfw/GlitterDrag Ah, maybe I should say "properly". Yes, you could calculate an index to position the new tab, but Fx won't know it's opened by following a link in another page.
Depends on: 1383669
Depends on: 1383939
With regards to search engines, we would like to be able to initiate a search with the current selected engine (like when user select some non-url text and then drag it).
Depends on: 1268401
Depends on: 1345158
Keywords: meta
Priority: -- → P2
Depends on: 1394304, 1420969
Depends on: 1315616
No longer depends on: 1318532, 1316097
Depends on: 1344749, 1412263
Product: Toolkit → WebExtensions
Component: Untriaged → General
No longer depends on: 1383939
Depends on: 1556296
Priority: P2 → P3
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.