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

Vision Support Update (0.0.4.5) #36

Merged
merged 30 commits into from
Jun 9, 2024
Merged

Vision Support Update (0.0.4.5) #36

merged 30 commits into from
Jun 9, 2024

Conversation

wz-ml
Copy link
Collaborator

@wz-ml wz-ml commented Jun 3, 2024

Feature update with image support and bugfixes! New additions:

  • Support for .jpeg / .png image uploads on both prompt-on-the-fly and chatGPT popups (including multi-image support).
  • A screenshot + snipping feature to make sending snips of your current webpage to chatGPT that much easier.
  • Added keyboard shortcuts for image uploading & screenshots.

Major Bugfixes:

  • Fixed a major bug introduced in the last update where opening a new chat popup while an existing one receives a stream causes the text stream to flow to the new one instead.
  • Fixed a bug regarding text streams not stopping after creating a popup after a refresh.
  • Promptless selection text now appears for both ChatGPT and prompt-on-the-fly interfaces.

Notes:

  • The snipping feature fails when the current tab's a PDF! Chrome renders PDFs in an iframe, which Chrome's CaptureVisibleTab is unable to capture. Seems difficult to work around. This may be intentional behavior on Chrome's part as a security feature to prevent extensions from reading iframe data.
  • I've done some manual bug-hunting, but I'm far from comprehensive. @giosilvi Let me know if any bugs appear! In particular, I haven't played around with the hotkeys, so testing on that would be nice.

wz-ml and others added 28 commits February 16, 2024 10:52
Merging GPT-4 turbo updates.
@wz-ml wz-ml added bug Something isn't working enhancement New feature or request labels Jun 3, 2024
@wz-ml wz-ml requested a review from giosilvi June 3, 2024 06:09
@wz-ml wz-ml self-assigned this Jun 3, 2024
Copy link
Owner

@giosilvi giosilvi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great so far!

Copy link
Owner

@giosilvi giosilvi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I pushed a fix to the title of the popup, to make it less clutter let's just use the last user message.

@giosilvi giosilvi merged commit 7f531dd into master Jun 9, 2024
3 checks passed
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants