yes to be precise it a bad way to use spans as buttons user should reach the button via keyboard
<span tabindex="0">Button</span>
Also a good practice is to adding ARIA - attributes:
<span tabindex="0" role="button" aria-pressed="false">Button</span>