Audacity Bug Summary
••• Introduction •••
••• Keywords •••
    Audacity 3.0.3 development began 19th April 2021

Audacity Bugzilla



Bug 2093 - Note Track: Vertical Zoom Reset produces max. zoom out and not a reset
Note Track: Vertical Zoom Reset produces max. zoom out and not a reset
Status: RESOLVED FIXED
Product: Audacity
Classification: Unclassified
Component: User Interface
2.3.2
Per OS All
: P4 RepeatableAll
Assigned To: Default Assignee for New Bugs
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2019-04-14 05:34 UTC by Peter Sampson
Modified: 2019-04-16 06:12 UTC (History)
6 users (show)

See Also:
Steps To Reproduce:
1) import a MIDI file into a Note track 2) right click in the Vertical Scale (piano keys for Note Track) 3) Observe that it zooms out in one step to show the entire range of notes (from C -1 to G 9).
Release Note:
First Git SHA:
Group: ---
Workaround:
Closed: 2019-04-16 00:00:00
james.k.crook: Must‑Test‑All‑OS-
petersampsonaudacity: Test‑OK‑Win+


Attachments
Note Track after a Vertical Zoom Reset (55.64 KB, image/png)
2019-04-14 05:37 UTC, Peter Sampson
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Peter Sampson 2019-04-14 05:34:44 UTC
Using the Reset in the right-click context menu of the Vertical Scale in a Note Track produces the same result as using Shift+Right-click in the Advanced Vertical Zooming mode - it produces a maximum zoom out to show the entire range of notes (from C -1 to G 9).

This a max zoom and not a Reset.  Reset should restore the same zoom level as that which obtained when the user first imported the MIDI track (this is what happens with audio tracks in waveform and spectrogram modes.

Correct behavior at Step 3) in the Steps to reproduce should be absolutely no change of display - i.e. no zooming at all.


And after any subsequent VZooms in/out a Reset shoul restore to the VZoom level at step1).



The Manual does actually descibe accurately the behavior that currently obtains with the VZoom Reset - but this is incorrect behavior - inconssistebt with the similar Reset for audio tracks.

https://alphamanual.audacityteam.org/man/Vertical_Zooming#note


This is not a regression - it is the same on earlier Audacity versions.
Comment 1 Peter Sampson 2019-04-14 05:37:27 UTC
Created attachment 819 [details]
Note Track after a Vertical Zoom Reset

Note Track after a Vertical Zoom Reset - note that this is not a Reset to "normal" zoom level.
Comment 2 Poke 2019-04-14 15:03:22 UTC
I think that I also included a change relevant to this in https://github.com/audacity/audacity/pull/232 -- but it was a while back.  My implementation did change the default zoom (to fix bug 1815).  It looks like (https://github.com/audacity/audacity/pull/232/commits/d6276f5279c2197db5979277a19a02627c594956) I also added a separate "max zoom" menu item that behaves the same as the current behavior, while zoom reset does do the default zoom (which was changed to show the top and bottom notes).  I also made it so that shift+right-click returned to the default zoom, unless it already was the default in which case it did the max zoom.

I'll try to create an updated version of that pull request, since it's been a while and I don't quite know how everything behaved.
Comment 3 Poke 2019-04-14 18:15:02 UTC
Updated pull request: https://github.com/audacity/audacity/pull/352
Comment 4 James Crook 2019-04-15 17:24:59 UTC
DEVEL - FIX MADE by Pokechu22
https://github.com/audacity/audacity/commit/1866be61453b6b2564deb961394c81e357f2baf9
Comment 5 Peter Sampson 2019-04-16 05:41:08 UTC
(In reply to James Crook from comment #4)
Testing on W10 with audacity-2.3.2-alpha-236-1866be61453b6b2564deb961394c81e357f2baf9

This now works as a proper Reset, restoring the vertical zoom level to that which obtained when the MIDI file was imported.

Tested with Multiple Midi tracks - each one resets to its own proper reset Vzoom level.