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

Adresses should automatically enter a new row after pasting #29

Closed
ghost opened this issue Mar 12, 2015 · 3 comments
Closed

Adresses should automatically enter a new row after pasting #29

ghost opened this issue Mar 12, 2015 · 3 comments

Comments

@ghost
Copy link

ghost commented Mar 12, 2015

Otherwise the next pasted address is trailed to the previous

@MrS0m30n3
Copy link
Owner

With the current code state this is not possible (as far as i can think of right now) without keeping the insert state in a third variable (basically you can use the wx IsModified() method to detect when the text is modified by the user but even then you need to also detect the backspace else a new line gets added when you press backspace) and i am not going to edit the EVT_TEXT handler, so just hit Enter.

This behavior will change on the new GUI where you will have a single line textctrl and when you paste the url the textctrl will autoclear itself to be ready for the next url.

UPDATE
Basically it was easier than i though i just had to overwrite the default Paste() method. I don't really like using the clipboard but now it should automatically add a newline after pasting.

I am closing this issue.

@ghost
Copy link
Author

ghost commented Mar 16, 2015

Thanks, works when pasting via context menu, but not when pasting selected text with middle mouse button, which is a linux feature

@MrS0m30n3 MrS0m30n3 reopened this Mar 17, 2015
@MrS0m30n3
Copy link
Owner

@ThebestwaytodesignJoinGitHub

I am closing this issue.
Now the middle mouse click automatically adds a newline when you paste urls.

Give it a try and feel free to re-open the issue if you have any problems.

jose1711 pushed a commit to jose1711/youtube-dl-gui that referenced this issue Jun 29, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants