New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

A round of editorial changes #676

Merged
merged 1 commit into from Dec 27, 2018

Conversation

Projects
None yet
3 participants
@ndw
Contributor

ndw commented Dec 26, 2018

Over Christmas, I tried to do an end-to-end read through and make some editorial improvements. I haven't yet got all the way through, but here's a first bunch.

I don't think there's anything likely to be controversial in here, or viewed as non-editorial. Certainly that wasn't my intent.

The most striking change is probably that I redid the graphics because they were awful. (I can say that because I'm responsible for the ones in the 1.0 spec.) I also added a diagram at the beginning of section 6 which (I hope) clarifies what constitutes a connection.

Comments, etc.most welcome.

I've just discovered that diffs aren't working again. Will fix that later today.

@ndw ndw requested review from gimsieke , eriksiegel and xml-project Dec 26, 2018

@xml-project
Contributor

xml-project left a comment

Graphics look great. Thanks @ndw
Hope to do my issues until Sunday, so its looks as if we are almost done.

@eriksiegel

This comment has been minimized.

Contributor

eriksiegel commented Dec 26, 2018

I suppose there is a way to see the end result as a web page?

@ndw

This comment has been minimized.

Contributor

ndw commented Dec 26, 2018

@ndw ndw merged commit ff5e43e into xproc:master Dec 27, 2018

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment