Skip to content

Commit 81e0be7

Browse files
Apply suggestions from code review
Co-authored-by: Elliott Sales de Andrade <quantum.analyst@gmail.com>
1 parent dff2887 commit 81e0be7

2 files changed

Lines changed: 5 additions & 4 deletions

File tree

lib/matplotlib/backends/backend_webagg_core.py

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -350,10 +350,10 @@ def _handle_key(self, event):
350350

351351
def handle_toolbar_button(self, event):
352352
name = event['name']
353-
allowed = {item[3] for item in self.toolbar.toolitems if item[3] is not None}
354-
if name not in allowed:
355-
return
356-
getattr(self.toolbar, name)()
353+
for item in self.toolbar.toolitems:
354+
if item[3] is not None and name == item[3]:
355+
getattr(self.toolbar, name)()
356+
break
357357

358358
def handle_refresh(self, event):
359359
if self.manager:

lib/matplotlib/tests/test_backend_webagg.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
import os
22
import sys
33
from unittest.mock import MagicMock
4+
45
import pytest
56

67
import matplotlib.backends.backend_webagg_core

0 commit comments

Comments
 (0)