Add keyboard shortcut for last annotation

[lehecht]

• Add shortcut to videos
• Add shortcut to images
• Add shortcut to the manual

Closes #499.

[mzur]

I saw that you added a new controller for this. The original idea was to just take the annotation with the highest ID and select it. This does not require a controller etc.

[lehecht]

I saw that you added a new controller for this. The original idea was to just take the annotation with the highest ID and select it. This does not require a controller etc.

Sorry, I thought that the annotation list could not be up-to-date at some point, but I was wrong. I reverted all my changes.

[mzur]

Sorry to be this nitpicky about such a small change again but, well, that's my job 😁

Is there a specific reason why the shortcut is handled by the annotationCanvas/videoScreen? It could also be handled directly in the annotatorContainer/videoContainer. Then there is no need to pass and handle an extra event.

Also I'm ambivalent about Ctrl+l. The l shortcut is already associated with the attach/swap tool. While this action is also about "selecting something", it's not directly related to the tool. Maybe we find an unused key and can add the shortcut without modifier (Ctrl)?

Finally, the new shortcut must be added to the manual.

[mzur]

Please use handleSelectAnnotation() so there are not too many different places that handle selecting of annotations.

Suggested change

	lastAnnotation.selected = true;
	remainingAnnotations.map(a => a.selected = false);
	this.handleSelectAnnotation(lastAnnotation);

[mzur]

Have you tried this and it works? When an annotation is selected (e.g. in the sidebar) the tool should seek the video to the start of the annotation. In the videoContainer annotation.selected is then set to this time. Same as above, it should work fine if you reuse selectAnnotations():

Suggested change

	lastAnnotation.selected = true;
	remainingAnnotations.map(a => a.selected = false);
	this.selectAnnotations([lastAnnotation], [], lastAnnotation.startFrame);

[lehecht]

Thanks! When I tested it, I created multiple annotations in the same frame, which is why I didn't notice that.

[lehecht]

Did you leave the second parameter empty on purpose? Because this causes the last annotation only to be selected but not to seek the video, if another annotation was already selected on the timeline.

[mzur]

No. I just wanted to avoid selectAnnotation() because this can have unexpected side effects with attaching or swapping labels. You should use:

this.selectAnnotations([lastAnnotation], this.selectedAnnotations, lastAnnotation.startFrame);

[lehecht]

@mzur

Is there a specific reason why the shortcut is handled by the annotationCanvas/videoScreen?

Because all keyboard events are handled in these components and transfer the events to the parent components. So I just copied what was already there, but it is no problem to change that.