[dagit] Fix divider covering the “Launch Execution” button #4370
ClosedPublic
Actions

Authored by • bengotow on Jul 27 2021, 9:16 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Tue, Oct 4, 5:54 AM

	Unknown Object (File)
	Tue, Oct 4, 5:54 AM

	Unknown Object (File)
	Tue, Oct 4, 5:54 AM

	Unknown Object (File)
	Mon, Oct 3, 1:11 AM

	Unknown Object (File)
	Sat, Oct 1, 12:04 AM

	Unknown Object (File)
	Sun, Sep 25, 6:57 PM

	Unknown Object (File)
	Sat, Sep 24, 9:06 AM

	Unknown Object (File)
	Mon, Sep 19, 10:55 PM

View All 46 Files

Subscribers

None

Details

Reviewers

• dish

Commits

R1:0c4c67051738: [dagit] Fix divider covering the “Launch Execution” button #4370

Summary

Fix z-index issue by moving the Launch Execution action button up out of the split panel

Test Plan

Run dagit, visit page

Diff Detail

Repository

R1 dagster

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

• bengotow created this revision.Jul 27 2021, 9:16 PM

Harbormaster completed remote builds in B34333: Diff 42437.Jul 27 2021, 9:28 PM

• bengotow requested review of this revision.Jul 27 2021, 9:28 PM

• dish accepted this revision.Jul 28 2021, 2:24 PM

This revision is now accepted and ready to land.Jul 28 2021, 2:24 PM

Closed by commit R1:0c4c67051738: [dagit] Fix divider covering the “Launch Execution” button #4370 (authored by • bengotow). · Explain WhyJul 29 2021, 3:03 PM

This revision was automatically updated to reflect the committed changes.

• bengotow added a commit: R1:0c4c67051738: [dagit] Fix divider covering the “Launch Execution” button #4370.

Revision Contents
Changeset List

Path

Size

js_modules/

dagit/

packages/

core/

src/

execute/

ExecutionSessionContainer.tsx

16 lines

RunPreview.tsx

3 lines

Diff 42579

View Options

js_modules/dagit/packages/core/src/execute/ExecutionSessionContainer.tsx