-
Notifications
You must be signed in to change notification settings - Fork 39
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
Cycle decoration improvements #184
Cycle decoration improvements #184
Conversation
grahambates
commented
Nov 30, 2022
- Increase width to handle 3 digit ranges
- Add byte size on hover
- Add horizontal rules between sections in hover
- Prevent misalignment / jumping while editing due to delay updating counts. Every line always has an 'empty' decoration to add horizontal space and the actual counts are positioned in front of this.
- Increase width to handle 3 digit ranges - Add byte size - Add rules between sections
Prevents misalignment / jumping while editing due to the delay adding decorations. The idea is that *every* line always has an 'empty' decoration to add horizontal space. We can add this immediately to new lines on edit. The actual cycle count decoration is added in addition to this and absolutely positioned on top, so doesn't affect layout.
Nice! This was bothering mee, too, but I never got around to doing something about it :) |
Btw, is there a way to immediately add that empty space? when cyling through assembly files in the explorer, I still see the text on the left before the space is inserted for about 0.5s or so.. |
Also, I was just testing it with your M68000 assembly extension, and it parses and colorizes correctly, but shows this error:
This happens only after opening a |
The space should be added first. Maybe it's just down to how the rendered view updates. I'll have another look at this though. |
It appears sooner than the cycle count, but still a delay after the document is shown. |
Thanks for flagging the issue on my extension. It looks like Vasm is failing when trying to assemble to a temporary file, which it does to generate diagnostics. This might be permission related. |
Maybe it runs too often, even parallel and multiple processes try to access |
Setting the empty decorations is the first thing that happens on |
Yeah it's a bit strange as the text comes out fully colorized at the same time. |
Yeah that's pretty much what it looks like for me. I'm not sure there's much we can do about it. On the m68k diagnostics thing, I suspect it might be Windows specific. The thing is though, we really don't need two extensions assembling the same file and generating the same diagnostics! When it does run successfully you just end up with duplicate messages: I'm going to open a PR to have it disable the |
Looks like the delay is a known issue: |
Alrighty. No more we can do then. |