Box Select (Axis Range) taking selection box proportions into account #94104

Closed
opened 2021-12-15 10:09:35 +01:00 by Garek · 6 comments

Blender Version
Broken: version: 3.0.0, branch: master (modified), commit date: 2021-12-02 18:35, hash: f1cca30557
Broken: version: 2.50-alpha0 (first introduced)
Worked: never

Short description of error
Box Select (Axis Range) (as it seems) intended to work like this: if an element (NLA strip, keyframe etc.) exists on the same axis range (horizontal or vertical) as selection box, it's selected. But since it's introduction in 2.50 it's been broken and worked like this:

  • If an element on the same horizontal axis range as selection box and vertical length of a selection box is smaller than it's horizontal length then it's selected.
  • If an element on the same vertical axis range as selection box and horizontal length of a selection box is smaller than it's vertical length then it's selected.
    See attached video for clarification

Exact steps for others to reproduce the error

  1. Open attached .blend file.
  2. In NLA editor make sure you use Box Select (Axis Range).
  3. Draw wide box under NLA strip - it is selected, which is expected.
  4. Click on emptry spase to deselect.
  5. Draw narrow box under NLA strip - it is not selected.
  6. Draw narrow box to the right or left side of NLA strip - it is selected, which is expected.
  7. Draw wide box to the right or left side of NLA strip - it is not selected

Box Selection Axis Range bug 2021-12-15 13-57-50-890.mp4
Box Selection Axis Range bug.blend

**Blender Version** Broken: version: 3.0.0, branch: master (modified), commit date: 2021-12-02 18:35, hash: `f1cca30557` Broken: version: 2.50-alpha0 (first introduced) Worked: never **Short description of error** `Box Select (Axis Range)` (as it seems) intended to work like this: if an element (NLA strip, keyframe etc.) exists on the same axis range (horizontal or vertical) as selection box, it's selected. But since it's introduction in 2.50 it's been broken and worked like this: - If an element on the same horizontal axis range as selection box **and vertical length of a selection box is smaller than it's horizontal length** then it's selected. - If an element on the same vertical axis range as selection box **and horizontal length of a selection box is smaller than it's vertical length** then it's selected. See attached video for clarification **Exact steps for others to reproduce the error** 1. Open attached .blend file. 2. In NLA editor make sure you use `Box Select (Axis Range)`. 3. Draw wide box under NLA strip - it is selected, which is expected. 4. Click on emptry spase to deselect. 5. Draw narrow box under NLA strip - it is not selected. 6. Draw narrow box to the right or left side of NLA strip - it is selected, which is expected. 7. Draw wide box to the right or left side of NLA strip - it is not selected [Box Selection Axis Range bug 2021-12-15 13-57-50-890.mp4](https://archive.blender.org/developer/F12745127/Box_Selection_Axis_Range_bug_2021-12-15_13-57-50-890.mp4) [Box Selection Axis Range bug.blend](https://archive.blender.org/developer/F12745128/Box_Selection_Axis_Range_bug.blend)
Author

Added subscriber: @Garek

Added subscriber: @Garek
Member

Added subscriber: @BClark

Added subscriber: @BClark
Author

So I investigated it a bit... It was added in 34baf51dcb. And it seems that it is not a bug. On commit message::

ALT-BKEY will select either all the keyframes in the frame-range specified or the channel range specified, depending on which axis of the select region was larger. This method is prejudiced towards frame-range selection.

In code comments:

mode depends on which axis of the range is larger to determine which axis to use

So reading this there is really some logic to it. It depends if you want to select all keyframes of certain channels or all keyframes in certain frame range. Main problem is that it's unclear to user a purpose of this tool and how it is working.

So I investigated it a bit... It was added in 34baf51dcb. And it seems that it is not a bug. On commit message:: > ALT-BKEY will select either all the keyframes in the frame-range specified or the channel range specified, depending on which axis of the select region was larger. This method is prejudiced towards frame-range selection. In code comments: > mode depends on which axis of the range is larger to determine which axis to use So reading this there is really some logic to it. It depends if you want to select all keyframes of certain channels or all keyframes in certain frame range. Main problem is that it's unclear to user a purpose of this tool and how it is working.

