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

Allow color settings to save to Profile, instead of workspace. #517

Open
AferraroMezrah opened this issue Mar 10, 2023 · 18 comments
Open

Allow color settings to save to Profile, instead of workspace. #517

AferraroMezrah opened this issue Mar 10, 2023 · 18 comments

Comments

@AferraroMezrah
Copy link

Now that Profiles have left beta, I think it's reasonable to allow peacock settings to be saved based on profile and not just workspace.

I think it would be best to have peacock save the color setting to the active profile that is running.

Given the original purpose of peacock is project based colors I can see why you might want to leave it alone. But I think profiles mirror the workspace spirit in that it can be different for any number of projects.

For us, when we share workspace files to make sure the entire team is using the same settings for certain connections/extensions, not everyone wants a color change.

@johnpapa
Copy link
Owner

Thank you @AferraroMezrah for creating this issue!

@Grant-Keiter-FHR
Copy link

I would second this issue, I continually have to set and remove peacock settings in the projects I work on where workspaces or settings files are tracked.

@robokozo
Copy link

robokozo commented Apr 6, 2023

Glad to see this has been requested! I'd hate to force my questionable color choices on others :)

@thoroc
Copy link

thoroc commented May 22, 2023

Would that be covered by #7 ?

@noisy
Copy link

noisy commented Jun 21, 2023

I would love to see a progress on that. Settings saved to workspace is a deal-breaker for me.

@jadnhm
Copy link

jadnhm commented Feb 1, 2024

I think @thoroc is right this is a duplicate of #7

@thoroc
Copy link

thoroc commented Feb 2, 2024

Thanks for the ping. I completely forgot all about that one. :)

@AferraroMezrah
Copy link
Author

@thoroc #7 was written before profiles though. I think it makes sense to include that information on #7, although activity has been slow on that request. Or keep the dedicated profiles request here. Either way!

@jadnhm
Copy link

jadnhm commented Feb 7, 2024

@AferraroMezrah
Please read my summary of #7 here
#7 (comment)

You can see that the 'location where the colour information is saved' is dependant upon VSCode itself and unless someone can figure out an amazing hack there is not much https://github.com/johnpapa/vscode-peacock/ can do about it, unfortunately

@ncamaa
Copy link

ncamaa commented Apr 23, 2024

I think this is a crucial feature for people who work for organizations.
Maybe a hack that adds a hook for when the project is opened, and also some hooks before and after committing...

@jadnhm
Copy link

jadnhm commented Apr 26, 2024

@ncamaa I'm not sure I understand what you're suggesting. It's possible you don't understand the limitation described by #7 (comment)

If you want to see this issue resolved please pester the VSCode team about this issue
microsoft/vscode#40233

@AlexSapoznikov
Copy link

I rarely close my workspace. Could we just have an option to not save anything at all? It's easier to re-apply theme from favourites when opening workspace.

@jadnhm
Copy link

jadnhm commented May 29, 2024

@AlexSapoznikov I think it's possible you don't understand the limitation described by #7 (comment)

  • tl;dr: there is no way to tell VSCode to change the colours WITHOUT SAVING TO THE SETTINGS FILE

If you want to see this issue resolved please pester the VSCode team about this issue
microsoft/vscode#40233

@AlexSapoznikov
Copy link

AlexSapoznikov commented Jun 3, 2024

@jadnhm So you are saying that you can not apply new colors without touching .vscode/settings.json?
I thought that the limitation affects only restoring colors when opening vscode.

What I specifically was wondering that would it be possible to apply colors without touching .vscode/settings.json so that when I re-open my vscode it would not restore anything at all? Because right now when I apply a theme and reset settings in git, the theme stays as long as I close and re-open the window. I would just prefer using this without resetting git all the time.

@jadnhm
Copy link

jadnhm commented Jun 7, 2024

@AlexSapoznikov

So you are saying that you can not apply new colors without touching .vscode/settings.json?

yes that's exactly what I'm saying. Please read the linked comment from #7 and peruse that issue's other comments for more specifics

#7 (comment) :

image

What Peacock is attempting is not 'changing the theme' although that's something that could be attempted (probably by another extension?). Peacocks job is to leave EVERY OTHER SETTING besides window 'chrome' colours alone, and themes consist of many settings besides colour (such as fonts, for example)


I would just prefer using this without resetting git all the time

we all would prefer that which is why we're all so angry that microsoft/vscode#40233 has been ignored for so long and why I directed you there in my previous comment.

@AlexSapoznikov
Copy link

AlexSapoznikov commented Jun 11, 2024

@jadnhm I did read the whole thread before writing here. I just managed to misunderstand that vscode settings is also needed for applying colors and not only for restoring. In that case we can close this thread.
Thanks for clarifying!

@AferraroMezrah
Copy link
Author

@AferraroMezrah Please read my summary of #7 here #7 (comment)

You can see that the 'location where the colour information is saved' is dependant upon VSCode itself and unless someone can figure out an amazing hack there is not much https://github.com/johnpapa/vscode-peacock/ can do about it, unfortunately

I understand the limitation of the requirement to write to the settings.json file now. But the original question was about if the "Profile" feature of vscode can be used here. Does the profile create it's own settings.json? How does VSCode treat profiles.

Right now on my team we share a workspace configuration, but I've noticed that I can use different profiles without affecting our source control. And that's what led to the question, to both better understand profiles and what they do under the hood, and also see if that allows them to be used to control peacock settings in a way that isn't the workspace. Because clearly profiles are storing settings configurations per profile somehow. I was hoping it would apply here?

@jadnhm
Copy link

jadnhm commented Jul 29, 2024

@AferraroMezrah why don't you try out what you have in mind and report back?

The goal is to apply colours the way peacock does but you can't touch the file at .vscode/settings.json because that file is checked into source control.

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

No branches or pull requests

9 participants