Atom-ide-ui: Misplaced gutter icons

Created on 31 Oct 2017  路  5Comments  路  Source: facebookarchive/atom-ide-ui

Description

Gutter icons from the diagnostics module pop up to the left of line numbers. In fact, while no icons are visible it looks fine, as soon as a warning pops up, the whole line number row is moved to the right to make space for the icons. Not super important, but I find the way the linter package does it to be way more pleasant to the eye.

Expected Behavior

screen shot 2017-10-31 at 18 07 47

Actual Behavior

screen shot 2017-10-31 at 18 02 40

Versions

  • Atom: 1.21.1
  • Client OS: macOS High Sierra 10.13
  • atom-ide-ui: 0.5.3
diagnostics help wanted

Most helpful comment

Just to note, the side of the line numbers that the linter-ui-default shows the indicators on is configurable.

All 5 comments

Just to note, the side of the line numbers that the linter-ui-default shows the indicators on is configurable.

Yup, seems like a great configuration option / PR candidate! We've made this the default to play nicely with Nuclide's debugger functionality, which also puts breakpoints on the left - but I definitely see how the right side could be better for you. The gutters are created on https://github.com/facebook-atom/atom-ide-ui/blob/master/modules/atom-ide-ui/pkg/atom-ide-diagnostics-ui/lib/gutter.js#L83.

Well, yeah linter-ui-default can be configured to place the icons on the left, but space for those icons is reserved from the get go, so they just appear in the reserved space.

With diagnostics however there is no space reserved from the beginning, and when it is needed it just changes the UI to make space, which weirds me out.

Even if that is fixed, it would still be nice to have an option to put the icons on the right.

With diagnostics however there is no space reserved from the beginning, and when it is needed it just changes the UI to make space, which weirds me out.

Ah, yea that's something that was fixed pretty early in linter-ui-default's development. A _lot_ of people don't like their UI jumping around on them 馃槢.

I changed the default behavior using custom style:

.gutter[gutter-name=diagnostics-gutter] {
  order: 1;
}
Was this page helpful?
0 / 5 - 0 ratings

Related issues

ghost picture ghost  路  3Comments

gregorygenova picture gregorygenova  路  3Comments

mehcode picture mehcode  路  5Comments

fenric picture fenric  路  3Comments

tiagonervis picture tiagonervis  路  4Comments