taskcluster / unified-logviewer

Mozilla Public License 2.0
2 stars 5 forks source link

Remove upper margin in log container #34

Closed tojon closed 7 years ago

tojon commented 7 years ago

I think it might be helpful to remove the margin at the top of the TaskCluster and Treeherder instances of the Logviewer, since it doesn't appear to serve a purpose, other than consuming space. I understand TaskCluster has its 'settings gear' at the right hand side, but that operates happily over log line content during scrolling anyway.

If there are other apps using unified-logviewer other than TaskCluster and Treeherder, or scenarios where that margin is needed, let me know.

I will put up a PR so you can evaluate the difference, and merge or not if you wish.

eliperelman commented 7 years ago

Yeah, this would be welcome.

eliperelman commented 7 years ago

Resolved via #35.