|Anonymous | Login | Signup for a new account||2018-12-18 09:44 PST|
|My View | View Issues | Change Log | Roadmap|
|View Issue Details|
|ID||Project||Category||View Status||Date Submitted||Last Update|
|0007228||ardour||bugs||public||2017-02-06 11:56||2017-07-19 07:14|
|Priority||normal||Severity||minor||Reproducibility||have not tried|
|Product Version||5.X git (version in description)|
|Target Version||Fixed in Version|
|Summary||0007228: Region trim operations disabled when there's a track selection|
|Description||It used to be possible to use, for example, 'J' and 'K' to trim all regions on selected tracks at the edit point, but these now appear to be disabled when the region selection is empty.|
|Tags||No tags attached.|
|should be fixed in git now.|
Hmm, still doesn't work for me: all region trim menu items are still disabled when there's a track selection.
Also, I reckon that at least 'Trim to Loop' and 'Trim to Punch' should also be enabled in this case.
|ah right, i thought that just not calling "reg_sens()" on them was enough. it isn't. there needs to be a new group .... coming up.|
this is much harder than it looked. here are some notes i made while hacking on an experimental branch ...
problem is that when calling
sensitize_the_right_region_actions(), we call
get_regions_from_selection_and_edit_point() but with arguments
that may differ from those used to make the same call when an
action is actually carried out.
this means that get_regions_from_selection_and_edit_point() may return
different results in each context, which means that an action which
was shown as relevant (sensitive) in a menu may in fact do nothing.
secondarily, many operations use
get_regions_from_selection_and_edit_point() which means that the set
of regions potentially affected by an action depends on the action
itself and cannot be generalized.
Things should be much better as of 606c8d4
This took quite a hard engineering effort. I suspect there may still be ways to improve the overall code internally, but I think this now functions as expected from a user perspective.
|Yes, working for ages now: cheers!|
|2017-02-06 11:56||colinf||New Issue|
|2017-02-07 05:06||colinf||Description Updated||View Revisions|
|2017-02-07 08:24||paul||Note Added: 0019338|
|2017-02-07 08:24||paul||Assigned To||=> paul|
|2017-02-07 08:24||paul||Status||new => feedback|
|2017-02-07 08:29||colinf||Note Added: 0019340|
|2017-02-07 08:29||colinf||Status||feedback => assigned|
|2017-02-07 08:34||paul||Note Added: 0019342|
|2017-02-10 13:13||paul||Note Added: 0019361|
|2017-02-13 03:51||colinf||Relationship added||related to 0007077|
|2017-02-16 01:31||paul||Note Added: 0019402|
|2017-02-16 01:31||paul||Status||assigned => feedback|
|2017-07-19 07:14||colinf||Note Added: 0019902|
|2017-07-19 07:14||colinf||Status||feedback => assigned|
|2017-07-19 07:14||colinf||Status||assigned => resolved|
|2017-07-19 07:14||colinf||Resolution||open => fixed|
|Copyright © 2000 - 2018 MantisBT Team|