[vlc-devel] [PATCH 3/3] qml: Aspect ratio widget changes

Rohan Rajpal rohanvideolan at gmail.com
Wed Jul 24 02:41:44 CEST 2019


Hi, can someone review these patches?

On Fri, Jul 19, 2019, 5:17 PM Rohan Rajpal <rohan17089 at iiitd.ac.in> wrote:

> Remove hardcoding from aspect ratio
> ---
>  modules/gui/qt/qml/player/ControlButtons.qml | 5 ++---
>  modules/gui/qt/qml/style/VLCStyle.qml        | 2 ++
>  2 files changed, 4 insertions(+), 3 deletions(-)
>
> diff --git a/modules/gui/qt/qml/player/ControlButtons.qml
> b/modules/gui/qt/qml/player/ControlButtons.qml
> index d3a672d3cd..6ce0ca971b 100644
> --- a/modules/gui/qt/qml/player/ControlButtons.qml
> +++ b/modules/gui/qt/qml/player/ControlButtons.qml
> @@ -394,10 +394,9 @@ Item{
>      Component{
>          id: aspectRatioDelegate
>          Utils.ComboBoxExt {
> -            id: combo
>              Layout.alignment: Qt.AlignVCenter
> -            height: 28 * scale
> -            width: 100 * scale
> +            width: VLCStyle.widthAspectRatio
> +            height: VLCStyle.heightAspectRatio
>              textRole: "display"
>              model: player.aspectRatio
>              onCurrentIndexChanged: model.toggleIndex(currentIndex)
> diff --git a/modules/gui/qt/qml/style/VLCStyle.qml
> b/modules/gui/qt/qml/style/VLCStyle.qml
> index bf0ca04933..971285531a 100644
> --- a/modules/gui/qt/qml/style/VLCStyle.qml
> +++ b/modules/gui/qt/qml/style/VLCStyle.qml
> @@ -97,6 +97,8 @@ Item {
>      property int widthSearchInput: 200 * scale;
>      property int widthSortBox: 150 * scale;
>      property int widthTeletext: 280 * scale;
> +    property int widthAspectRatio: 100 * scale;
> +    property int heightAspectRatio: 22 * scale;
>      property int heightInput: 22 * scale;
>
>      property int selectedBorder: 2
> --
> 2.17.1
>
> _______________________________________________
> vlc-devel mailing list
> To unsubscribe or modify your subscription options:
> https://mailman.videolan.org/listinfo/vlc-devel
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20190724/a13a043f/attachment.html>


More information about the vlc-devel mailing list