- From: Markus Lanthaler <markus.lanthaler@gmx.net>
- Date: Tue, 16 Sep 2014 13:49:09 +0200
- To: "'Robin Berjon'" <robin@w3.org>, "'Steve Glaser'" <sglaser@nvidia.com>, "'Keator, David'" <dbkeator@uci.edu>, <spec-prod@w3.org>
On 16 Sep 2014 at 12:32, Robin Berjon wrote:
> On 15/09/2014 23:37 , Steve Glaser wrote:
>> Just created a pull request to add this.
>>
>> https://github.com/darobin/respec/pull/336
>>
>> Recent email thread -- this matches my fork and avoids needless
differences.
>>
>> If you define overrideLogo, that is the URI of the image.
>> If you define overrideLogo and overrideLogoURI, overrideLogoURI is
>> the URI of the organization
>> If you define overrideLogo and overrideLogoWidth, overrideLogoWidth is
>> the width of the image.
>> If you define overrideLogo and overrideLogoHeight, overrideLogoHeight
>> is the height of the image.
>> If you define overrideLogo and overrideLogoAlt, overrideLogoAlt
>> is the alt text (default is 'Logo')
>>
>> If you don't define overrideLogo, the values of the other attributes are
>> ignored.
>
> This sounds good but I'd like to suggest a different syntax to avoid
> having lots of properties when in fact you're just defining one thing.
> How about (all defaults are to what is currently used):
>
> overrideLogo: {
> src: "logo URL"
> , href: "link"
> , width: "number"
> , height: "number"
> , alt: "some text"
> }
+1
In fact I suggested exactly the same on GitHub a couple of hours ago:
https://github.com/darobin/respec/pull/undefined#issuecomment-55716716
--
Markus Lanthaler
@markuslanthaler
Received on Tuesday, 16 September 2014 11:49:49 UTC