paper-icon-button icon indicating copy to clipboard operation
paper-icon-button copied to clipboard

[1.0] There is no ripple effect when button is pressed by "Enter" key

Open dzhioev opened this issue 10 years ago • 6 comments

In contrast to <paper-button>, <paper-icon-button> doesn't have the ripple effect when pressed by "Enter" key.

dzhioev avatar Jun 06 '15 00:06 dzhioev

Hmmm... for me there actually seems to be a ripple.

Can you post a reduced testcase? What browser was you using? There are some browser extensions known to be screwing up handlers, so can you try in an incognito window?

arthurvr avatar Jun 07 '15 19:06 arthurvr

@arthurvr Are you sure, that you used the Enter keyboard key when pressed a button? Steps to reproduce:

  1. Go to https://elements.polymer-project.org/bower_components/paper-icon-button/demo/index.html
  2. Focus any button using Tab key.
  3. Press Enter or Space key. My browser is Google Chrome 43.0.2357.81 (Official Build) (64-bit).

dzhioev avatar Jun 10 '15 00:06 dzhioev

  1. Focus any button using Tab key.

Indeed. Then I'm able to reproduce this. Try to be a little more complete next time when making a bug report :) If you click once and then do it again by pressing enter there actually is a ripple, strange.

arthurvr avatar Jun 10 '15 05:06 arthurvr

@notwaldorf Could you please look into this?

dzhioev avatar Sep 26 '15 01:09 dzhioev

This is also a material design bug - there is no differentiation in the "icon-toggles" section of the spec between focused and pressed state.

For non-toggle paper-icon-buttons, I think the solution here should be to replay the ripple animation when keyboard focused and then activated with <Space> or <Enter>, but keep the inky ring visible so that it remains focused.

tjsavage avatar Sep 26 '15 01:09 tjsavage

Just to provide some additional info:

paper-icon-button-light has an inverted problem - it does replay the ripple animation when activated, but removes the inky-focus after the animation has finished.

robrez avatar Aug 08 '16 18:08 robrez