Avidemux Forum

Avidemux => Main version 2.6 => Topic started by: mm0359 on August 22, 2016, 09:28:33 PM

Title: "Audio Output" number of tracks (not) updating
Post by: mm0359 on August 22, 2016, 09:28:33 PM
v2.6.13 on Windows Vista and "v2.6.13 + usp10.dll" on WinXP.
(Issue already exists since at least v2.6.8...)

Steps to reproduce:
1. Load a .ts file, which has multiple audio tracks.
2. Run .js project.
adm = Avidemux()
adm.audioClearTracks()
adm.audioAddTrack(0)

2-Result. "Audio Output" unchanged = "(N track(s))".
3. Open "Audio > Select Track".
4. "Cancel" this dialog.
4-Result. "Audio Output" updated to "(1 track(s))".

Issue 1: Update should happen at step 2.
Possible solutions: UI notices this change too, or Project Run updates this UI too.

Issue 2: Update should not happen at step 4.
"Cancel" shouldn't trigger any UI update, should it?
(Only "OK" should (if any change).)
Title: Re: "Audio Output" number of tracks (not) updating
Post by: mm0359 on March 22, 2017, 11:06:56 PM
win32/avidemux_2.6.18_r170322_win32.exe (http://www.avidemux.org/nightly/win32/avidemux_2.6.18_r170322_win32.exe)
Bug(s) still there.
Title: Re: "Audio Output" number of tracks (not) updating
Post by: eumagga0x2a on March 23, 2017, 03:08:10 PM
Updating all kind of stuff on running a script should be easy, but this is really a very, very minor issue IMHO. Maybe in the next release cycle.

To 2: why not? (NACK)
Title: Re: "Audio Output" number of tracks (not) updating
Post by: mm0359 on March 23, 2017, 09:28:07 PM
QuoteTo 2: why not? (NACK)

"Cancel" means "revert/forget": it's expected not to accept/trigger any change (ever)...
Title: Re: "Audio Output" number of tracks (not) updating
Post by: eumagga0x2a on March 23, 2017, 09:33:47 PM
"Cancel" doesn't accept or trigger any changes, but it must propagate the current state to the GUI.

edit: s/"it"/"exiting the dialog"/
Title: Re: "Audio Output" number of tracks (not) updating
Post by: mm0359 on March 23, 2017, 10:12:24 PM
Quote"Cancel" [...] must propagate the current state to the GUI.

If "Cancel" (actually) allows no changes, then "current state"(!?) should be as valid as it was before opening the dialog, hence no need to propagate anything.
It should simply be(have) as if the dialog was never opened.