Follow up to MESH_OT_offset_edge_loops

Committing this as a separate commit since we may want to keep this mode syncing
elsewhere. But for now it makes sense to do it like this.
This commit is contained in:
Dalai Felinto 2018-08-31 17:23:33 -03:00
parent efd843b51c
commit f6b902a9ca
1 changed files with 18 additions and 8 deletions

View File

@ -6466,9 +6466,17 @@ void MESH_OT_wireframe(wmOperatorType *ot)
static int edbm_offset_edgeloop_exec(bContext *C, wmOperator *op)
{
bool mode_change = false;
const bool use_cap_endpoint = RNA_boolean_get(op->ptr, "use_cap_endpoint");
int ret = OPERATOR_CANCELLED;
Object *obedit = CTX_data_edit_object(C);
BMEditMesh *em_edit = BKE_editmesh_from_object(obedit);
if (em_edit->selectmode == SCE_SELECT_FACE) {
EDBM_selectmode_to_scene(C);
mode_change = true;
}
ViewLayer *view_layer = CTX_data_view_layer(C);
uint objects_len = 0;
Object **objects = BKE_view_layer_array_from_objects_in_edit_mode_unique_data(view_layer, &objects_len);
@ -6476,6 +6484,16 @@ static int edbm_offset_edgeloop_exec(bContext *C, wmOperator *op)
Object *obedit = objects[ob_index];
BMEditMesh *em = BKE_editmesh_from_object(obedit);
/** If in face-only select mode, switch to edge select mode so that
* an edge-only selection is not inconsistent state.
*
* We need to run this for all objects, even when nothing is selected.
* This way we keep them in sync. */
if (mode_change) {
em->selectmode = SCE_SELECT_EDGE;
EDBM_selectmode_set(em);
}
if (em->bm->totedgesel == 0) {
continue;
}
@ -6490,14 +6508,6 @@ static int edbm_offset_edgeloop_exec(bContext *C, wmOperator *op)
BM_mesh_elem_hflag_disable_all(em->bm, BM_VERT | BM_EDGE | BM_FACE, BM_ELEM_SELECT, false);
/* If in face-only select mode, switch to edge select mode so that
* an edge-only selection is not inconsistent state */
if (em->selectmode == SCE_SELECT_FACE) {
em->selectmode = SCE_SELECT_EDGE;
EDBM_selectmode_set(em);
EDBM_selectmode_to_scene(C);
}
BMO_slot_buffer_hflag_enable(em->bm, bmop.slots_out, "edges.out", BM_EDGE, BM_ELEM_SELECT, true);
if (!EDBM_op_finish(em, &bmop, op, true)) {