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

Add floating toolbar display mode option #747

Merged
merged 1 commit into from
Jul 9, 2024

Conversation

gadelkareem
Copy link
Contributor

@gadelkareem gadelkareem commented Jul 8, 2024

Summary of changes:
This commit introduces a new display mode option for the ChatGPT extension, allowing users to choose between a sidebar and a floating toolbar. The changes span across three files, implementing the new feature and updating the configuration options.

Key changes:

  1. In src/config/index.mjs:

    • Added a new DisplayMode object with two options: sidebar and floating toolbar.
    • Updated the defaultConfig object to include a displayMode property.
  2. In src/content-script/index.jsx:

    • Implemented logic to render either the existing sidebar or the new floating toolbar based on the user's display mode preference.
    • Added code to create and position the floating toolbar container.
  3. In src/popup/sections/GeneralPart.jsx:

    • Added a new dropdown menu in the extension's settings to allow users to select their preferred display mode.

Overall, this commit enhances the extension's flexibility by giving users the option to choose how they want the ChatGPT interface to be displayed on web pages.

@josStorer josStorer merged commit b96ba7c into josStorer:master Jul 9, 2024
1 check passed
PeterDaveHello added a commit to PeterDaveHello/chatGPTBox that referenced this pull request Jul 21, 2024
josStorer added a commit that referenced this pull request Jul 30, 2024
Co-authored-by: josc146 <josStorer@outlook.com>
@josStorer
Copy link
Owner

josStorer commented Aug 8, 2024

f50249e 226fb10

Made some improvements, changed to a switch option, and made the effect comply with the user's trigger mode settings.

image

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.

2 participants