k0gen / serge

A web interface for chatting with Alpaca through llama.cpp. Fully dockerized, with an easy to use API.
https://serge.chat
MIT License
0 stars 0 forks source link

Automatically reveal Model Settings when Settings are clicked in the sidebar #7

Open sweep-ai[bot] opened 1 year ago

sweep-ai[bot] commented 1 year ago

Description

This PR updates the code in web/src/routes/+layout.svelte to automatically reveal the "Model Settings" section when the "Settings" button is clicked in the sidebar. Currently, the "Model Settings" section is not automatically shown when the "Settings" button is clicked. This PR modifies the click event handler for the "Settings" button to toggle the visibility of the "Model Settings" section.

Summary of Changes

Please review and merge this PR to automatically reveal the "Model Settings" section when the "Settings" button is clicked in the sidebar.

Fixes #6.


To checkout this PR branch, run the following command in your terminal:

git checkout sweep/automatically-reveal-model-settings

🎉 Latest improvements to Sweep:


💡 To get Sweep to edit this pull request, you can:

k0gen commented 1 year ago

The Model Settings checkbox is still not collapsing also don't add comments to the code

sweep-ai[bot] commented 1 year ago

🚀 Wrote Changes

The Model Settings checkbox is still not collapsing also don't add comments to the code

Hi @k0gen,

I decided to make the following changes:

File Path Proposed Changes
web/src/routes/+layout.svelte Modify web/src/routes/+layout.svelte with contents:
• Remove the comments in the code.
• Modify the toggleBar function to update the modelSettingsVisible statement when the Settings button is clicked.