Re: ISSUE-164: Correct default for charging time if unknown in battery (was: [battery] next steps and questions)

> Personally I think it flows better when that paragraph is in the BatteryManager Interface section.

I agree

You can have non-normative text in the normative section to make it clear in context.

regards, Frederick

Frederick Hirsch, Nokia
Chair DAP
@fjhirsch



On Jul 2, 2014, at 12:39 PM, <cathy.chan@nokia.com> <cathy.chan@nokia.com> wrote:

>> From: ext Kostiainen, Anssi [mailto:anssi.kostiainen@intel.com]
>> 
>> On 02 Jul 2014, at 00:43, <cathy.chan@nokia.com> <cathy.chan@nokia.com>
>> wrote:
>> 
>>> Yep, that takes care of the inconsistency, which was my major concern.
>>> 
>>> Can we consolidate those into one paragraph, or is there some subtlety
>> that I'm missing? Each time I read through that part of the spec, I found
>> myself trying to figure out what the second paragraph is telling me that the
>> first wasn't.
>> 
>> The text above the IDL reads more like an introduction, while the text below
>> contains the normative assertions.
>> 
>> To clarify this, I moved the introductory-ish text that contains no normative
>> assertions to the Introduction section, 
> 
> Personally I think it flows better when that paragraph is in the BatteryManager Interface section.
> 
>> and made the other half of the text
>> talking about the defaults a note that I placed below the normative text
>> introducing the concept:
>> 
> 
> This is perfect! Thanks Anssi!
> 
> Regards, Cathy.
> 
> 
>> https://dvcs.w3.org/hg/dap/rev/3ab4af97cf9e
>> 
>> Please let us know if this addresses your concern, or whether we should
>> revert back to the previous. To ease the review, here are the updated
>> sections:
>> 
>> https://dvcs.w3.org/hg/dap/raw-
>> file/default/battery/Overview.html#introduction
>> https://dvcs.w3.org/hg/dap/raw-file/default/battery/Overview.html#the-
>> batterymanager-interface
>> 
>>> Also, with this latest edit, I think we can also s/the implementation is
>> unable to report the remaining charging time while the battery is charging
>> but not full/the implementation is unable to report the remaining charging
>> time/ in the description of the chargingTime. That extra phrase ("while the
>> battery is charging but not full") was added to address my previous feedback,
>> but I think we don't need it anymore. Mounir, WDYT?
>> 
>> Dropped "while the battery is charging but not full":
>> 
>> https://dvcs.w3.org/hg/dap/rev/eb02dc5983af
>> 
>> Mounir - feel free to revert or make further clarification as needed.
>> 
>> Thanks,
>> 
>> -Anssi
>

Received on Wednesday, 2 July 2014 19:19:35 UTC