View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0007368 | MMW v4 | Framework: Scripts/Extensions | public | 2011-02-13 02:00 | 2011-10-18 08:04 |
| Reporter | zvezdan | Assigned To | |||
| Priority | none | Severity | minor | Reproducibility | always |
| Status | closed | Resolution | no change required | ||
| Product Version | 4.0 | ||||
| Target Version | 4.0 | ||||
| Summary | 0007368: Custom node should not be selected and OnFillTracks event should not be fired with right-click on node [MM4] | ||||
| Description | Since you changed the behavior of the right-click on some node in MM4, OnFillTracks event should be modified as well, i.e. it should not be fired in such cases because the built-in nodes also doesn't fill the tracklist on the right-click on some node. However, we still need a possibility to handle the event when an user has right-clicked some node with e.g. OnNodeFocused. Well, technically OnNodeFocused is not the right event for such thing since the old (current) node should not change its focus on the right click of some another node. Maybe you should add some new event, e.g. OnRightClick or OnContextMenu, but because of the compatibility with the existing scripts I suggest that you generate OnChangedSelection event when user right-click on some node. | ||||
| Steps To Reproduce | 1. Install Magic Nodes add-on; 2. select and expand Magic Nodes node; 3. right-click on the first sub-node, i.e. "Album Artist [A-Z], Album (by Year)" -> clicked node is selected and tracklist is filled. 4. right-click on any built-in node -> clicked node is not selected nor tracklist is filled. | ||||
| Tags | No tags attached. | ||||
| Fixed in build | |||||
|
|
Well, I just discovered that nodes are not selected on the right-click only for nodes on the topmost level - all other sub-nodes are selected on the right-click which is even worse and inconsistent. |
|
|
This issue was reverted over the course of MM4 development. |
|
|
Can you please triage this for 4.1? |
|
|
Since this was reverted, I don't think it's an issue anymore, is it? |
|
|
Right, closing. |
