Skip to content
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.

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

feat(ui): UX improvements & updated design #5270

Merged
merged 29 commits into from
Dec 29, 2023

Conversation

psychedelicious
Copy link
Collaborator

@psychedelicious psychedelicious commented Dec 12, 2023

What type of PR is this? (check all applicable)

  • Refactor
  • Feature
  • Bug Fix
  • Optimization
  • Documentation Update
  • Community Node Submission

Have you discussed this change with the InvokeAI team?

  • Yes
  • No, because:

Description

  • Add storybook
  • Add new primitive components
    • New components are prefixed with Inv, old IAI-prefixed components removed
    • Improved structure for primitive components, supporting future separate package
    • More complex components, like slider and number input, have refactored internal logic
    • New components added, most notably InvAutosizeTextarea and InvSelect
    • Removed all mantine components; still using mantine form in Model Manager tab
  • Styling updates
  • Numerous minor bug fixes
  • Lots of cleanup

QA Instructions, Screenshots, Recordings

You'll need to pnpm i to install updated dependencies.

To run storybook:

  • run pnpm storybook to start storybook
  • visit localhost:6006 (or whatever URL the terminal says) to preview at the storybook

Known Issues

Canvas bounding box is wonky. Will address in a followup.

Merge Plan

This PR can be merged once approved. It will leave main in a fully-functional state (excepting canvas issue described above), but with a WIP design.

Any remaining UX/redesign changes will need to be completed before the next full release.

@hipsterusername
Copy link
Member

Looking good! Got storybook to run, and was able to poke around. Let me know once there's more to check out!

@psychedelicious psychedelicious force-pushed the feat/ui/component-library-1 branch 5 times, most recently from a33f14a to 6c86824 Compare December 20, 2023 10:51
@psychedelicious psychedelicious force-pushed the feat/ui/component-library-1 branch 3 times, most recently from 2729d0b to b338787 Compare December 28, 2023 09:46
@psychedelicious psychedelicious marked this pull request as ready for review December 28, 2023 12:54
@psychedelicious psychedelicious changed the title feat(ui): component library feat(ui): UX improvements & updated design Dec 28, 2023
@hipsterusername hipsterusername merged commit 52f9749 into main Dec 29, 2023
7 checks passed
@hipsterusername hipsterusername deleted the feat/ui/component-library-1 branch December 29, 2023 13:26
@psychedelicious psychedelicious mentioned this pull request Jan 1, 2024
10 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: ✅ v3.6.x
Development

Successfully merging this pull request may close these issues.

2 participants