Closed Bug 2019925 Opened 18 days ago Closed 11 days ago

Using the `Move tab to Split View` option on a Split View that is not in a group will not displaye the Split View as expected

Categories

(Firefox :: Tabbed Browser: Split View, defect, P1)

Desktop
All
defect

Tracking

()

VERIFIED FIXED
150 Branch
Tracking Status
firefox149 --- verified
firefox150 --- verified

People

(Reporter: atrif, Assigned: kcochrane)

References

(Blocks 1 open bug)

Details

(Whiteboard: [fidefe-splitview])

Attachments

(3 files)

Attached image split_7.gif

Found in

  • 149.0b1

Affected versions

  • 150.0a1 (2025-01-26)
  • 149.0b1

Tested platforms

  • Affected platforms: Ubuntu 24, Windows 11, macOS 14
  • Unaffected platforms: none

Steps to reproduce

  1. Create a Split View with YouTube and Reddit and add it to a group.
  2. Outside the group, create another Split View and use move tab to Split View option on Reddit or YouTube.

Expected result

  • The Split View is displayed as expected.

Actual result

  • The Split View is not displayed as expected.

Regression range

  • Not a regression. Happens after implementation of the option (bug 2003892)

Additional notes

  • Attached a screen recording.
Severity: -- → S3
Priority: -- → P1
Assignee: nobody → kcochrane

firefox-beta Uplift Approval Request

  • User impact if declined: Split view panels may be displayed unexpectedly when moving a tab from one split view to another using the urlbar 'Move tab to split view' action.
  • Code covered by automated testing: yes
  • Fix verified in Nightly: no
  • Needs manual QE test: yes
  • Steps to reproduce for manual QE testing: Create a split view
    Right click a tab outside of the split view and select "Add Split View"
    In the urlbar for the new split view, select one of the tabs from the first split view ensuring the "Move tab to split view" indicator is shown for that item in the urlbar results
    Ensure both panels are displayed at roughly 50% width after the tab has moved to the new split view
  • Risk associated with taking this patch: low
  • Explanation of risk level: 6 line change in tabsplitview.js covered by automated tests
  • String changes made/needed: N/A
  • Is Android affected?: no
Attachment #9550140 - Flags: approval-mozilla-beta?
Flags: qe-verify+
Pushed by kcochrane@mozilla.com: https://github.com/mozilla-firefox/firefox/commit/3e5bcbb12ff1 https://hg.mozilla.org/integration/autoland/rev/788374ba3539 Ensure split view tab panels are displayed properly when moving a tab from one split view to another split view r=tabbrowser-reviewers,jsudiaman
Pushed by abutkovits@mozilla.com: https://github.com/mozilla-firefox/firefox/commit/6aa2db43ecc8 https://hg.mozilla.org/integration/autoland/rev/f7d32736fa46 Revert "Bug 2019925 - Ensure split view tab panels are displayed properly when moving a tab from one split view to another split view r=tabbrowser-reviewers,jsudiaman" for causing failures at browser_switchTab_override.js.

Lets have QA verify this first before uplifting as there are a lot of overlapping changes landing atm.

Pushed by kcochrane@mozilla.com: https://github.com/mozilla-firefox/firefox/commit/256cc51a809f https://hg.mozilla.org/integration/autoland/rev/4e97fa600829 Ensure split view tab panels are displayed properly when moving a tab from one split view to another split view r=tabbrowser-reviewers,jsudiaman
Flags: needinfo?(kcochrane)
Status: NEW → ASSIGNED
Status: ASSIGNED → RESOLVED
Closed: 11 days ago
Resolution: --- → FIXED
Target Milestone: --- → 150 Branch
QA Whiteboard: [qa-found-in-b149] [S3][QA-4411] → [qa-found-in-b149] [S3][QA-4411][qa-ver-needed-c150/b149][uplift]

Verified fixed with Firefox 150.0a1 (2026-03-05) on Windows 11, macOS 26 and Ubuntu 24. The Split View is correctly displayed after following the steps from comment 0.

QA Contact: atrif
Attachment #9550140 - Flags: approval-mozilla-beta? → approval-mozilla-beta+

Verified fixed with Firefox 149.0b6 (20260309091903) on Windows 11, macOS 26 and Ubuntu 24. The Split View is correctly displayed after following the steps from comment 0.

Status: RESOLVED → VERIFIED
QA Whiteboard: [qa-found-in-b149] [S3][QA-4411][qa-ver-needed-c150/b149][uplift] → [qa-found-in-b149] [S3][QA-4411][qa-ver-done-c150/b149][uplift]
Flags: qe-verify+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: