Improve stroke line control#137
Conversation
pehala
left a comment
There was a problem hiding this comment.
LGTM! Nice and short patch.
|
@PhilippvK Can this be merged? |
While the code looks fine, I still would like to test your changes locally. I hope I will find time for this either today or tomorrow. Sorry for the delay. |
|
No problem, take your time. I have more coming once this is merged. |
|
Good point. I edited the cursor to have black cross inside a white cross, so now it should be visible anywhere. It does not look so original anymore, but at least is visible, since I don't know how to create dynamic contrast to it. And my default cursor looked rather different to yours (if I had same as your I would have not created a custom one), I guess it depends on the cursors installed on the system. Using custom cursor at least makes it similar everywhere. |
|
I tested the patch and it seems to work (and look) fine! @nuhakala would you mind adding a paragraph to the README which introduces the relevant keyboard/mouse controls? |
Edits the logic for drawing dashed line on mouse button press instead of keyboard button press to enable original behavior.
Adds custom cursor which is closer to original cursor. Renames method162 to setStrokeListeners.
|
Added documentation and updated the PR description to fix the related issue. While I checked if I had missed some controls, I noticed that I had forgot to edit the hacked shot behavior. I removed also the keyboard controls from there, so that the shooting mode is controlled only through mouse. And when checking that, I finally figured out how to use the hacked shot and tested it myself, was kinda fun :D Anyway, it was drawing only white rectangle to the target location, so I fixed that and now it draws the player ball there. |
|
@nuhakala Thanks! There is a small listing-related issue in the CI. Fixing it should only take a minute (or less using https://github.com/PhilippvK/playforia-minigolf/actions/runs/20236961663/job/58326018187#step:6:1 |
|
Applied now the spotless. |


Hello, this PR
Fixes #122