Closed Bug 1479129 Opened 2 years ago Closed 2 years ago

Implement support for updateProperties.highlighted in browser.tabs.update()

Categories

(WebExtensions :: General, enhancement, P5)

enhancement

Tracking

(firefox63 fixed)

RESOLVED FIXED
mozilla63
Tracking Status
firefox63 --- fixed

People

(Reporter: Oriol, Assigned: Oriol)

References

Details

(Keywords: dev-doc-complete)

Attachments

(1 file, 1 obsolete file)

browser.tabs.update() should be able to change whether a tab is highlighted.

Currently it's not supported.

https://searchfox.org/mozilla-central/rev/704612cf4426f0f0510b3e160895578c319a3270/browser/components/extensions/schemas/tabs.json#834-835
Priority: -- → P5
Severity: normal → enhancement
Assignee: nobody → oriol-bugzilla
Status: NEW → ASSIGNED
Comment on attachment 9002318 [details]
Bug 1479129 - Implement support for updateProperties.highlighted in browser.tabs.update(). r=mixedpuppy

Not sure why phabricator created a different patch
Attachment #9002318 - Attachment is obsolete: true
Comment on attachment 9002489 [details]
Bug 1479129 - Implement support for updateProperties.highlighted in browser.tabs.update(). r=mixedpuppy,jaws

Shane Caraveo (:mixedpuppy) has approved the revision.
Attachment #9002489 - Flags: review+
:jaws is away. Do I need another reviewer for the tabbrowser changes or does your approval suffice?
Flags: needinfo?(mixedpuppy)
I think someone should, ask Gijs if he can review.
Flags: needinfo?(mixedpuppy) → needinfo?(gijskruitbosch+bugs)
Flags: needinfo?(gijskruitbosch+bugs)
Comment on attachment 9002489 [details]
Bug 1479129 - Implement support for updateProperties.highlighted in browser.tabs.update(). r=mixedpuppy,jaws

:Gijs (he/him) has approved the revision.
Attachment #9002489 - Flags: review+
Keywords: checkin-needed
Pushed by csabou@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/073b8e49194a
Implement support for updateProperties.highlighted in browser.tabs.update(). r=mixedpuppy,Gijs
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/073b8e49194a
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla63
Keywords: dev-doc-needed
Can you please some STRs to this issue(and a test webextension if possible) or mark it as "qe-verify-" ?
Flags: needinfo?(oriol-bugzilla)
Covered by automated tests
Flags: needinfo?(oriol-bugzilla) → qe-verify-
Updated the browser compatibility data for highlighted and added this to the release notes:

tabs.update supports changing the selection status of a tab by including highlighted: true in the updateProperties parameter (bug 1479129).
Flags: needinfo?(oriol-bugzilla)
That's good
Flags: needinfo?(oriol-bugzilla)
One thing I didn't ask is whether the support for Firefox on Android is the same. If not, should it be supported but with a note or not supported?
Flags: needinfo?(oriol-bugzilla)
QA Contact: ddurst
Android does not support multi-tab selections, so updateProperties.highlighted can't work there. It's not supported.
https://searchfox.org/mozilla-central/rev/924e3d96d81a40d2f0eec1db5f74fc6594337128/mobile/android/components/extensions/schemas/tabs.json#648
Flags: needinfo?(oriol-bugzilla)
QA Contact: ddurst
You need to log in before you can comment on or make changes to this bug.