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

fix: When pasting the entire URL, auto adjust the URL prefix #1762

Merged
merged 2 commits into from
Dec 2, 2024

Conversation

PrinOrange
Copy link
Contributor

@PrinOrange PrinOrange commented Nov 23, 2024

Fixes #1761

Description

Fix the issue #1761

The before,

cccc.mp4

The after effect,

bbb.mp4

PR Type

  • Feature
  • Bugfix
  • Hotfix
  • Other (please describe):

Linked Issues

#1761

  • I have updated the changelog/next.md with my changes.

@follow-reviewer-bot
Copy link

Thank you for your contribution. We will review it promptly.

Copy link

vercel bot commented Nov 23, 2024

@PrinOrange is attempting to deploy a commit to the RSS3 Team on Vercel.

A member of the Team first needs to authorize it.

@follow-reviewer-bot
Copy link

Suggested PR Title:

feat(discover): add clipboard paste handling for keywords

Change Summary:
Enhanced DiscoverForm to handle paste events for keywords, improving user input experience. Validates prefix against clipboard data before setting form value.

Code Review:
No change requests necessary.

Copy link

Linting and formatting issues were automatically fixed. Please review the changes.

@hyoban
Copy link
Member

hyoban commented Nov 25, 2024

Unable to undo the paste operation

@lawvs lawvs force-pushed the PrinOrange/issue1761 branch from 18f4608 to face05f Compare December 2, 2024 08:59
Copy link
Contributor

@lawvs lawvs left a comment

Choose a reason for hiding this comment

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

Although this PR impacts the undo feature, it fixes bugs in the old code, so I think it's worth merging.

@lawvs lawvs merged commit 3c7aeac into RSSNext:dev Dec 2, 2024
4 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants