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

Completed Components Folder Restructure and Added New MessageItem Component #147

Conversation

fingerthief
Copy link
Owner

@fingerthief fingerthief commented Jun 8, 2024

Pull Request Summary

Description:
This pull request includes a series of commits focused on restructuring the components folder, adding new features, and improving the overall application performance and user experience. Below is a summary of the changes:

  1. New Features:

    • Model Selection Sidebar: Introduced a new model selecting experience with a sidebar for mobile layout. This sidebar allows users to select between models and general configuration.
    • MessageItem Component: Added a new MessageItem component to represent a single message item. The previous MessageItem component has been renamed to MessagesList to accurately reflect its function.
  2. Improvements:

    • Selected State for Model Sections: Added a selected state for model sections in the new sidebar to indicate when a GPT or Claude model is selected, even when the list is collapsed.
    • Config Sections: Fixed issues related to expanding and closing configuration sections.
    • Message Layout Adjustments: Made slight adjustments to the maximum width of messages in both desktop and mobile layouts for better readability.
  3. Code Cleanup:

    • Unused Files and Packages: Removed unused files and uninstalled unused packages, including testing suites, to reduce overall application bloat.
    • SCSS Typing: Added SCSS lang type to all style tags for consistency across the application.
  4. Folder Restructure:

    • Components Folder: Completed the reorganization of the components folder to improve maintainability and clarity.

Impact:
These changes enhance the user experience, improve the maintainability and performance of the application, and ensure consistency in styling. The new model selection sidebar and MessageItem component add valuable functionality, while the cleanup and restructuring efforts reduce bloat and improve code organization.

New Mobile Model Selector

Auto Generated Notes (Do Not Change)


Contents (#147)

Uncategorised!

  • Finished organizing the components folder. Added a new MessageItem component representing a single message item.
  • Uninstalled unused packages like testing suites and delete associated files to reduce overall application bloat.
  • Adding a new model selecting experience with a sidebar when in mobile layout.
  • Added selected state for model sections in the new sidebar to indicate a GPT or Claude model is selected when the lsit is collapsed.
  • remove unused file
  • tweak mobile sidebar expanded item colors a bit

…mponent representing a single message item.

renamed previous MessageItem component to MessagesList to accurately reflect it's function.

added scss typing to all style tags for consistency.
…e a GPT or Claude model is selected when the lsit is collapsed.

Fixed issue expanding/closing config sections.

Slight adjustments to max width of messages in desktop and mobile layout
@fingerthief fingerthief added bug Something isn't working enhancement New feature or request Usability Issue with the usability of the application labels Jun 8, 2024
@fingerthief fingerthief added this to the 6.2.2 milestone Jun 8, 2024
@fingerthief fingerthief self-assigned this Jun 8, 2024
Copy link

github-actions bot commented Jun 8, 2024

Visit the preview URL for this PR (updated for commit a06327c):

https://minimalgpt--pr147-dev-completed-compon-a5pc8gwb.web.app

(expires Sat, 15 Jun 2024 13:20:20 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: 95b9f8da4861f0258375bb9789bfa436b4ed7d98

@fingerthief fingerthief merged commit f789511 into main Jun 8, 2024
6 checks passed
@fingerthief fingerthief deleted the dev/completed-components-folder-restructure-add-new-messageitem-component branch June 8, 2024 13:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request Usability Issue with the usability of the application
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

None yet

1 participant