Skip to content

Unify button look and rearrange buttons to make them more compact#712

Merged
cjpais merged 4 commits intomozilla-ai:mainfrom
corebonts:button-arrangement
Mar 14, 2025
Merged

Unify button look and rearrange buttons to make them more compact#712
cjpais merged 4 commits intomozilla-ai:mainfrom
corebonts:button-arrangement

Conversation

@corebonts
Copy link
Copy Markdown
Contributor

@corebonts corebonts commented Mar 12, 2025

  • Unify icons (fontawesome public domain fonts)
  • Unify button sizes and alignment
  • Make the button arrangement beside the input more compact
  • Add icon for the menu-trigger and fix it's alignment
  • Fix the wrapping of the title (it overflown when using ollama sha-named model files)

image

@cjpais
Copy link
Copy Markdown
Collaborator

cjpais commented Mar 13, 2025 •
edited
Loading

Thanks! This will likely make it into the next release when I do my testing for it and see how it compares to the current. As far as I can see from the code and screenshot it will make it.

Will need to check on the icons, could you give the link to the source?

@corebonts
Copy link
Copy Markdown
Contributor Author

corebonts commented Mar 13, 2025

Sure! They are part of this collection: https://fontawesome.com/search?o=r&ic=free&s=solid&ip=classic
License: https://fontawesome.com/license/free

Attribution is required by MIT, SIL OFL, and CC BY licenses. Downloaded Font Awesome Free files already contain embedded comments with sufficient attribution, so you shouldn't need to do anything additional when using these files normally.

@corebonts
Copy link
Copy Markdown
Contributor Author

corebonts commented Mar 13, 2025 •
edited
Loading

I just learned that the "completion" UI has different buttons so I also modify that.

...done
image

@corebonts
Apply styles from the chat mode to the completion mode also
43d8f44 
- images
- button styles
- textarea style
@cjpais
Copy link
Copy Markdown
Collaborator

cjpais commented Mar 14, 2025

looks great, thanks

@cjpais cjpais merged commit c344ac7 into mozilla-ai:main Mar 14, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

llamafile

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@corebonts @cjpais