adobe/brackets — pull request #3365 by njx (2013-04-05 22:41:18+00:00)
Title: Correct fix for gutter padding issue in #3211

The original fix in #3212 (overriding a style change made in CodeMirror) was a temporary measure until we figured out the real problem. It turns out that the issue is that we were setting vertical `padding` on both `.CodeMirror-lines` and `.CodeMirror-gutters` (since we want a little space at the top of the code area). I think the gutter padding used to be necessary in v2 because the line numbers were actually separate, but now that the line numbers are laid out as part of the lines themselves, it seems like we're only supposed to have it on `.CodeMirror-lines`. (I'm confirming this with Marijn, but it seems correct just looking at how the DOM is structured now.)

(Also, it was probably wrong for this padding to be on `.CodeMirror-gutters` to begin with--it was probably on `.CodeMirror-gutter` in v2, but it was in a block of styles that ended up needing to be on `.CodeMirror-gutters` instead, so it just got moved along with them.)

The line numbers still seem to line up properly, and the original bug no longer reproduces for me with this change.