Release Toggle Track Pitch Envelope v1.0 - #1611
Conversation
|
|
||
| chunk = chunk | ||
| :gsub("ACT %d+", "ACT " .. (show and 1 or 0)) | ||
| :gsub("VIS %d+ %d+ %d+", "VIS " .. (show and "1 1 1" or "0 0 0")) |
There was a problem hiding this comment.
Feedback from the previous PR still apply: https://github.com/ReaTeam/ReaScripts/pull/1571/files#r2083305666
(This is not a packaging problem though so won't block a release here. Just a heads up for likely bugs.)
There was a problem hiding this comment.
Disabling that setting is the only way I got it to work. might be another way but it has worked fine for me for the last month or 2 ive been using it hours daily
Fixed stray -+
…eaPitch and which envelope for searchability
|
|
||
| local track = reaper.GetSelectedTrack(0, 0) | ||
| if not track then | ||
| reaper.PreventUIRefresh(0) |
There was a problem hiding this comment.
By the way, while this line here (and same a bit below) is correctly undoing the PreventUIRefresh from above, it forgets to the the same for Undo_BeginBlock.
An easy solution would be to wait until later before enabling PreventUIRefresh and Undo_BeginBlock. 😉
Thankfully REAPER has a fail-safe against scripts doing this kind of mistake so there are no ill effects.
There was a problem hiding this comment.
Schwa suggests here https://forum.cockos.com/showpost.php?p=2297817&postcount=2
You should always call PreventUIRefresh(1) to initiate and PreventUIRefresh(-1) to end
Is 0 a valid argument as well?
There was a problem hiding this comment.
My bad, nope it is not. (Well, not "invalid" per se, but adding 0 just does nothing.)
3 participants
No description provided.