Closed mimischi closed 5 years ago
IF you can fix this today @davesmith4398, I think we can still get it into the final version for the issue.
I just finished reading and pushed all commits. If necessary, I can group all/related commits by squashing.
Some more things that caught my eye, but might only be nitpicking:
/
(forward slash), but omitted other times (i.e. Table 3)Hi @mimischi - Thank you very much for your edits and suggestions. We have accepted all of your edits. In terms of your additional suggestions:
(1) For consistency, we have added full stops to all list items in the checklist, but not in the tables. (2) We have eliminated all white space immediately before and after forward slashes. (3) Per certain bibliographic guidelines, we have eliminated capitalization on the one "X-ray" instance.
@mimischi and @orbeckst - If/when you locate the appropriate lipidbook link, please let us know, and we will make the update. Thank you!
@mrshirts - The appropriate files (including published version 1) should be updated per this conversation.
@davesmith4398 Lipidbook is online again, as per https://github.com/davesmith4398/best_practice_membranes/pull/10#discussion_r246194620, and available at https://lipidbook.bioch.ox.ac.uk/.
The introduction contains a small typo, leading to a line break in the
.tex
file. I'm still reading and will update the PR if I discover anything else.