Page MenuHomeElementl

Add step contextual menus, url-persisted pagination to partition matrix

Authored by bengotow on May 5 2021, 2:25 AM.
Referenced Files
F2777292: D7736.diff
Wed, Feb 8, 5:20 PM
Unknown Object (File)
Mon, Jan 30, 7:10 PM
Unknown Object (File)
Sun, Jan 29, 8:50 PM
Unknown Object (File)
Mon, Jan 23, 5:15 AM
Unknown Object (File)
Fri, Jan 20, 12:49 PM
Unknown Object (File)
Sun, Jan 15, 6:29 PM
Unknown Object (File)
Sat, Jan 14, 10:25 PM
Unknown Object (File)
Jan 9 2023, 9:31 PM



This diff adds a contextual menu to the red/green squares on the partition matrix that replaces the modal click behavior and makes it much easier to jump to the step logs in the most recent run (the common case). It's still two clicks, but the clicks are right next to each other! (

I also switched the "step logs" click handlers to use standard links so that command-clicking on the menu item / button opens the run logs in a new tab.

This diff also moves partition matrix pagination state to the URL so that if you click Prev a few times and navigate away and come back, you're still on the same page. ( This actually exposed a few bugs in the useQueryPersistedState hook which I fixed and added test cases for. Now if you memoize the "options" passed to the hook, the setState method it returns remains identical between renders (matching the React.useState behavior more closely)

image.png (470×734 px, 68 KB)

Test Plan

Run two new tests

Diff Detail

R1 dagster
Lint Not Applicable
Tests Not Applicable