Avidemux Forum

Avidemux => Main version 2.6 => Topic started by: Jan Gruuthuse on March 13, 2017, 10:46:19 AM

Title: fixed: Menu access availabilty 2.6.18 / Qt5 2017/03/14
Post by: Jan Gruuthuse on March 13, 2017, 10:46:19 AM
Fixed: commit 22aaede            [UI] Disable menu items during playback, fix menus disabled after drag-n-drop
After installing todays build:
723b908            Merge branch 'master' of github.com:mean00/avidemux2
I noticed the menu is (partial) unavailable until you played the loaded video 1st.
Could you check and confirm the issue?
Title: Re: Menu access availabilty 2.6.18 / Qt5
Post by: eumagga0x2a on March 13, 2017, 12:07:20 PM
It should not (and it doesn't) matter that a video was played, but it must be loaded. Disabling unusable menu items depending on context is the purpose of the merged first patch. More to follow.

edit: I didn't notice any anomalies during my testing, but if I missed something, it would be helpful to know which particular menu items were disabled after a video was loaded but got enabled once it was played (just played, no meddling with markers and such).
Title: Not Ok
Post by: Jan Gruuthuse on March 13, 2017, 01:24:08 PM
I was able to narrow this down.
Indicator is when video is loaded that above left icons (save and info) stay grey. (see attached VideoLoadedNotOK)
when this is happening several options are greyed out in several menus
Example Edit menu: options greyed out Cut, Copy, Delete, Set Marker A, Set Marker B & Reset Markers (see attached EditNotOK)

Title: OK
Post by: Jan Gruuthuse on March 13, 2017, 01:26:57 PM
With video loaded normally you would get:
Title: Re: Menu access availabilty 2.6.18 / Qt5
Post by: Jan Gruuthuse on March 13, 2017, 01:32:07 PM
video forwarded to developer(s) pm
Title: Re: Menu access availabilty 2.6.18 / Qt5
Post by: eumagga0x2a on March 13, 2017, 01:54:37 PM
Thanks, never got loaded video not detected in my testing. Some refresh issues with QtGl? Will retest later today.
Title: Re: Menu access availabilty 2.6.18 / Qt5
Post by: Jan Gruuthuse on March 13, 2017, 02:27:29 PM
Is Drag and Drop related/required. Behaviour not limited to posted .mkv. Does the same with mpeg-ts.
Happens on all selected Display Options.

Preferences: [Display]: OpenGl, VDPAU, Xvideo & QT
with and without selected:
Gui Rendering Options
[ ] Enable OpenGl support

Loading with recent file all is OK.
Title: Re: Menu access availabilty 2.6.18 / Qt5
Post by: eumagga0x2a on March 13, 2017, 03:05:09 PM
Thank you for the hint. Indeed, I didn't test with drag&drop. Quite possible that it doesn't fire the signal.
Title: Re: Menu access availabilty 2.6.18 / Qt5
Post by: Jan Gruuthuse on March 13, 2017, 04:18:10 PM
Took me a while to understand what was going on and test this.
Title: Re: Menu access availabilty 2.6.18 / Qt5
Post by: eumagga0x2a on March 13, 2017, 06:45:36 PM
I've submitted a pull request (https://github.com/mean00/avidemux2/pull/110) which includes the fix for this bug. Thank you for catching this.
Title: Re: Menu access availabilty 2.6.18 / Qt5
Post by: Jan Gruuthuse on March 14, 2017, 06:43:53 AM
Fixed: commit 22aaede            [UI] Disable menu items during playback, fix menus disabled after drag-n-drop
Thanks for fixing, today's build does work fine now.
Title: Re: fixed: Menu access availabilty 2.6.18 / Qt5 2017/03/14
Post by: me444 on March 14, 2017, 05:42:10 PM
Another defect apperars to be related to this issue:

Win7 64 bit, nightlies after march 04:
if you start ADM on a selected file from the windows explorer environment
using the ShellExecute resp. ShellExecuteEx  (windows api call) method (start the app using a file name as parameter),
then now the File-Append (Ctrl-A) does no longer work.  The menu item is grayed out (disabled).
version march 04 was ok, afterwards (march 13 and 14) not ok.
-
If i start the adm executable directly and load a file afterwards explicitely, then the File-Append menu item will be enabled.


Title: Re: fixed: Menu access availabilty 2.6.18 / Qt5 2017/03/14
Post by: eumagga0x2a on March 14, 2017, 05:51:18 PM
I hope this will be fixed once [UI] Fix menus not enabled if video was loaded on command line (https://github.com/mean00/avidemux2/pull/113) gets merged.
Title: Re: fixed: Menu access availabilty 2.6.18 / Qt5 2017/03/14
Post by: me444 on March 14, 2017, 05:57:13 PM
Thanks for the info, eumagga0x2a !
Title: Re: fixed: Menu access availabilty 2.6.18 / Qt5 2017/03/14
Post by: me444 on March 15, 2017, 09:18:21 AM
With the second build (#360) from mar 14 it appears to be fine again :)