Skip to content
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

Disable pan if active_tools=[] #5754

Merged
merged 1 commit into from Jun 12, 2023
Merged

Disable pan if active_tools=[] #5754

merged 1 commit into from Jun 12, 2023

Conversation

Hoxbro
Copy link
Member

@Hoxbro Hoxbro commented Jun 11, 2023

Fixes #5753

If an empty list is added to active_tools, it will actively remove Pan as an active tool.

image

@codecov-commenter
Copy link

codecov-commenter commented Jun 11, 2023

Codecov Report

Merging #5754 (445746b) into main (818e649) will decrease coverage by 0.01%.
The diff coverage is 50.00%.

@@            Coverage Diff             @@
##             main    #5754      +/-   ##
==========================================
- Coverage   88.18%   88.18%   -0.01%     
==========================================
  Files         306      306              
  Lines       62735    62737       +2     
==========================================
+ Hits        55325    55326       +1     
- Misses       7410     7411       +1     
Impacted Files Coverage Δ
holoviews/plotting/bokeh/element.py 88.14% <50.00%> (-0.05%) ⬇️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@Hoxbro Hoxbro merged commit c5cb45f into main Jun 12, 2023
13 checks passed
@Hoxbro Hoxbro deleted the no_active_tools branch June 12, 2023 14:32
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.

Unable to turn off all active tools in the toolbar
2 participants