Re: Syntax of new constraint proposal

I think 'ideal' will make this less inscrutable, e.g.

videoCfg = {
 require: ["width"],
 width: { min: 640, ideal: 1920 },
};

which to me reads even more cleanly that the mandatory/optional syntax.


On Sun, May 18, 2014 at 9:48 AM, Harald Alvestrand <harald@alvestrand.no>wrote:

> I have 10 lines of JavaScript that does the conversion. I do not see this
> as a good enough reason to reopen that discussion.
>
>
> On 18. mai 2014 14:53:04 GMT+00:00, "Cullen Jennings (fluffy)" <
> fluffy@cisco.com> wrote:
>>
>>
>> The old syntax looked something like
>>
>> videoCfg = {
>>  mandatory:
>>     { width: { min: 640 } }
>>  optional: [
>>     { width: { min: 1920 } },
>>     { width: { max: 2560 } } ]
>> }
>>
>> It’s pretty easy to read that and guess what it means.
>>
>> The new syntax looks like
>>
>> videoCfg = {
>>  require: ["width"],
>>  width: { min: 640 },
>>  advanced: [
>>     { width: { min: 1920 } },
>>     { width: { max: 2560 } } ]
>> };
>>
>> I find that much more opaque when it comes to guessing what it does. I prefer the old syntax to the new proposal. I think we need some discussion on how to make this less confusing to developers.
>>
>>
>>
>>
>>
>>
>>
>>
>>
> --
> Sent from my Android device with K-9 Mail. Please excuse my brevity.
>