@obrienben That looks good. There are just some tiny stuffs to be considered:
There is a configuration item in webapp/resources/application.properties to enable/disable the screenshot feature from webapp side:
# Screenshot settings
enableScreenshots=true
Some command lines embeded in the page (harvest-screenshot-configuration-guide.html) are presented on one line. That would be better if they can be separated to individual lines.
I can't find a hyperlink in the contens index to link to the new added page (harvest-screenshot-configuration-guide.html)
There is a section in "system-administrator-guide.html" to describe the screenshot, named "Using the screenshot". The section is not formated well, and the contents can be covered by "harvest-screenshot-configuration-guide.html". That would be good if you could get rid of the section.
Read the Docs guide for the screenshot feature. Covers