# Ambiguous control point in right-click context menu

**URL:** <https://discourse.slicer.org/t/ambiguous-control-point-in-right-click-context-menu/20274>\
**Category:** Support\
**Tags:** markups\
**Created:** [October 20, 2021, 7:26pm UTC](https://discourse.slicer.org/t/ambiguous-control-point-in-right-click-context-menu/20274 "2021-10-20T19:26:31Z")\
**Posts on this page:** 6\
**Page:** 1

<div class="post-metadata">

**Author:** ![jamesobutler](https://sea2.discourse-cdn.com/flex002/user_avatar/discourse.slicer.org/jamesobutler/32/7511_2.png) [@jamesobutler](https://discourse.slicer.org/u/jamesobutler)\
**Post date:** [October 20, 2021, 7:26pm UTC](https://discourse.slicer.org/t/ambiguous-control-point-in-right-click-context-menu/20274/1 "2021-10-20T19:26:31Z")

</div>

When hovering the mouse over a Markups Line, Angle, Curve or Closed Curve node where you are highlighting and not hovering over a control point, then right-clicking to bring up the context menu has confusing options to manipulate a control point.

There are options to rename, toggle, jump, and delete control point however it is ambiguous about which control point it will be acting upon since none is highlighted. This does not happen for a Markups Plane or ROI node which just provides the ability to delete the markup node.

It also seems odd that there would be an ability to rename a control point when the individual control point label is not even being shown. Only the label for the markup name is being shown.

 ![image](https://us1.discourse-cdn.com/flex002/uploads/slicer/original/3X/3/7/371f7dcdfe63005665b2f7dba0e8474bb406e254.png)  
 ![image](https://us1.discourse-cdn.com/flex002/uploads/slicer/original/3X/c/d/cdb03cb11f2dbf8d920ef66a53c505463ba570fc.png)

---

<div class="post-metadata">

**Author:** ![jamesobutler](https://sea2.discourse-cdn.com/flex002/user_avatar/discourse.slicer.org/jamesobutler/32/7511_2.png) [@jamesobutler](https://discourse.slicer.org/u/jamesobutler)\
**Post date:** [October 21, 2021, 4:32pm UTC](https://discourse.slicer.org/t/ambiguous-control-point-in-right-click-context-menu/20274/2 "2021-10-21T16:32:24Z")

</div>

The control point menu actions when hovering over the object and not at a control point, appears to manipulate the closest control point even though it isn’t highlighted. If I hover over the line between the first and second control point, but I’m 25% along the line so closer to the first control point then renaming will rename the first control point which is “OC-1”.

@lassoan It would be clearer for control point modification to only be when the control point is highlighted and not just along the line. Can it be change to be like the plane and ROI node and when hovering over the object such as along the line it instead only show context options for the markup node rather than the control point?

---

<div class="post-metadata">

**Author:** ![lassoan](https://sea2.discourse-cdn.com/flex002/user_avatar/discourse.slicer.org/lassoan/32/13_2.png) [@lassoan](https://discourse.slicer.org/u/lassoan)\
**Post date:** [October 21, 2021, 4:41pm UTC](https://discourse.slicer.org/t/ambiguous-control-point-in-right-click-context-menu/20274/3 "2021-10-21T16:41:23Z")

</div>

I agree, it could make sense to only show control point actions (except jump to closest control point) if the active component is a control point. It would be great if you could submit a pull request with the change.

---

<div class="post-metadata">

**Author:** ![muratmaga](https://sea2.discourse-cdn.com/flex002/user_avatar/discourse.slicer.org/muratmaga/32/3622_2.png) [@muratmaga](https://discourse.slicer.org/u/muratmaga)\
**Post date:** [October 21, 2021, 4:43pm UTC](https://discourse.slicer.org/t/ambiguous-control-point-in-right-click-context-menu/20274/4 "2021-10-21T16:43:40Z")

</div>

One action that would make sense is to delete the whole curve (or change its colors/edit properties). Currently it is possible but it is indicated delete markup. Better to rename it to “delete curve”

---

<div class="post-metadata">

**Author:** ![jamesobutler](https://sea2.discourse-cdn.com/flex002/user_avatar/discourse.slicer.org/jamesobutler/32/7511_2.png) [@jamesobutler](https://discourse.slicer.org/u/jamesobutler)\
**Post date:** [October 21, 2021, 4:46pm UTC](https://discourse.slicer.org/t/ambiguous-control-point-in-right-click-context-menu/20274/5 "2021-10-21T16:46:32Z")

</div>

So rename:

- Delete markup
- Edit markup terminology…
- Edit properties…

to something with markup type name like the following?

- Delete curve
- Edit curve terminology…
- Edit curve properties…

---

<div class="post-metadata">

**Author:** ![jamesobutler](https://sea2.discourse-cdn.com/flex002/user_avatar/discourse.slicer.org/jamesobutler/32/7511_2.png) [@jamesobutler](https://discourse.slicer.org/u/jamesobutler)\
**Post date:** [October 23, 2021, 9:12pm UTC](https://discourse.slicer.org/t/ambiguous-control-point-in-right-click-context-menu/20274/6 "2021-10-23T21:12:46Z")

</div>

PR posted for this work:

> <https://github.com/Slicer/Slicer/pull/5961>
>
> Originally discussed over at https://discourse.slicer.org/t/ambiguous-control-po…int-in-right-click-context-menu/20274
> 
> | | Before | This PR |
> |-|---------|---------|
> |Right-click point|!\[image\](https://user-images.githubusercontent.com/15837524/138571657-98c0ed20-90ad-4aa8-a478-11dfe2444c64.png)|!\[image\](https://user-images.githubusercontent.com/15837524/138571663-04f5ab62-6b76-4080-a4b8-ee80051ba1a4.png)|
> |Right-click node|!\[image\](https://user-images.githubusercontent.com/15837524/138571682-47a484de-454b-44a1-b8db-8a2abaf2eb3c.png)|!\[image\](https://user-images.githubusercontent.com/15837524/138571688-ed341910-1cdc-4dbc-8196-12a796ab2d63.png)|