Skip to content

Conversation

@AmberByte
Copy link
Contributor

Fixed a text selection bug where a user could not select text in the terminal.
This is really annoying as someone may like to copy the text a command returns to paste it somewhere else.

The solution is quite simple, It just checks now If the user selected anything
If it's not empty, setting the focus is skipped.

Tests still pass.

@ndabAP
Copy link
Owner

ndabAP commented Dec 9, 2020

Hey @TobiasBeck, thanks a lot for that. Could you rewrite your Git history so it uses semver. It's necessary for automatic library building.

I wonder why the Git hooks doesn't alarm when you don't follow the schema (should be). But that's another topic.

@AmberByte
Copy link
Contributor Author

@ndabAP Hey yea, of course, I've never worked with that tool before, however, if I understand it right I need to prefix the comment with fix: or do I have to do something different?

@ndabAP
Copy link
Owner

ndabAP commented Dec 10, 2020

Yeah, exactly! Just fix: and please use imperative.

@AmberByte
Copy link
Contributor Author

Done. I've rewritten the commit.

@ndabAP ndabAP merged commit e1913f8 into ndabAP:master Dec 10, 2020
@ndabAP
Copy link
Owner

ndabAP commented Dec 10, 2020

🎉 This PR is included in version 23.0.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants