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

[web] edit styles #168

Merged
merged 39 commits into from
Jul 1, 2023
Merged

[web] edit styles #168

merged 39 commits into from
Jul 1, 2023

Conversation

qizheng7
Copy link
Collaborator

@qizheng7 qizheng7 commented Jun 24, 2023

  • Change the layout and style of the dashboard page
  • Change the layout and style of the agent profile page
  • Change the layout and style of the agent profile edit page
  • Change the layout, the style and the functionality of the create conversation page

@vercel
Copy link

vercel bot commented Jun 24, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
skyagi ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 30, 2023 5:21am

Comment on lines 196 to +201
{#if $isAgentFormEditing}
<Button type="button" class="" color="light" on:click={() => isAgentFormEditing.set(false)}
>Cancel</Button
>
{/if}
<Button type="submit" class="">Submit</Button>
<Button type="submit" size="xl">Submit</Button>
Copy link
Collaborator

Choose a reason for hiding this comment

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

nit: keep the size/style of the buttons consistent

</div>
<div class="scroller">
{#each characters as character, i}
<a href="agent/{character.id}">
Copy link
Collaborator

Choose a reason for hiding this comment

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

The focus area of the avatar seems too larger than the size of the avatar.
Like if click the area in the middle of C and A-Serena, the agent page will be opened.
Tagging @serenaqiaoo myself here so I will fix this in another PR.
Screenshot 2023-06-24 at 1 20 00 AM

@qizheng7 qizheng7 marked this pull request as ready for review June 30, 2023 05:21
@qizheng7 qizheng7 merged commit 9f5d47d into main Jul 1, 2023
@qizheng7 qizheng7 deleted the qiz/style branch July 1, 2023 02:54
@qizheng7 qizheng7 linked an issue Jul 1, 2023 that may be closed by this pull request
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.

[web] UI standarization
2 participants