Focus remains on button about moveto HOT 8 CLOSED

Hi @shikkaba

You can do it by yourself.

const moveTo = new MoveTo();
const trigger = document.getElementsByClassName('js-trigger')[0];

moveTo.registerTrigger(trigger, function(target){
  trigger.blur();
  if (target instanceof HTMLElement) {
     target.focus();
  }
});

Thank you.

from moveto.

Thank you for writing this, but I'm just saying that this would be a good thing to have in the script as well.

from moveto.

Hi @shikkaba

Thank you for your sugesstion but, I am not sure this is a task that the plugin should do. I dont want to make this plugin complicated. I want to keep it simple and lightweight.

Thank you.

from moveto.

@hsnaydd by "not making it complicated" you make it even more complicated for others people to use. The code above doesn't work for me.

const moveTo = new MoveTo()
    const triggers = document.getElementsByClassName('js-trigger')
    for (var i = 0; i < triggers.length; i++) {
      moveTo.registerTrigger(triggers[i], function (target) {
        triggers[i].blur()
        if (target instanceof HTMLElement) {
          target.focus()
        }
      })
    }

Uncaught TypeError: Cannot read property 'blur' of undefined

Thank you.

from moveto.

Callback doesn't return an element in the loop. It return undefined or 4 (I got 4 elements). It doesn't return actual element!

from moveto.

Hi @iamdubx
Can you show me an example on jsfiddle ?

from moveto.

@hsnaydd sure, here - https://jsfiddle.net/Ltstnk0s/

from moveto.

Hi @iamdubx

Check this;
https://jsfiddle.net/Ltstnk0s/1/

from moveto.