Added subscriber: @mano-wii

Added subscriber: @mano-wii

Changed status from 'Needs Triage' to: 'Archived'

Changed status from 'Needs Triage' to: 'Archived'

Seeing the code, this is intentional.
See https://developer.blender.org/diffusion/B/browse/master/source/blender/editors/space_nla/nla_select.c$382-387
So can't be considered a bug.

Thanks for the report anyway.

For user requests and feedback, please use other channels: https://wiki.blender.org/wiki/Communication/Contact#User_Feedback_and_Requests

For more information on why this isn't considered a bug, visit: https://wiki.blender.org/wiki/Reference/Not_a_bug

Seeing the code, this is intentional. See https://developer.blender.org/diffusion/B/browse/master/source/blender/editors/space_nla/nla_select.c$382-387 So can't be considered a bug. Thanks for the report anyway. For user requests and feedback, please use other channels: https://wiki.blender.org/wiki/Communication/Contact#User_Feedback_and_Requests For more information on why this isn't considered a bug, visit: https://wiki.blender.org/wiki/Reference/Not_a_bug
Sign in to join this conversation.
No Label
Interest
Alembic
Interest
Animation & Rigging
Interest
Asset Browser
Interest
Asset Browser Project Overview
Interest
Audio
Interest
Automated Testing
Interest
Blender Asset Bundle
Interest
BlendFile
Interest
Collada
Interest
Compatibility
Interest
Compositing
Interest
Core
Interest
Cycles
Interest
Dependency Graph
Interest
Development Management
Interest
EEVEE
Interest
EEVEE & Viewport
Interest
Freestyle
Interest
Geometry Nodes
Interest
Grease Pencil
Interest
ID Management
Interest
Images & Movies
Interest
Import Export
Interest
Line Art
Interest
Masking
Interest
Metal
Interest
Modeling
Interest
Modifiers
Interest
Motion Tracking
Interest
Nodes & Physics
Interest
OpenGL
Interest
Overlay
Interest
Overrides
Interest
Performance
Interest
Physics
Interest
Pipeline, Assets & IO
Interest
Platforms, Builds & Tests
Interest
Python API
Interest
Render & Cycles
Interest
Render Pipeline
Interest
Sculpt, Paint & Texture
Interest
Text Editor
Interest
Translations
Interest
Triaging
Interest
Undo
Interest
USD
Interest
User Interface
Interest
UV Editing
Interest
VFX & Video
Interest
Video Sequencer
Interest
Virtual Reality
Interest
Vulkan
Interest
Wayland
Interest
Workbench
Interest: X11
Legacy
Blender 2.8 Project
Legacy
Milestone 1: Basic, Local Asset Browser
Legacy
OpenGL Error
Meta
Good First Issue
Meta
Papercut
Meta
Retrospective
Meta
Security
Module
Animation & Rigging
Module
Core
Module
Development Management
Module
EEVEE & Viewport
Module
Grease Pencil
Module
Modeling
Module
Nodes & Physics
Module
Pipeline, Assets & IO
Module
Platforms, Builds & Tests
Module
Python API
Module
Render & Cycles
Module
Sculpt, Paint & Texture
Module
Triaging
Module
User Interface
Module
VFX & Video
Platform
FreeBSD
Platform
Linux
Platform
macOS
Platform
Windows
Priority
High
Priority
Low
Priority
Normal
Priority
Unbreak Now!
Status
Archived
Status
Confirmed
Status
Duplicate
Status
Needs Info from Developers
Status
Needs Information from User
Status
Needs Triage
Status
Resolved
Type
Bug
Type
Design
Type
Known Issue
Type
Patch
Type
Report
Type
To Do
No Milestone
No project
No Assignees
3 Participants
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: blender/blender#94104
No description provided.