Maniphest T64267

'Add Snap Point' does not work with Ctrl pressed
Closed, ResolvedDESIGN

Assigned To
Germano Cavalcante (mano-wii)
Authored By
Konstantin (miller)
May 7 2019, 8:15 PM
Tags
  • BF Blender
  • Modeling
  • User Interface
Subscribers
Campbell Barton (campbellbarton)
Ethan Simon (razed)
Germano Cavalcante (mano-wii)
Konstantin (miller)
Philipp Oeser (lichtwerk)
Richard Antalik (ISS)
Sergey (borschberry)
1 More Subscribers
Tokens
"Like" token, awarded by franMarz."Heartbreak" token, awarded by miller.

Description

System Information
Operating system: Windows-10-10.0.17763 64 Bits
Graphics card: Intel(R) HD Graphics 620 Intel 4.5.0 - Build 23.20.16.4973

Blender Version
Broken: version: 2.80 (sub 60), branch: blender2.7, commit date: 2019-05-07 00:16, hash: rBbfef3bce1a93
Worked: NONE

Short description of error
When you toggle snapping during transform you can use A button to add midpoints snaps.
But A button wont work if activate snapping holding Ctrl.

Exact steps for others to reproduce the error

  1. edit mode
  2. switch snap target to vertex/edge/face
  3. toggle snapping ON
  4. select and move one vert
  5. push A to add mid points

6. Sucsess! it working

  1. toggle snapping OFF
  2. select and move one vert
  3. hold CTRL to enable snapping
  4. push A to add mid points -> wont work

Revisions and Commits

rB Blender

Event Timeline

Konstantin (miller) created this task.May 7 2019, 8:15 PM
Sebastian Parborg (zeddb) assigned this task to Campbell Barton (campbellbarton).May 8 2019, 11:58 AM
Sebastian Parborg (zeddb) lowered the priority of this task from 90 to Low.
Konstantin (miller) awarded a token.Oct 4 2019, 8:49 PM
Konstantin (miller) added a comment.EditedOct 4 2019, 8:51 PM

actually if you were fixed that.
no need in new snap to edge center option.
how to snap to face center then?

Dalai Felinto (dfelinto) removed Campbell Barton (campbellbarton) as the assignee of this task.Dec 23 2019, 4:34 PM
Dalai Felinto (dfelinto) added a project: Tracker Curfew.
Dalai Felinto (dfelinto) added a subscriber: Campbell Barton (campbellbarton).
Richard Antalik (ISS) changed the subtype of this task from "Report" to "Bug".Feb 2 2020, 9:36 PM
Richard Antalik (ISS) edited projects, added Modeling, User Interface; removed Tracker Curfew.
Richard Antalik (ISS) added a subscriber: Richard Antalik (ISS).

This doesn't seem to work as intended now at all.

Germano Cavalcante (mano-wii) added subscribers: William Reynish (billreynish), Germano Cavalcante (mano-wii).Feb 6 2020, 5:43 AM

This is not exactly a bug, but in fact it would be a nice addition.

diff --git a/release/scripts/presets/keyconfig/keymap_data/blender_default.py b/release/scripts/presets/keyconfig/keymap_data/blender_default.py
index 8b398d1541c..60ab2939b76 100644
--- a/release/scripts/presets/keyconfig/keymap_data/blender_default.py
+++ b/release/scripts/presets/keyconfig/keymap_data/blender_default.py
@@ -4465,6 +4465,7 @@ def km_transform_modal_map(_params):
         ("SNAP_INV_ON", {"type": 'RIGHT_CTRL', "value": 'PRESS', "any": True}, None),
         ("SNAP_INV_OFF", {"type": 'RIGHT_CTRL', "value": 'RELEASE', "any": True}, None),
         ("ADD_SNAP", {"type": 'A', "value": 'PRESS'}, None),
+        ("ADD_SNAP", {"type": 'A', "value": 'PRESS', "ctrl": True}, None),
         ("REMOVE_SNAP", {"type": 'A', "value": 'PRESS', "alt": True}, None),
         ("PROPORTIONAL_SIZE_UP", {"type": 'PAGE_UP', "value": 'PRESS'}, None),
         ("PROPORTIONAL_SIZE_DOWN", {"type": 'PAGE_DOWN', "value": 'PRESS'}, None),
diff --git a/release/scripts/presets/keyconfig/keymap_data/industry_compatible_data.py b/release/scripts/presets/keyconfig/keymap_data/industry_compatible_data.py
index 2af3bc4898c..c8fe17f8362 100644
--- a/release/scripts/presets/keyconfig/keymap_data/industry_compatible_data.py
+++ b/release/scripts/presets/keyconfig/keymap_data/industry_compatible_data.py
@@ -3537,6 +3537,7 @@ def km_transform_modal_map(_params):
         ("SNAP_INV_ON", {"type": 'RIGHT_CTRL', "value": 'PRESS', "any": True}, None),
         ("SNAP_INV_OFF", {"type": 'RIGHT_CTRL', "value": 'RELEASE', "any": True}, None),
         ("ADD_SNAP", {"type": 'A', "value": 'PRESS'}, None),
+        ("ADD_SNAP", {"type": 'A', "value": 'PRESS', "ctrl": True}, None),
         ("REMOVE_SNAP", {"type": 'A', "value": 'PRESS', "alt": True}, None),
         ("PROPORTIONAL_SIZE_UP", {"type": 'PAGE_UP', "value": 'PRESS'}, None),
         ("PROPORTIONAL_SIZE_DOWN", {"type": 'PAGE_DOWN', "value": 'PRESS'}, None),

@William Reynish (billreynish), any objection to adding this keyitem for the modal keymap used in the transform operator?

Germano Cavalcante (mano-wii) changed the subtype of this task from "Bug" to "Design".Feb 7 2020, 12:21 PM
Fran Marz (franMarz) awarded a token.Feb 7 2020, 1:46 PM
Campbell Barton (campbellbarton) moved this task from Backlog to Under Discussion on the Modeling board.Aug 5 2020, 9:53 AM
Germano Cavalcante (mano-wii) renamed this task from Mid point snaps to 'Add Snap Point' does not work with Ctrl pressed.Sep 1 2020, 1:47 PM
Germano Cavalcante (mano-wii) added subscribers: Ethan Simon (razed), Philipp Oeser (lichtwerk).
Germano Cavalcante (mano-wii) closed this task as Resolved by committing rB9259052e15e7: Fix T64267: 'Add Snap Point' does not work with Ctrl pressed.Sep 1 2020, 2:54 PM
Germano Cavalcante (mano-wii) claimed this task.
Germano Cavalcante (mano-wii) added a commit: rB9259052e15e7: Fix T64267: 'Add Snap Point' does not work with Ctrl pressed.
Germano Cavalcante (mano-wii) added subscribers: Sergey (borschberry), Yevgeny Makarov (jenkm).