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

Highlight colors are not saved in VS2015 #37

Closed
adigostin opened this issue Jan 2, 2017 · 21 comments
Closed

Highlight colors are not saved in VS2015 #37

adigostin opened this issue Jan 2, 2017 · 21 comments
Assignees
Labels

Comments

@adigostin
Copy link

adigostin commented Jan 2, 2017

See the attached image. I changed a few highlight colors, but the Asm Dude plugin ignores them and uses the predefined colors instead. Restarting VS doesn't help.
(I have v1.7.3.2 on VS2015.)
highlight

@adigostin
Copy link
Author

Turns out Asm Dude keeps two sets of colors for highlighting:

  • one in the "AsmDude" page of the Options dialog (as far as I can tell these colors are ignored).
  • one in the Environment -> Fonts and Colors page (these are actually used).
    Why the unused colors?
    And how can I make the font not italic?

@HJLebbink
Copy link
Owner

I'll look into it.

Can you recall when this behaviour first occurred, was this bug introduced in the last release?

@adigostin
Copy link
Author

I can't tell when this was introduced. I just installed the addon.

@HJLebbink
Copy link
Owner

You will need to restart VS for these changes to be picked up. Have you restarted VS?

@adigostin
Copy link
Author

Yes, I had restarted it. No effect.
I just tried again with a new color that I configured in Options -> AsmDude -> General. When I closed the dialog with "OK" it asked for a restart. I restarted VS and again the color I had just set it ignored. The colors actually used are those in Options -> Environment -> Fonts and Color -> Display Items -> AsmDude-XXX.

@HJLebbink
Copy link
Owner

You are right, VS2015 does not save the these options to the properties you mentioned (Options -> Environment -> Fonts and Color -> Display Items -> AsmDude-XXX). VS2017 works fine. For the time being you could edit the AsmDude-XXX values.

@HJLebbink
Copy link
Owner

@adigostin, I've release a new version (1.7.3.5) in which I cannot reproduce the issue in which the colours are not saved.

Could you please try this new version and check whether you still have that problem in VS2015.

@adigostin
Copy link
Author

adigostin commented Jan 3, 2017

I'm not sure what you changed in 1.7.3.5, cause the issue seems to still be there. See the screenshot below; that's after a VS restart.
What has changed in 1.7.3.5 (i.m.o. for the better) is the font, which used to be italic and now it's not.

(Why not remove those color settings altogether? The AsmDude-XXX colors seem to do the job well, and they don't need VS to be restarted.)

highlight

@HJLebbink HJLebbink changed the title Highlight colors configured by the user in the Options dialog are ignored Highlight colors are not saved Jan 3, 2017
@HJLebbink
Copy link
Owner

I'll need to check this on a fresh machine.

@ghost
Copy link

ghost commented Jan 25, 2017

In my PC change color directive work.

@HJLebbink
Copy link
Owner

@francescogomez Do you use VS2015 or VS2017 RC?

@HJLebbink HJLebbink changed the title Highlight colors are not saved Highlight colors are not saved in VS2015 Jan 26, 2017
@ghost
Copy link

ghost commented Jan 26, 2017

@HJLebbink VS2017 RC

@rarmenia
Copy link

rarmenia commented Feb 2, 2017

Was looking into this bug. I uninstalled ASMDude and reinstalled it. Colors updated to the preferences I wanted.

@HJLebbink
Copy link
Owner

@lazerzes To help me reproduce this, which version of AsmDude and VS did you use? Did you download an installer (from GitHub) or did you use the extension installer embedded in VS?

@rarmenia
Copy link

rarmenia commented Feb 3, 2017

Using VS2015(w/ Update 3 idk if that matters.)

I used the Marketplace link with version 1.7.3.10 of AsmDude.
I have duplicated it on multiple machines at this point as well.

@therealdreg
Copy link

therealdreg commented Apr 14, 2017

I have the same problem with the last release in VS 2015:

confirmed

@stonedreamforest
Copy link

I have the same problem

@theawkwardchild
Copy link

I just installed v1.9.2.0 of asm-dude from Visual Studio Marketplace running on VS2017 15.3.4 and I am also experiencing colors not being saved. Other settings are saving such as I turned off Keyword Highlighting for Backgound and Border but could not get the colors to change.

@stonedreamforest
Copy link

stonedreamforest commented Dec 5, 2017

and vs 2017 15.5.0

  • Please don‘t use blue for the default color,It's too blue in the dark theme(:

@HJLebbink
Copy link
Owner

HJLebbink commented Dec 23, 2017

I'm (still) working on a fix. (You could add a bounty to this question, I may help... https://stackoverflow.com/questions/47956223/how-to-change-a-classificationformatdefinition)

@stonedreamforest and @artemisart If you have a color scheme that works for both the dark theme and the regular theme, I will make it the default.

@HJLebbink HJLebbink self-assigned this Dec 23, 2017
@HJLebbink
Copy link
Owner

Hopefully fixed this bug in version 1.9.3.6. MEF components are cached and only refreshed when windows is restarted, and not always when VS is restarted. The bugfix is flushing this cache. Hopefully this has fixed your bug, it at least has fixed it for me, for the moment...

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

No branches or pull requests

6 participants