The Presenter console code lives in: sdext/source/presenter/* It was written by someone with no respect at all for the non-renewable resource that is vertical space :-) Which means there are five lines of blanks between each method, where there should be only one: so you can see more on the screen at once. That's quite annoying when editing - it'd be nice to fix :-)
Created attachment 55487 [details] [review] sdext/source/presenter/*: remove extraneous vertical whitespace Patch to collapse multiple vertical lines of whitespace into single lines. This is my first contribution so I'm not sure of the etiquette. Should I also get git to email the patch to the dev list?
Hi Rick, > This is my first contribution Welcome ! :-) I pushed the patch; thanks ... > so I'm not sure of the etiquette. Should I also get git to email > the patch to the dev list? It's by far the best to drop a patch to the dev list with [PATCH]: in the Subject - that gets the most attention and review, but no need for this. What we really would need is to have a mail to the developer list with an MPL/LGPLv3+ license statement, so we can add you to the wiki: http://wiki.documentfoundation.org/Development/Developers Anyhow - great to have a first patch in, and ... nice to see some more Cambridge blood; I was at Downing until 2000 :-) Regards, Michael (who clearly prefers E-mail ;-)
Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.