Preserve Dashboard batch selections when switching mobile/desktop modes.

Review Request #10746 — Created Oct. 9, 2019 and submitted

Information

Review Board
release-4.0.x
6520549...

Reviewers

When the Dashboard switches between mobile and desktop modes, the batch
selection state gets all messed up. The internal model knows what's
supposed to be selected, but the checkboxes don't reflect it. The reason
is that the datagrid, behind the scenes, keeps up to two rendered copies
of the table: one for mobile, one for desktop. This means we have two
separate set of checkboxes to deal with, not one.

Now, whenever we switch modes, we re-apply the model's version of the
selection back to the page, ensuring it's always consistent.

Manually tested by going back-and-forth between modes, toggling
checkboxes each time. Verified that the model and checkboxes were always
in sync.

Unit tests unfortunately cannot be provided due to limitations in the
base DataGrid support, preventing the simulation of mobile/desktop mode.

Description From Last Updated

Col: 11 Missing semicolon.

reviewbotreviewbot
Checks run (1 failed, 1 succeeded)
flake8 passed.
JSHint failed.

JSHint

chipx86
david
  1. Ship It!
  2. 
      
chipx86
Review request changed

Status: Closed (submitted)

Change Summary:

Pushed to release-4.0.x (bd855c0)
Loading...