Improved descend report

[dave-khuon]

No description provided.

[dave-khuon]

Replacing PR #1820. Refactoring get_gender_symbol to utils.py per Nick's suggestion.

[Nick-Hall]

You probably don't want to instantiate a new Symbols class for each call. Some of our existing code creates a new instance in response to font changes. Does this give us a problem in reports where the system font isn't used and a font can be defined at the paragraph level?

It is also worth noting that the gender field cannot be None, so your function reduces to a single line. Perhaps it is better not to use a function at all. This needs more thought.

[Nick-Hall]

Actually, we probably don't have a problem with reports. The font issue is related to fallback symbols which are not defined for genders. If a chosen report font doesn't support the gender characters, then there isn't much we can do about it.

[dave-khuon]

Hi Nick,

Since this PR# has been successfully merged, is it ok for me to push similar changes for the other 2 reports (PRs #1819, #1818) that are dependent on the new utils.py to benefit from the same feature benefits? Please let me know.

[Nick-Hall]

Yes, please go ahead. I'll merge them when you're ready.