merge-conflicts icon indicating copy to clipboard operation
merge-conflicts copied to clipboard

Conflict header overlay positioning bug

Open simonlayfield opened this issue 9 years ago • 17 comments

The conflict headers overlay the list of conflicts when scrolling.

atom-merge-issue

Using Atom v1.0.19 Merge Conflicts v1.3.6

simonlayfield avatar Oct 28 '15 00:10 simonlayfield

Hmm, it isn't doing this for me:

screen shot 2015-11-02 at 3 31 17 pm

What themes were you using?

The version of Atom may effect this as well, if the z-index of overlay decorations was changed in core somehow. Once I publish 1.3.7, can you confirm that this is still happening on 1.1.0?

smashwilson avatar Nov 02 '15 20:11 smashwilson

@smashwilson this just happened to me too… atom v1.1.0 merge-conflicts v1.3.7 Theme: Atom Light Syntax: Solarized Light

screen shot 2015-11-03 at 12 22 08

If you need any other info, just let me know.

hughker avatar Nov 03 '15 17:11 hughker

Hey @smashwilson - sure.

This is still happening for me. I'm using:

Atom v1.1.0 Merge Conflicts v1.3.7 Theme: One Dark Syntax: One Dark

simonlayfield avatar Nov 09 '15 03:11 simonlayfield

Add into atom styles.less file:

atom-text-editor {
  overflow: hidden;
}

It helps me.

maxkazar avatar Nov 25 '15 20:11 maxkazar

I've the same issue with : Atom v1.2.4 Merge Conflicts v1.3.7 Theme: One Dark Syntax: One Dark

thibauds avatar Nov 26 '15 15:11 thibauds

Thanks @maxkazar. In case anybody can't get to styles.less, it's in preferences -> config folder -> styles.less, and add the snippet that he posted.

bhtruong93 avatar Dec 06 '15 01:12 bhtruong93

same thing is happening for me, it's overlapping everything

screen shot 2016-01-08 at 12 57 05 pm

Atom v1.3.2 Merge Conflicts v1.3.7 Theme: One Dark Syntax: One Dark

andersryanc avatar Jan 08 '16 20:01 andersryanc

The overflow trick helps, but still looks a little odd when only parts of the UI overlap the line numbers

screen shot 2016-01-08 at 12 55 10 pm

andersryanc avatar Jan 08 '16 20:01 andersryanc

@andersryanc can you post your styles.less file?

bhtruong93 avatar Jan 08 '16 21:01 bhtruong93

I had to add "atom-text-editor" to line 15 (probably could have just replaced the class really - my styles file must be outdated)

/*
 * Your Stylesheet
 *
 * This stylesheet is loaded when Atom starts up and is reloaded automatically
 * when it is changed.
 *
 * If you are unfamiliar with LESS, you can read more about it here:
 * http://www.lesscss.org
 */

.tree-view {

}

atom-text-editor, .atom-text-editor {
  overflow: hidden;
}

.atom-text-editor::shadow .cursor {

}

/* !- Taken from Seti-UI */
.icons(@icons) when (@icons = true) {
  @import "styles/icons";
}
/*
 * Set below to false to not import Seti-UI icons.
 * Requries View -> Reload after changing value.
 * Mainly to use with file-icons package.
*/
.icons(false);

andersryanc avatar Jan 08 '16 21:01 andersryanc

I don't think there's anything wrong with your stylesheet. What's also odd is that your issue is a bit different from the previous ones. Yours is off in the x-direction, but the 'fix' seemed to have remedied the y-direction. :\

/*
 * Your Stylesheet
 *
 * This stylesheet is loaded when Atom starts up and is reloaded automatically
 * when it is changed and saved.
 *
 * Add your own CSS or Less to fully customize Atom.
 * If you are unfamiliar with Less, you can read more about it here:
 * http://lesscss.org
 */


/*
 * Examples
 * (To see them, uncomment and save)
 */

// style the background color of the tree view
.tree-view {
  // background-color: whitesmoke;
}

// style the background and foreground colors on the atom-text-editor-element itself
atom-text-editor {
  // color: white;
  // background-color: hsl(180, 24%, 12%);
}

// To style other content in the text editor's shadow DOM, use the ::shadow expression
atom-text-editor::shadow .cursor {
  // border-color: red;
}

atom-text-editor {
  overflow: hidden;
}

bhtruong93 avatar Jan 09 '16 00:01 bhtruong93

My guess would be that the other posts just didn't try scrolling left/right or had soft-wrap on or something so they couldn't.

andersryanc avatar Jan 09 '16 00:01 andersryanc

You're absolutely correct. I have the same issue haha. Note: At a small enough width, it snaps back into place (it's too small a width though).

bhtruong93 avatar Jan 09 '16 01:01 bhtruong93

https://github.com/smashwilson/merge-conflicts/issues/188#issuecomment-159723232 worked :+1:

rajashekar007 avatar Mar 04 '16 09:03 rajashekar007

Yes, on scrolling left it's overlay is on top of number line.

rajashekar007 avatar Mar 04 '16 09:03 rajashekar007

+1.

Atom 1.7.0 merge-conflicts 1.4.1 One Dark UI theme Oceanic Next syntax theme OSX 10.11.3

EDIT: Actually, it's worse. If the window loses focus, the UI here starts flickering like crazy. https://gyazo.com/8dfe2579a181366af614b1721f99785b

henrebotha avatar Apr 19 '16 13:04 henrebotha

2016-05-03_22-36-55

Same issue.

Atom 1.7.3 Merge-conflicts 1.4.2 One dark (UI theme) Oceanic next (Syntax theme)

aimuzov avatar May 03 '16 18:05 aimuzov