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

UX Optimization: When pasting the entire URL, it supposed to auto remove the extra protocol head from clipboard. #1761

Closed
4 of 5 tasks
PrinOrange opened this issue Nov 23, 2024 · 3 comments · Fixed by #1762

Comments

@PrinOrange
Copy link
Contributor

Describe the bug

在添加 RSS 的 URL 输入框,因为输入框里本身就有 https:// 前缀。但是,如果用户复制的 URL 中如果也有 https:// 前缀,直接粘贴时会出现冗余,还需要手动删除多余的前缀。

如下视频所示。

aaaaaa.mp4

可能的解决方案:监听 Input 的粘贴事件,如果粘贴时检测到剪切板里的 URL 也有 https://rsshub:// 前缀,就自动调整粘贴内容,避免多余的前缀。

Feed Info

NULL

Reproduction Video

No response

Environment

No response

Validations

  • Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
  • Check that this is a concrete bug. For Q&A, please open a GitHub Discussion instead.
  • This issue is valid

Contributions

  • I am willing to submit a PR to fix this issue
  • I am willing to submit a PR with failing tests (actually just go ahead and do it, thanks!)
PrinOrange added a commit to PrinOrange/Follow that referenced this issue Nov 23, 2024
…o remove the extra protocol head from clipboard.

Fixes RSSNext#1761
@PrinOrange
Copy link
Contributor Author

I have fixed this issue in pr #1762

@kowyo
Copy link

kowyo commented Nov 30, 2024

I noticed this as well. But later I found that actually we can paste RSS url (containing https://) directly in the 搜索 part to follow. So this issue might be unnecessary.

lawvs pushed a commit to PrinOrange/Follow that referenced this issue Dec 2, 2024
…o remove the extra protocol head from clipboard.

Fixes RSSNext#1761
lawvs pushed a commit that referenced this issue Dec 2, 2024
* fix: UX Optimization: When pasting the entire URL, it supposed to auto remove the extra protocol head from clipboard.
Fixes #1761

* refractor: remove `useEffect` listening for keyword change. Use `onChange` handle instead.
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 a pull request may close this issue.

2 participants