Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

broken with latest insiders? #20

Closed
chrisdias opened this issue Nov 23, 2017 · 11 comments · Fixed by #24
Closed

broken with latest insiders? #20

chrisdias opened this issue Nov 23, 2017 · 11 comments · Fixed by #24

Comments

@chrisdias
Copy link

with latest insiders, the explosions don't follow the cursor, they always appear at the beginning of the line. i know some work was done to optimize decorators in VS Code last month, maybe something broke?

@tonyxwz
Copy link

tonyxwz commented Nov 25, 2017

I have this issue too

@BillYang2016
Copy link

me too

2 similar comments
@SinarPandora
Copy link

me too

@jshamble
Copy link

me too

@hoovercj
Copy link
Owner

I have no idea why I'm not receiving emails for this repo, but I get them for my other repos. I'm so sorry that this has made it out of insiders and into the stable release (or so I assume from the reviews I'm getting). Unfortunately I'm traveling without a laptop for the holidays but I'll try to have this fixed early next year.

Thanks for your patience!

@hoovercj
Copy link
Owner

Update: the settings issue is resolved and I should be notified for all future activity here and in New issues.

@adrienluitot
Copy link

Hello,
I'm sorry to reopen this issue but I've the same problem but I don't think I've an insider version, i'm in 1.19.1.
I don't know if it's important but I'm on linux (ArchLinux with xfce4).

So can someone help me ?

Thanks in advance ! (I'm french, so sorry if I don't speak very well ^^)

@hoovercj
Copy link
Owner

@alfiory the "real" issue has not yet been fixed, I haven't come back from holidays yet.

The only "issue" I have fixed was with GitHub notification settings. Apologies for being unclear and happy new year!

@adrienluitot
Copy link

Okay sorry I didn't understand sorry, thanks so.

Thanks ! Happy New Year to you too ;)

@kirkone
Copy link
Contributor

kirkone commented Jan 12, 2018

Update
I just found this: Microsoft/vscode/#39259
So my assumption from below was not so far off.
After some fiddling around I realized that positioning with top and left is not an option anymore.
So there are some other ways to move elements around in CSS, right?

Replacing this:

left: `-${leftValue}ch`,
top: `-${topValue}rem`,

with:

['margin-left'] : `-${leftValue}ch`,
['margin-top']: `-${topValue}rem`,

did the trick.
I also changed the after in before and the result was awesome!

Can someone please check this changes?
I'm afraid I've overlooked something

Old Post

Hi,

did some debugging on this an have an idea what has changed.
My assumption is that the way how VSCode handles ::before and ::after has changed in the .19 update.
If I use the after property in DecorationRenderOptions they add a new span after the span with the current sign:

<span>
    <span class="mtk1">sdfgsd&nbsp;gsdfg&nbsp;sdffgs&nbsp;sdfgsdfaasdf</span>
    <span class="mtk1 ced-TextEditorDecorationType12-1 ced-TextEditorDecorationType13-1">x</span>
    <span class="ced-TextEditorDecorationType12-4 ced-TextEditorDecorationType13-4"></span>
</span>

the x is my last typed letter and the span after that has the ::after element:
image
This span does not have set position: relative. That is why the ::after element is positioned at the beginning of the line because this is the next element with relative positioning.

At the moment I have no clue how to fix this but I thought I will share my progress so far.

so long,
KirK

@kirkone kirkone mentioned this issue Jan 12, 2018
hoovercj added a commit that referenced this issue Jan 13, 2018
@hoovercj
Copy link
Owner

@kirkone provided a patch which fixes this and I've just published it in version 2.1.0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

8 participants