Closed Bug 2029099 Opened 2 months ago Closed 2 months ago

tabs.group() does not work with split views

Categories

(WebExtensions :: Frontend, defect, P2)

defect

Tracking

(firefox151 fixed)

RESOLVED FIXED
151 Branch
Tracking Status
firefox151 --- fixed

People

(Reporter: robwu, Assigned: robwu)

References

(Blocks 1 open bug)

Details

(Keywords: dev-doc-complete, Whiteboard: [addons-jira])

Attachments

(1 file)

tab.group() does not work with split views. It may separate split views, or fail outright with "An unexpected error occurred".

Originally reported at https://github.com/piroor/treestyletab/issues/3806#issuecomment-4115398870

I've realized that browser.tabs.group() cannot create new group with split view (on Nightly 151.0a1). For example, assume there are tabs:

  • tab#10(splitViewId=1)
  • tab#11(splitViewId=1)
  • tab#12(splitViewId=-1)

Results:

  • browser.tabs.group({ tabIds: [10, 11, 12], createProperties: { windowId: 1 } }) => unexpected error
  • browser.tabs.group({ tabIds: [10, 11], createProperties: { windowId: 1 } }) => unexpected error
  • browser.tabs.group({ tabIds: [11, 12], createProperties: { windowId: 1 } }) => unexpected error
  • browser.tabs.group({ tabIds: [12], createProperties: { windowId: 1 } }) => success

I successfully add split view tabs to an existing group, but Firefox automatically separate split tabs.

  • browser.tabs.group({ tabIds: [10, 11], groupId: 1 }) => success, but tab#10 and tab#11 become regular tabs.
  • browser.tabs.group({ tabIds: [10], groupId: 1 }) => success, but only tab#10 is grouped, and both tab#10 and tab#11 become regular tabs.
Blocks: 2029534
Severity: -- → S3
Status: NEW → ASSIGNED
Priority: -- → P2
Keywords: dev-doc-needed
Status: ASSIGNED → RESOLVED
Closed: 2 months ago
Resolution: --- → FIXED
Target Milestone: --- → 151 Branch

Documentation updates available in Release 151 split-view tab group and move fixes #43821

You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: