Skip to content

ENH: Allow Brush to draw without holding down the mouse button#2980

Merged
peircej merged 1 commit intopsychopy:masterfrom
m-macaskill:brushbutton
Jul 26, 2020
Merged

ENH: Allow Brush to draw without holding down the mouse button#2980
peircej merged 1 commit intopsychopy:masterfrom
m-macaskill:brushbutton

Conversation

@m-macaskill
Copy link
Contributor

Currently to draw with the Brush stimulus requires holding down the mouse button. Allowing drawing without a button press can be useful, and may be easier with trackpads, touch interfaces, and so on.
Also exposed this functionality via the Builder component UI.
NB this has been implemented only for PsychoPy. Equivalent changes would need to be made for PsychoJS. Any volunteers?

Made in response to this forum topic:
https://discourse.psychopy.org/t/creating-a-mouse-trail-for-trail-tracing/15138

Also exposed this functionality via the Builder component UI.
*NB* this has been implemented only for PsychoPy. Equivalent changes would need to be made for PsychoJS.

Made in response to this forum topic:
https://discourse.psychopy.org/t/creating-a-mouse-trail-for-trail-tracing/15138
@codecov-commenter
Copy link

codecov-commenter commented Jul 24, 2020

Codecov Report

Merging #2980 into master will decrease coverage by 0.01%.
The diff coverage is 50.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2980      +/-   ##
==========================================
- Coverage   43.09%   43.07%   -0.02%     
==========================================
  Files         256      256              
  Lines       48307    48327      +20     
  Branches     8196     8201       +5     
==========================================
- Hits        20818    20817       -1     
- Misses      25342    25361      +19     
- Partials     2147     2149       +2     
Impacted Files Coverage Δ
psychopy/experiment/components/__init__.py 51.72% <0.00%> (-0.91%) ⬇️
psychopy/visual/brush.py 68.05% <50.00%> (-2.10%) ⬇️
psychopy/experiment/components/brush/__init__.py 100.00% <100.00%> (ø)
psychopy/web.py 36.53% <0.00%> (-1.29%) ⬇️
psychopy/app/coder/coder.py 30.76% <0.00%> (-0.34%) ⬇️
psychopy/app/builder/builder.py 47.31% <0.00%> (-0.05%) ⬇️
psychopy/app/utils.py 67.18% <0.00%> (ø)
psychopy/app/runner/runner.py 60.77% <0.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6ca49ea...2123621. Read the comment docs.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.003%) to 47.18% when pulling 2123621 on m-macaskill:brushbutton into 699cdd6 on psychopy:master.

@peircej peircej merged commit 50cb7ee into psychopy:master Jul 26, 2020
@m-macaskill m-macaskill deleted the brushbutton branch July 26, 2020 21:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants