Skip to content

Fix duplicate menus in GradientEditorItem#444

Merged
j9ac9k merged 3 commits intopyqtgraph:developfrom
zachlowry:duplicate-menus
Jun 1, 2020
Merged

Fix duplicate menus in GradientEditorItem#444
j9ac9k merged 3 commits intopyqtgraph:developfrom
zachlowry:duplicate-menus

Conversation

@zachlowry
Copy link
Copy Markdown
Contributor

Add call to ev.accept in Tivk.mouseClickEvent to prevent parent menu from opening on a right click of a Tick.

Add call to ev.accept in Tivk.mouseClickEvent to prevent parent menu from opening on a right click of a Tick.
@codecov-io
Copy link
Copy Markdown

Codecov Report

Merging #444 into develop will increase coverage by 1.3%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           develop     #444     +/-   ##
==========================================
+ Coverage    34.36%   35.66%   +1.3%     
==========================================
  Files          198      198             
  Lines        27334    27335      +1     
  Branches      4555     4555             
==========================================
+ Hits          9393     9749    +356     
+ Misses       17083    16712    -371     
- Partials       858      874     +16
Impacted Files Coverage Δ
pyqtgraph/graphicsItems/GradientEditorItem.py 56.79% <ø> (-0.12%)
pyqtgraph/graphicsItems/ViewBox/ViewBox.py 56.92% <ø> (-0.21%)
pyqtgraph/multiprocess/remoteproxy.py 57.96% <ø> (+0.55%)
pyqtgraph/functions.py 39.83% <ø> (+1.01%)
pyqtgraph/opengl/items/GLVolumeItem.py 10.43% <ø> (+10.43%)
pyqtgraph/opengl/MeshData.py 11.34% <ø> (+11.34%)
pyqtgraph/opengl/items/GLMeshItem.py 12.78% <ø> (+12.78%)
pyqtgraph/opengl/items/GLSurfacePlotItem.py 12.98% <ø> (+12.98%)
pyqtgraph/opengl/items/GLScatterPlotItem.py 14.81% <ø> (+14.81%)
pyqtgraph/opengl/GLViewWidget.py 19.8% <ø> (+19.8%)
... and 9 more

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 dd5a8bf...6221f41. Read the comment docs.

Copy link
Copy Markdown
Contributor

@2xB 2xB left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

With this change, both lines 868 and 877 accept the event. How about switching lines 867 and 868 (the if statement and ev.accept()) and removing line 877 (the new ev.accept())?

@j9ac9k
Copy link
Copy Markdown
Member

j9ac9k commented May 31, 2020

blerg, you would think i would have figured out how to do this without messing up people's branches... sorry about that @zachlowry I'll try and get this cleaned up

@j9ac9k
Copy link
Copy Markdown
Member

j9ac9k commented Jun 1, 2020

Thanks for the PR @zachlowry sorry it's taken us so long to get this merged.

@j9ac9k j9ac9k merged commit c903546 into pyqtgraph:develop Jun 1, 2020
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