October 23, 2020, 10:11:47 pm

News:

--


"Audio Output" number of tracks (not) updating

Started by mm0359, August 22, 2016, 09:28:33 pm

Previous topic - Next topic

mm0359

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).)


eumagga0x2a

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)

mm0359

QuoteTo 2: why not? (NACK)


"Cancel" means "revert/forget": it's expected not to accept/trigger any change (ever)...

eumagga0x2a

March 23, 2017, 09:33:47 pm #4 Last Edit: March 23, 2017, 09:37:09 pm by eumagga0x2a
"Cancel" doesn't accept or trigger any changes, but it must propagate the current state to the GUI.

edit: s/"it"/"exiting the dialog"/

mm0359

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.