ExtensionPanel.onSearch event is unimplemented

NEW
Assigned to

Status

()

Toolkit
WebExtensions: Developer Tools
P2
normal
3 months ago
3 months ago

People

(Reporter: Wladimir Palant, Assigned: rpl)

Tracking

(Blocks: 2 bugs)

Trunk
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: triaged)

(Reporter)

Description

3 months ago
devtools.panel.create() API has been implemented in bug 1300587. However, while the ExtensionPanel.onSearch event has been added to the schema, the implementation is currently merely a TODO comment: https://dxr.mozilla.org/mozilla-central/rev/74566d5345f4cab06c5683d4b620124104801e65/browser/components/extensions/ext-c-devtools-panels.js#120. This causes an exception in Adblock Plus with the current Nightly build.
(Reporter)

Updated

3 months ago
See Also: → bug 1300587

Updated

3 months ago
Assignee: nobody → lgreco
Priority: -- → P2
Whiteboard: triaged
(Assignee)

Updated

3 months ago
Blocks: 1370525
(Assignee)

Updated

3 months ago
No longer blocks: 1211859
(Assignee)

Comment 1

3 months ago
The main reason why the panel.onSearch event wasn't part of the initial subset of devtools API is due to some differences in the "search feature" between the Firefox Developer Toolbox and the Chrome one:

- on Chrome when the developer tools are focused and the user press Ctrl-F, a find bar appears 
  (usually on the bottom on the panel, on some panel it appears in a different mode and/or position)
- on Firefox every panel defines and manages its own search field (and the keyboard shortcut)

My feeling is that to implement this API properly, we should evaluate the changes from a "DevTools toolbox point of view" first:

most of the changes are probably going to be part of the devtools toolbox itself (e.g. the additional XUL elements that compose its UI, listening to the keyboard shortcut to make this find bar visible, and emitting an event on the toolbox when the
search field has been updated), and then wrapped into the WebExtensions panel.onSearch event.

For the above reasons, I would like to evaluate and double-check the priority (and interest) related to the these toolbox changes with the DevTools Team first, and then we will update the priority of this WebExtensions API accordingly.

Needinfo Bryan, as we agreed during a brief chat on the topic, to determine who are the best contacts from the DevTools Team for additional feedbacks and thoughts on this.
Flags: needinfo?(clarkbw)
Yes, I'm adding in Patrick here. I'll also bring this up with Victoria (our UX designer).  I don't think we'll be able to make this happen until later in the year.  However I want to say that I agree that we should implement this kind of UI first and then we can hook up the API to it afterward.
Flags: needinfo?(clarkbw)
You need to log in before you can comment on or make changes to this bug.