Skip to content

graft batch 22: 1 land from backlog-1 - #185

Closed
thedancingdeveloper wants to merge 1 commit into
mainfrom
graft/batch-22
Closed

thedancingdeveloper wants to merge 1 commit into
mainfrom
graft/batch-22

Conversation

@thedancingdeveloper

Copy link
Copy Markdown
Collaborator

Batch 22 of graft line backlog-1, one commit per land, oldest first, by graft-mirror (decision D26). Merge it with rebase-merge to keep the commits (D28). graft never merges or queues this PR.

  • Base: main at e914e0b (graft c16d6868d7021001492e3d5f9b88334a7d06c05bf5d5c1e35353150c540dea807)
  • Lands: 1 (c9baf6155e6aa15bff623bbd9a5d9fd67550a602211bf652b636313f09efa9609..c372bab851b322eca9bc34eb9f077801b3818544781bd4118109675d7fd5769b5)
Changeset Author Session Work item Attestations Subject
c372bab851b32 thedancingdeveloper ses_01M65Z15N2Z0CBQ1V8ZN5CHVYK - none Let listbox popups take input above a modal Dialog

Graft-Batch: 22
Graft-Line: backlog-1
Graft-Changeset: c372bab851b322eca9bc34eb9f077801b3818544781bd4118109675d7fd5769b5

The Dropdown/Combobox/TagPicker listbox draws on its own Foreground layer that
was not in egui's area order, so a Dialog's modal layer blocked its clicks.
Register it as a sublayer of the opening layer (or move it to top over the
page) and set its area state for hit testing. Adds overlay_in_dialog tests.

Graft-Changeset: c372bab851b322eca9bc34eb9f077801b3818544781bd4118109675d7fd5769b5
Graft-Session: ses_01M65Z15N2Z0CBQ1V8ZN5CHVYK
Claimed-Author: thedancingdeveloper <306930456+thedancingdeveloper@users.noreply.github.com>
Graft-Batch: 22
@thedancingdeveloper

Copy link
Copy Markdown
Collaborator Author

Review blocked: overlay tests fail with the fix. Re-cut after the fix lands.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant