Skip to content

refactor(ui): canvas flow #8069

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Draft
wants to merge 74 commits into
base: main
Choose a base branch
from
Draft

refactor(ui): canvas flow #8069

wants to merge 74 commits into from

Conversation

psychedelicious
Copy link
Collaborator

@psychedelicious psychedelicious commented May 30, 2025 •
edited
Loading

Summary

WIP

TODO:

  • Update all translation strings for canceling vs deleting queue items
  • Figure out how to delete staged images when discarding them

Related Issues / Discussions

QA Instructions

Merge Plan

Checklist

  • The PR has a short but descriptive title, suitable for a changelog
  • Tests added / updated (if applicable)
  • Documentation added / updated (if applicable)
  • Updated What's New copy (if doing a release after this PR)

@psychedelicious psychedelicious marked this pull request as draft May 30, 2025 14:02
@github-actions github-actions bot added frontend-deps PRs that change frontend dependencies frontend PRs that change frontend files api python PRs that change python files services PRs that change app services labels May 30, 2025
@psychedelicious psychedelicious force-pushed the psyche/refactor/ui/canvas-viewer-flow branch 4 times, most recently from 18319ba to 965bf02 Compare June 6, 2025 10:47
@psychedelicious
feat(ui): move socket events handling into ctx component
82da4ed
@psychedelicious
feat(ui): prevent flicker of image action buttons
e6911e6
@psychedelicious
feat: canvas flow rework (wip)
dec8958
@psychedelicious
docs(ui): add comment about auto-switch not being quite right yet
b155e4a
@psychedelicious
feat(ui): tweak canvas scroll to zoom feel
ea319eb
@psychedelicious
feat(app): support deleting queue items by id or destination
54c219c
@psychedelicious
feat(ui): rough out canvas staging area
0c66169
@psychedelicious
chore(ui): lint (partial cleanup)
5318d2f
@psychedelicious
feat(ui): update canvas session state handling for new staging strat
92f7962
@psychedelicious
feat(ui): more staging fixes
4835a36
@psychedelicious
feat(ui): improved staging placeholders
d7a1ab0
@psychedelicious
feat(ui): improved staging placeholders
13d961f
@psychedelicious
fix(ui): reset layers when changing session type
559d39e
@psychedelicious
fix(ui): ensure canvas tool modules are destroyed
fb4c81d
@psychedelicious
feat(ui): add staging area toolbar to simple session
7727013
@psychedelicious
feat(ui): tweak staging image display
bcc9f65
@psychedelicious
feat(ui): when discarding last item, select new last instead of first
025c81b
@psychedelicious
feat(ui): finish generation when discarding last item
8c618c1
@psychedelicious
build(ui): temporarily ignore all knip issues
25959fc
@psychedelicious
fix(ui): hide layers when not on canvas tab
5ca17fa
@psychedelicious
fix(ui): mini preview bg color
64eb44a
@psychedelicious
refactor(ui): migrate from canceling queue items to deleteing, make q…
c634334 
…ueue hook APIs consistent
@psychedelicious
feat(ui): remove vary and edit as control buttons
745a608
@psychedelicious
feat(ui): simple session initial state
7a7d379
@psychedelicious
feat(ui): remove technical progress message from full preview
d3f926d
@psychedelicious
feat(ui): fiddle w/ staging area header
98870a0
@psychedelicious
feat(ui): add startover button to canvas toolbar
27fb7fd
@psychedelicious
feat(ui): make main panel styling and title consistent
2a072f6
@psychedelicious
chore(ui): lint
0dea63e
@psychedelicious psychedelicious force-pushed the psyche/refactor/ui/canvas-viewer-flow branch from 965bf02 to 0dea63e Compare June 6, 2025 10:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@blessedcoolant blessedcoolant Awaiting requested review from blessedcoolant blessedcoolant is a code owner

@maryhipp maryhipp Awaiting requested review from maryhipp maryhipp is a code owner

@hipsterusername hipsterusername Awaiting requested review from hipsterusername hipsterusername is a code owner

@jazzhaiku jazzhaiku Awaiting requested review from jazzhaiku jazzhaiku will be requested when the pull request is marked ready for review jazzhaiku is a code owner

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
api frontend PRs that change frontend files frontend-deps PRs that change frontend dependencies python PRs that change python files services PRs that change app services
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@psychedelicious