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

UI Streamline Updates and Bug Fixes #77

Merged
merged 11 commits into from
Jan 5, 2025
Merged

Conversation

appatalks
Copy link
Owner

Lot's of changes! Mostly usability and got rid of stuff I didn't like. Trying new things.

CoPilot Generated Pull Request Description

This pull request includes significant changes to the JavaScript functions and CSS styles in the core directory. The most important changes involve the removal of the autoSelect function, updates to the lmsSend function, and several style adjustments to improve the user interface.

JavaScript changes:

  • core/js/lm-studio.js: The lmsSend function had several commented-out lines added, including different openAIUrl and model options. Additionally, the sendData function was commented out as it was deemed redundant. [1] [2]
  • core/js/options.js: The autoSelect function was completely removed, simplifying the logic for selecting models. The sendData function was updated to remove references to the autoSelect function. [1] [2]
  • core/js/options.js: Added checks in the insertImage function to ensure elements exist before proceeding, preventing potential errors.

CSS changes:

  • core/style.css: Added a new #settingsMenu style block to define the appearance and positioning of the settings menu. [1] [2]
  • core/style.css: Updated various styles to improve the layout and appearance of the chat input and buttons, including the addition of a .chat-input-container class and adjustments to the #btnSend and .mic-button styles. [1] [2]

@appatalks appatalks merged commit 3307e09 into main Jan 5, 2025
4 checks passed
@appatalks appatalks deleted the appatalks-settings-menu branch January 5, 2025 07:46
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.

1 participant