HomePhabricator

Really fix movable panels inside tab panels by changing the JX.Request…

Description

Really fix movable panels inside tab panels by changing the JX.Request serializer

Summary:
Depends on D20475. Ref T13272. Currently, if you JX.Request with data like {x: null}, we submit that as ?x=null, i.e. as though null was the string "null".

This is weird and almost certainly never intended/desiarable. In particular, it causes a bug where panels embedded inside tab panels are incorrectly draggable.

It's possible this breaks something which relied on the buggy behavior, but that seems unlikely.

Test Plan: Tried to drag a panel inside a tab panel, it really truly didn't work.

Reviewers: amckinley

Reviewed By: amckinley

Maniphest Tasks: T13272

Differential Revision: https://secure.phabricator.com/D20476

Details

Provenance
epriestleyAuthored on Apr 25 2019, 6:57 PM
epriestleyPushed on May 1 2019, 10:38 PM
Reviewer
amckinley
Parents
rPc43618a3a8bb: Add "Move Left" and "Move Right" to dashboard tab panels
Branches
Unknown
Tags
Unknown
Tasks
T13272: Remove the hard-coded homepage, and other Dashboards improvements
Build Status
Buildable 22754
Build 31197: Run Core Tests