[tracking] mozillaonline addons/firefox CN distribution

NEW
Unassigned

Status

defect
P2
normal
3 years ago
3 months ago

People

(Reporter: mixedpuppy, Unassigned)

Tracking

(Depends on 16 bugs, Blocks 1 bug, {meta})

49 Branch
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: triaged)

(Reporter)

Comment 1

3 years ago
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
(Reporter)

Updated

3 years ago
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

Updated

3 years ago
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
(Reporter)

Updated

2 years ago
Blocks: 1303905
TCPSocket will be necessary for our Email notifier extension which has a very minimal implementation of IMAP.
Depends on: 1247628
(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

Updated

2 years ago
Depends on: 1331559

Updated

2 years ago
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

Comment 12

2 years ago
(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

Updated

2 years ago
Keywords: meta
Priority: -- → P2

Updated

a year ago
Depends on: 1394304, 1420969
Depends on: 1315616
No longer depends on: 1318532, 1316097
Depends on: 1344749, 1412263

Updated

10 months ago
Product: Toolkit → WebExtensions
Bulk move of bugs per https://bugzilla.mozilla.org/show_bug.cgi?id=1483958
Component: Untriaged → General
No longer depends on: 1383939
You need to log in before you can comment on or make changes to this bug.