<div dir="auto"><div><br><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Jul 29, 2019, 1:52 PM Pierre Lamot <<a href="mailto:pierre@videolabs.io">pierre@videolabs.io</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 2019-07-29 09:35, Rohan Rajpal wrote:<br>
> On Mon, Jul 29, 2019, 1:02 PM Alexandre Janniaux <<a href="mailto:ajanni@videolabs.io" target="_blank" rel="noreferrer">ajanni@videolabs.io</a>><br>
> wrote:<br>
> <br>
>> Hi,<br>
>> <br>
>> On Mon, Jul 29, 2019 at 10:22:25AM +0530, Rohan Rajpal wrote:<br>
>>> On Mon, Jul 29, 2019 at 2:38 AM Alexandre Janniaux<br>
>> <<a href="mailto:ajanni@videolabs.io" target="_blank" rel="noreferrer">ajanni@videolabs.io</a>><br>
>>> wrote:<br>
>>> <br>
>>>> Hi,<br>
>>>> <br>
>>>> It seems strange to assign an aspect ratio component to a width.<br>
>>>> Is it about setting the size of the aspect ratio button ?<br>
>>>> <br>
>>> Yeah.<br>
>>> I've got a better idea. Add generic values(width_xsmall,<br>
>> width_large etc)<br>
>>> in VLCStyle and use those here. I need to use these values in the<br>
>> editor<br>
>>> patches anyway.<br>
>> <br>
>> This looks far better indeed, but would these values being used for<br>
>> the whole<br>
>> interface or just the buttons ?<br>
> <br>
> Whole interface.<br>
> <br>
<br>
width_xsmall, width_large are too vague in my opinion, it provides no <br>
information on where it can be used. Everything has a width.<br>
I'd prefer something like combobox_width_small in you case.<br></blockquote></div></div><div dir="auto">Alright will do that. </div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
>> Regards,<br>
>> --<br>
>> Alexandre Janniaux<br>
>> VideoLabs<br>
>> _______________________________________________<br>
>> vlc-devel mailing list<br>
>> To unsubscribe or modify your subscription options:<br>
>> <a href="https://mailman.videolan.org/listinfo/vlc-devel" rel="noreferrer noreferrer" target="_blank">https://mailman.videolan.org/listinfo/vlc-devel</a><br>
> _______________________________________________<br>
> vlc-devel mailing list<br>
> To unsubscribe or modify your subscription options:<br>
> <a href="https://mailman.videolan.org/listinfo/vlc-devel" rel="noreferrer noreferrer" target="_blank">https://mailman.videolan.org/listinfo/vlc-devel</a><br>
</blockquote></div></div></div>