-
-
Notifications
You must be signed in to change notification settings - Fork 353
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
22843-Cannot-select-all-in-windows #2157
22843-Cannot-select-all-in-windows #2157
Conversation
…his OS returns keyCharacters as uppercase
ifFalse: [ self keyCharacter ] | ||
self flag: #hack. | ||
"Hack me. When Ctrl is pressed, the key ascii value is not right and we have to do something ugly" | ||
^(self hasSpecialCTRLKeyValue and: [ (#(MacOSX Windows) includes: Smalltalk os current platformFamily) ]) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Smalltalk os platformFamily
is enough here. Why the "current" ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
meh, I see it was like that before.
still no need :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@estebanlm I don't know if you know, but now you can suggest one line changes in reviews to make it easier to contribute.
You can use this icon.
Except the little cleanup requested, the PR works for me on Windows 7. |
Co-Authored-By: pavel-krivanek <pavel.krivanek@gmail.com>
Test failure not related. |
do an image patch for Windows VM inconsistency with other platforms. This OS returns keyCharacters as uppercase