googlearchive / paper-menu

A Material Design menu
https://www.webcomponents.org/element/PolymerElements/paper-menu
27 stars 48 forks source link

paper-submenu jumps to first element after scrolling down then selecting #89

Closed brian-cruickshank closed 8 years ago

brian-cruickshank commented 8 years ago

Please see paper-menu/issues/17 for a good description of this problem. I've found this occurring in v1.2.2 of the paper-menu component.

There is a two part fix:

      /**
       * Handler that is called when the menu receives focus.
       *
       * @param {FocusEvent} event A focus event.
       */
      _onFocus: function(event) {
        // this.__trigger && this.__trigger.focus();
      }
ktiedt commented 8 years ago

Hitting this too, can we get some feeback, I am happy to convert this to a PR if the above fixes are acceptable and @brian-cruickshank doesnt have the extra cycles.

It is rather critical for us.

ktiedt commented 8 years ago

Ping? Still critical, even more so actually, because now more users are complaining about it.