Several updates to web project #718
Conversation
Lamothe
commented
May 6, 2024
Lamothe
commented
- Updated to .NET 8
- Updated to Bootstrap 5.3.3
- Use libman for client side packages
- Enabled razor runtime compilation in dev
- Added fav icon
- Removed the footer
- Added theme switcher
* Updated to Bootstrap 5.3.3 * Use libman for client side packages * Enabled razor runtime compilation in dev * Added fav icon * Removed the footer * Added theme switcher
|
For your consideration. This will be the first of many updates that I want to make to the website sample. Please note that .NET 7 is out of support in less than 2 weeks so I updated it to .NET 8 while I was in there. |
|
Hi, thank you for this contribution! However more than 2000 files are included in this PR, in which most of them are resource files such as icon. Is it possible to exclude them in git and fetch the files when building it instead? |
|
Agreed, especially because I'm only using 6 of those icons. This is my first time using the new Bootstrap Icons, I've only used Font Awesome icons in the past. There's a way that I can get what I need from a CDN without pulling the entire module which includes the individual icon files. I'll do that for now and see if that helps. |
There was a problem hiding this comment.
The overall looks good to me except a few concerns. Could you please also update the LLama.Web readme since this PR has changed the UI?
Co-authored-by: Rinne <[email protected]>
|
it looks ok to me. it seems that there is a conflict in ModelOptions.cs. Could you solve this one before merge the PR? I was thinking on introducing the capability to use Multi-Modal models on the Web UI. I would like to know your plans to change the Web UI to not duplicate work. |
|
Merging this PR now, thank you a lot for the contribution! |
