vscode-peacock icon indicating copy to clipboard operation
vscode-peacock copied to clipboard

peacock in devcontainers doesn't revert to peacock.Color when peacock.remoteColor isn't specified

Open jamiekt opened this issue 1 year ago • 4 comments

Versions (please complete the following information):

  • OS: mac
  • VS Code version: 1.77.3 (Universal)
  • Peacock Version: v4.2.2

Describe the bug Documentation states that peacock.color will be used in devcontainers if peacock.remoteColor isn't specified:

When a workspace is opened in a remote context, if a peacock.remoteColor is set, it will be applied. Otherwise, the regular peacock.color is applied.

https://papapeacockstorage.z13.web.core.windows.net/guide/#remote-development-integration

The behaviour I'm seeing though is different. If I only specify peacock.Color then no color is displayed when I open the repo in the devcontainer. if I specify peacock.remoteColor then it works as expected.

To Reproduce Steps to reproduce the behavior:

issue

cd /tmp \
  && mkdir -p peacocktest/.vscode \
  && echo '{"peacock.color": "#6217a4"}' >> peacocktest/.vscode/settings.json \
  && code peacocktest

VSCode should open up and peacock will color it a lovely shade of purple.

image

Now open up the command palette (CMD+Shift+P), choose "Dev Containers: Add Dev Container Configuration Files..."

image

Choose any image (Alpine is fine)

image

We're now running inside the devcontainer but the peacock color has not been activated

image

Edit settings.json to the following:

{
    "peacock.color": "#6217a4",
    "peacock.remoteColor": "#6217a4"
}

Upon save the peacock color will be activated and workbench.colorCustomizations gets added to settings.json

image

Expected behavior If the docs are to be believed then I shouldn't have to add peacock.remoteColor

jamiekt avatar Apr 18 '23 20:04 jamiekt

Thank you @jamiekt for creating this issue!

johnpapa avatar Apr 18 '23 20:04 johnpapa

You may need https://github.com/johnpapa/vscode-peacock/pull/486

xgdgsc avatar Apr 25 '23 14:04 xgdgsc

You may need #486

I may need what? Sorry, I don't understand your meaning.

jamiekt avatar Apr 26 '23 22:04 jamiekt

It' s more of a reminder for the project owner to review my PR so this kind of issue might be related.

xgdgsc avatar Apr 27 '23 01:04 xgdgsc