Can't select items with arrow up/down keys when suggestion list has scrolled content about vue-simple-suggest

[jcarrera]

I agree but I ended up implementing something myself. Here's some code that others can use to get something working:

{
    onHover( suggestion, element ){
        // If the hover is via mouse, an element will be present. We are only interested in hovers with keyboard
        // so that we can scroll properly.
        if( !element ){
          // Wait for next tick so that the DOM is updated and the hover class is on the latest element instead
          // of the previous one.
          this.$nextTick(() => {
            const overlayElement = this.getOverlayElement();
            const hoveredElement = this.getHoveredElement();
            this.scrollToElement( hoveredElement, overlayElement );
          });
        }

        this.$emit( 'hover', suggestion, element );
      },
      scrollToElement( element, parentElement ){
        if( parentElement && parentElement.contains( element )){
          const { clientHeight, scrollTop } = parentElement;
          const { offsetHeight, offsetTop } = element;
          
         if( offsetTop < scrollTop ){
            parentElement.scrollTop = offsetTop; // scroll up
          } else {
            const offsetBottom = offsetTop + offsetHeight;
            const scrollBottom = scrollTop + clientHeight;
            if( offsetBottom > scrollBottom ){
              parentElement.scrollTop = offsetBottom - clientHeight; // scroll down
            }
          }
        }
      }
}

@panay

[Raiondesu]

Hello, @jcarrera.
Unfortunately, the feature you're talking about does not rest within vue-simple-suggest's responsibilities and can be classified as custom behaviour because there are many varieties of possible behaviours for suggestions scrolling.

Even though the behaviour you request is pretty much standard, we think you could still easily implement this yourself using the controls prop or listening to the hover event.

If I misunderstood your request - please let me know.

[panay]

Hi, @Raiondesu
I think that it should be included in this plugin? If you change the default browser behavior, then you should make it is work.