Re: Geolocation API Level 2 spec updated

Thanks for the review Wojciech!

On Thu, Jun 30, 2011 at 4:27 PM, Wojciech Masłowski
<wmaslowski@opera.com> wrote:
>
> Some editorial remarks:
>>
>>  Future versions of the API may allow additional attributes that provide
>> other information about this position (e.g. street addresses).
>
> Either remove this or change "street addresses" to something else as street
> address is a part of this and not future API.
>

Removed.

>>  interface Address {
>>    readonly attribute DOMString country;
>>    readonly attribute DOMString region;
>>    readonly attribute DOMString county;
>>    readonly attribute DOMString city;
>>    readonly attribute DOMString street;
>>    readonly attribute DOMString streetNumber;
>>    readonly attribute DOMString premises;
>>    readonly attribute DOMString additionalInformation;
>>    readonly attribute DOMString postalCode;
>>  };
>> All of the Address attributes are optional. If an implementation cannot
>> provide a particular attribute, its value must be null.
>
> If they are optional then shouldn't we add '?' to IDL?
>

Fixed.

Thanks,
Andrei

Received on Thursday, 30 June 2011 15:40:34 UTC