search

eoneill / JSHintMate.tmbundle

JSHinting for TextMate2
MIT License
0 stars 1 forks source link

JSHint resizes the main window #1

Open acdha opened 11 years ago

acdha commented 11 years ago

I noticed that when the JSHint output is shorter than the window height, the entire TextMate window shrinks to fit the output pane

eoneill commented 11 years ago

Hrm, which version of TextMate are you using?

I'm assuming this happens in the "detailed" view. e.g hitting ⌘K?

Thanks for reporting this!

acdha commented 11 years ago

Yeap - using 2.0-alpha.9427 if I hit ⌘K the window shrinks to the height of the detailed report view, but apparently not less than 14 lines or so:

screen shot 2013-06-03 at 5 37 44 pm

I noticed that TextMate has an “Auto–adjust window size” option but I don't have it enabled. Interestingly, if I switch displaying command output on the right to the default display on the bottom, the behaviour changes to shrink the window width rather than height.

acdha commented 11 years ago

If I delete the resizeBy call in output.html, the behaviour goes away with no apparent drawbacks:

https://github.com/eoneill/JSHintMate.tmbundle/blob/master/Support/output.html#L84

This doesn't change anything when there is no output and it doesn't interfere with the window size allowing scrolling when there's tons of output.

eoneill commented 11 years ago

Ah! My TextMate is set to show the output in a separate window.

screen shot 2013-06-03 at 4 50 47 pm

I'll look into removing the resizeBy method. Thanks for looking into this!