[vlc-devel] [PATCH] qml: avoid setting the focus to the recent video section when this one is hidden

Alexandre Janniaux ajanni at videolabs.io
Tue Aug 13 08:11:06 CEST 2019


Hi,

LGTM.

Regards,

--
Alexandre Janniaux
VideoLabs

On Wed, Jul 31, 2019 at 06:30:11PM +0200, Pierre Lamot wrote:
> ---
>  modules/gui/qt/qml/mediacenter/MCVideoDisplay.qml | 7 ++++++-
>  1 file changed, 6 insertions(+), 1 deletion(-)
>
> diff --git a/modules/gui/qt/qml/mediacenter/MCVideoDisplay.qml b/modules/gui/qt/qml/mediacenter/MCVideoDisplay.qml
> index 59dfaf8456..93548b3a95 100644
> --- a/modules/gui/qt/qml/mediacenter/MCVideoDisplay.qml
> +++ b/modules/gui/qt/qml/mediacenter/MCVideoDisplay.qml
> @@ -356,7 +356,12 @@ Utils.NavigableFocusScope {
>                          onActionLeft: root.actionLeft(index)
>                          onActionRight: root.actionRight(index)
>                          onActionDown: root.actionDown(index)
> -                        onActionUp: recentsGV.forceActiveFocus()
> +                        onActionUp: {
> +                            if (recentsSection.visible)
> +                                recentsGV.forceActiveFocus()
> +                            else
> +                                root.actionUp(index)
> +                        }
>                          onActionCancel: root.actionCancel(index)
>
>                          /*
> --
> 2.17.1
>
> _______________________________________________
> vlc-devel mailing list
> To unsubscribe or modify your subscription options:
> https://mailman.videolan.org/listinfo/vlc-devel


More information about the vlc-devel mailing list