search

w3c / touch-events

https://w3c.github.io/touch-events/
26 stars 24 forks source link

Reformat code #2

Closed cynthia closed 8 years ago

cynthia commented 9 years ago

The master source for the spec is pretty poorly formatted - should fix this. Think the agreement was to do 4 space indentation, but the rest is a bit fuzzy.

cynthia commented 9 years ago

Probably only needed for master, since that's the only branch that will get active updates.

patrickhlauke commented 9 years ago

Probably only needed for master, since that's the only branch that will get active updates.

And v1-errata ... (made a tiny start just tightening some of the markup anyway in my recent PR)

RByers commented 9 years ago

@cynthia are you still planning on doing anything here? This doesn't really bother me (but I've been doing some little cleanups as I go).

cynthia commented 9 years ago

Are all the bigger edits in? I think Patrick and you did some stuff, but I could go in and do a full sweep if there aren't any big changes in progress that might cause merge conflicts later.

RByers commented 9 years ago

I've got two pull requests open right now. Maybe once those land? Not urgent or anything, was just trying to clean up open issues.

cynthia commented 9 years ago

Ok, then it's probably safer to do after the two PRs are merged.

RByers commented 9 years ago

@cynthia All open pull requests have now landed, so now would be a good time to do reformatting if you're still interested.

cynthia commented 9 years ago

@RByers Cool, will try to slide a PR in this week.

patrickhlauke commented 8 years ago

Although there are still a few inconsistencies, overall the markup doesn't look as horrendous as i remember it. Is this still an issue, or can it be closed?