mgsloan / todoist-shortcuts

Browser extension which adds comprehensive keyboard shortcuts to Todoist, beyond what is offered by Todoist itself
MIT License
313 stars 23 forks source link

"shift+arrow key" broken #248

Open basilhalperin opened 1 year ago

basilhalperin commented 1 year ago

Hi!, thanks again for this amazing add-on. It looks like:

With the latest update to how tasks are re-ordered, "shift+left arrow"/"shift+right arrow"/etc used to reorder or indent tasks has broken.

I thought I'd flag the issue!

JohnWilliston commented 1 year ago

Can confirm. The shift + J/K and shift + down/up arrows also fail to move tasks with the latest update.

apontejosea commented 1 year ago

I can confirm this is also the case for me. Thanks for such useful extension.

mmcintyre123 commented 1 year ago

Same here. Looking forward to a fix.

atkinsmedley commented 1 year ago

Me, too. I'm a bit of a novice JS/TS dev, but I will take a look and see if I'm able to figure out the issue.

GaryFurash commented 11 months ago

I sent owner $ for coffee.

mgsloan commented 11 months ago

Thanks @GaryFurash ! I've tried to debug this one but so far haven't found the time chunk to really dig into it. Unfortunately the drag-and-drop simulation code is fairly complex, and the failure isn't obvious. I suspect that Todoist has changed how it responds to the mouse events and it would take some experimentation to fix.

mgsloan commented 11 months ago

In #249 @GaryFurash pointed out that there are some native todoist shortcuts for this. So maybe this can be fixed by delegating to those.

Hope to try that out next time I get a chunk of time

GaryFurash commented 11 months ago

@mgsloan Why can't we just get todoist to incorporate your stuff or add this as in integration. If users don't want to use your cool shortcuts then don't use them. arrg!

rickyribeiroo commented 10 months ago

@mgsloan Hey Michael, have you had the chance to take a look at this issue? Thank you!

mmcintyre123 commented 5 months ago

Any updates on this? Would love to have this back.

Hex-commits commented 2 months ago

I could also reproduce this, sadly found no solution here so far.