Rubber-band selection doesn't include comment markers #33

Open
opened 2026-04-21 14:22:08 +00:00 by AhmedHanafy725 · 1 comment
Member

Dragging a rubber-band selection over an area that contains comment markers selects every other object but skips the comments, so a group drag leaves comments behind.

Root cause: onMouseUp in tools.js iterates only layer.find('.object'), but comment markers have name: 'comment-marker' (they are not whiteboard objects).

Fix: after collecting .object intersections, also walk layer.find('.comment-marker') nodes and add the ones that intersect the rubber-band box to the transformer's selection.

Dragging a rubber-band selection over an area that contains comment markers selects every other object but skips the comments, so a group drag leaves comments behind. Root cause: `onMouseUp` in `tools.js` iterates only `layer.find('.object')`, but comment markers have `name: 'comment-marker'` (they are not whiteboard objects). Fix: after collecting `.object` intersections, also walk `layer.find('.comment-marker')` nodes and add the ones that intersect the rubber-band box to the transformer's selection.
Author
Member

Pull request opened: #35

Pull request opened: https://forge.ourworld.tf/lhumina_code/hero_whiteboard/pulls/35
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
lhumina_code/hero_whiteboard#33
No description provided.