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.

Sign up for GitHub

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

Jump to bottom

fix(No Paste): Fixed the code to support paste manually with long press. #491

Closed
wants to merge 1 commit into from
Closed

fix(No Paste): Fixed the code to support paste manually with long press. #491

wants to merge 1 commit into from

Conversation

userAdityaa
Copy link
Contributor

PR Description

This PR addresses an issue where users were encountering difficulties pasting content into the params and headers fields.

Before

Screen.Recording.2024-11-06.at.6.44.15.PM.mov

After

Screen.Recording.2024-11-06.at.6.42.54.PM.mov

Checklist

  • I have gone through the contributing guide
  • I have updated my branch and synced it with project main branch before making this PR
  • I have run the tests (flutter test) and all tests are passing

Added/updated tests?

We encourage you to add relevant test cases.

  • Yes
  • No, and this is why: please replace this line with details on why tests have not been included
    This modification don't need any append in test.

@userAdityaa
Copy link
Contributor Author

@ashitaprasad, Can you please review it once.

@ashitaprasad
Copy link
Member

This is not the correct approach as it will lead to the removal of environment variable feature support.
Also, copy-paste works in desktop, but the issue is in iOS which needs to be investigated.

@userAdityaa
Copy link
Contributor Author

This is not the correct approach as it will lead to the removal of environment variable feature support.

Yes, I thought so.

@userAdityaa
Copy link
Contributor Author

Also, copy-paste works in desktop, but the issue is in iOS which needs to be investigated.

It's not working as you can see in the "Before" part.

@ashitaprasad
Copy link
Member

Also, copy-paste works in desktop, but the issue is in iOS which needs to be investigated.

It's not working as you can see in the "Before" part.

Command+V / Ctrl+V works ... Right click -> Paste not working is a nice find ... it means you have definitely isolated the issue across all platforms

@ashitaprasad ashitaprasad mentioned this pull request Nov 8, 2024
Open
@ashitaprasad
Copy link
Member

Closing this PR. Feel free to raise a new PR when the issue is resolved. Thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Paste not working on iOS and desktop (Right Click -> Paste)
2 participants
@userAdityaa @ashitaprasad