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/improv: Design consistency, additional user guides and recreate brand new html #29

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

Chandima1990
Copy link

@Chandima1990 Chandima1990 commented Nov 13, 2023

Hi, I am Chandima, working as a full-stack developer with Dotnet and Angular. I found your effort on this project very thrilling, and I enjoyed a lot playing around with it.

I have made some improvements that I felt were nice to have. But as I am not an expert on NextJs, I might have made mistakes in the framework usage and best practices. Please add any comments you might see that I can work with and improve so I can correct them.

These are the features I have added to this PR.

  • Even though the drawing is the same, clicking the "make real" button multiple times made different HTML designs. This prevented the user from improving the same design more and coming up with a better-fitting design. I have sent the API the current HTML generated and updated the prompt to have that HTML and upgrade it based on the user's drawing.

Note : This might be a bit costly as the token count is going to be high with bigger designs. So maybe later we can let the user decide whether they need to continue with the same design. Or if GPT implements a solution to maintain consistency, it would be a very nice one to have. But for the time being, addition 3 will be helpful to prevent this

  • No matter how clear the design drawings are, the GPT will mess up the design and create something not requested. (Vertical and horizontal placements are messed up, etc...) So I added an additional guides section so the user can give specific guides if the GPT keeps failing at a certain design.

  • If the user feels like the design is completely off and not getting any better, there is an option to select a checkbox and let GPT know to trash the current HTML and go with a new one.

Option 2 and 3 are getting reset after a successful generation since they have already catered the instructions into the HTML and will no longer be needed.

This is the feature in action.

Create.Next.App.-.Brave.2023-11-14.01-32-50.mp4

I hope you find these improvements helpful to the users and a good addition to your repo.

Have a nice day!

 and instructed to maintain the consistency
So if the AI mess up somewhere user can emphasis points.
For the user to give provision to regenerate the html
@Chandima1990 Chandima1990 changed the title Design consistency, additional user guides and recreate original html Design consistency, additional user guides and recreate brand new html Nov 13, 2023
@Chandima1990 Chandima1990 changed the title Design consistency, additional user guides and recreate brand new html feat/improv: Design consistency, additional user guides and recreate brand new html Nov 13, 2023
app/api/toHtml/route.ts Outdated Show resolved Hide resolved
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.

3 participants