Conversation
|
@poikilotherm First of the last calls ;). Please review & merge if you're happy with the changes. |
|
@sdruskat I fixed some language (you instead of we) and added some screenshots behind toggles. Pictures often say more than a 1000 words, but I still wanted you to have the feeling of a short page. Please review and merge if you like it 😉 |
- For future reference, the sphinx-togglebutton extension provides a single global variable for changing toggle text, which is useless; we should investigate alternatives.
Cheers 👍
The way they are set up, the toggles disrupt the flow. I'll change the text on them to make them slightly more useful.
I'll make some changes on top of yours. No hard feelings, but in the future, I wouldn't mind working with review suggestions rather than one content commit ;). Makes changes more atomic and much easier to work with. |
poikilotherm
left a comment
poikilotherm
left a comment
There was a problem hiding this comment.
LGTM!
Sorry for the forced changes in the commit, usually not doing that... 😉
2 participants
Includes improvements such as mentioning setting a license in the CFF file for open access publication, and a list of improvements from @led02's post-merge comments in #180.