Skip to content

Ux changes - #9

Open
anaganisk wants to merge 1 commit into
mikeesto:masterfrom
anaganisk:master
Open

Ux changes#9
anaganisk wants to merge 1 commit into
mikeesto:masterfrom
anaganisk:master

Conversation

@anaganisk

@anaganisk anaganisk commented May 18, 2025

Copy link
Copy Markdown

Hey I made some UX changes.
Totally upto you if you want to merge.
A quick summary

  • Output is streamed in html rather than JSON dump.
  • Split the main container into 2 column flex.
  • made the model name env variable for switching models (make sure u add the environment variable. PUBLIC_GOOGLE_MODEL="")
  • I ran the formatter included in your package.json, so a few files have the formatting applied.

preview changes

@mikeesto mikeesto left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the PR @anaganisk . I like the improved JSON parsing and reset button. If you'd like to open a PR with just those changes, I'd be happy to merge it. Otherwise, the project is available under MIT if you'd like to maintain a fork with your preferred UI etc : )

@anaganisk

anaganisk commented May 19, 2025

Copy link
Copy Markdown
Author

Sure, will do.
Its simple enough to be a single project, don't think a fork is necessary for now.

@anaganisk

Copy link
Copy Markdown
Author

@mikeesto I updated the PR only with the UX changes without other formatting changes, I also noticed I forgot adding the footer earlier. I added it to the main card.

image

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.

2 participants