Hi Ronnie,

As Olivier commented, this update is what we hoped for.

A few further comments for you to consider:

SUMMARY SUB-SECTION
1. Should the summary state that this function calculates the “principal value”, which is an accepted term I believe. 2. Similarly, would it be worth mentioning “arc cosine” as well as “inverse trigonometric cosine”.

ARGUMENTS SUB-SECTION
1. Since the Syntax sub-section gives the argument name as Number, I would suggest that the singular form is used in this subsection too. 2. A very minor point, but is it worth putting a plus sign before “1”? 3. I would be wary about using the acronym NaN which may not be familiar to some of our users. Perhaps replace with the more specific “an invalid numeric value (#NUM!) error”. 4. Apologies but I do not know what the last sentence really means (“Because ACOS is a static method of Math, you always use it as Math”).

EXAMPLES SUB-SECTION
1.    I think the format looks great.
2. In description for third and fourth examples, radian and degree should both be plural.
3.    In description for fourth example, insert “by” before 180/PI().
4. In description for fifth example, again I would not bother with the NaN acronym. So maybe change “error (NaN)” to “an error”. Also I suspect you could safely delete “which is between -1 and 1” as it is already been stated. 5. A point for discussion – is the dynamic GIF appropriate for functions like this or is animation unnecessary for such a simple function? Would a static GIF convey the necessary information adequately? 6. Another point for discussion – as the GIF covers the whole of the function and isn’t really an example, would it be more appropriately placed in the Additional details sub-section?

I hope these thoughts help. And please keep up the good work!

Regards,

Steve


------ Original Message ------
From: "Olivier Hallot" <[email protected]>
To: [email protected]
Sent: 01/09/2020 15:58:56
Subject: Re: [libreoffice-documentation] Please give your feedback for this document edit

Hi Ronnie

I think we're on track. A small note below.

Em 01/09/2020 11:25, GANDHI RONNIE escreveu:
 Hello all,

 I have edited this Calc function ACOS's wiki page where I made a few
 changes:
 1. Following a tabular example format
 2. Added bad case example
 3. Added a gif I created myself to avoid copyright issues besides getting
 benefit of customizing to our needs

The vertical axis should mention "radians" and pi.

 4. Adding related functions
 5. Linked to corresponding ODF section

https://wiki.documentfoundation.org/Documentation/Calc_Functions/ACOS

 I would be applying similar changes to Calc functions one by one with an
 exception of Gifs and images only wherever needed.

 Looking forward to your feedback.
 Regards,
 Ronnie

regards
--
Olivier Hallot
LibreOffice Documentation Coordinator
Comunidade LibreOffice
Rio de Janeiro - Brasil - Local Time: UTC-03:00
http://tdf.io/joinus

--
To unsubscribe e-mail to: [email protected]
Problems? https://www.libreoffice.org/get-help/mailing-lists/how-to-unsubscribe/
Posting guidelines + more: https://wiki.documentfoundation.org/Netiquette
List archive: https://listarchives.libreoffice.org/global/documentation/
Privacy Policy: https://www.documentfoundation.org/privacy


--
To unsubscribe e-mail to: [email protected]
Problems? https://www.libreoffice.org/get-help/mailing-lists/how-to-unsubscribe/
Posting guidelines + more: https://wiki.documentfoundation.org/Netiquette
List archive: https://listarchives.libreoffice.org/global/documentation/
Privacy Policy: https://www.documentfoundation.org/privacy

Reply via email to