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

Change VS Code launcher icon for the code-server icon #2417

Closed
marcelovilla opened this issue Apr 19, 2024 · 5 comments
Closed

Change VS Code launcher icon for the code-server icon #2417

marcelovilla opened this issue Apr 19, 2024 · 5 comments
Assignees
Labels
area: integration/code-server impact: low 🟩 Minimally disruptive or niche needs: discussion 💬 Needs discussion with the rest of the team type: enhancement 💅🏼 New feature or request

Comments

@marcelovilla
Copy link
Member

Context

Currently, it seems that the only reason we're using the vscode-binder package is to add the VS Code icon to the Jupyterlab launcher:

https://github.com/nebari-dev/nebari-docker-images/blob/25697f163e296a7f03a513d0140b2b31ac109029/jupyterlab/environment.yaml#L61

If this is the only use for it, we should probably remove it and find a way of adding the code-server icon instead.

Value and/or benefit

Besides getting rid of an extra dependency, it seems there are some issues in vscode-binder for newer versions of code-server.

Anything else?

No response

@marcelovilla marcelovilla added the needs: triage 🚦 Someone needs to have a look at this issue and triage label Apr 19, 2024
@krassowski
Copy link
Member

I'm on it as a sub-task in JupyterLab launcher redesign. I had opened betatim/vscode-binder#60 yesterday.

@viniciusdc
Copy link
Contributor

@krassowski can we assign this issue to you? you seem to have a better understanding of what needs to be addressed in both places already

@viniciusdc
Copy link
Contributor

@marcelovilla @krassowski the reason for the dep. being there was due to this PR #1044. The goal was to replace the installation script that we rely on for installing the code-server and use only the extension -- but we had issues with that approach (probably due to support with jupyterhub versions), and them we cycled back (there is a PR somewhere), and I guess the extension was missed

@viniciusdc viniciusdc added type: enhancement 💅🏼 New feature or request needs: discussion 💬 Needs discussion with the rest of the team impact: low 🟩 Minimally disruptive or niche area: integration/code-server and removed needs: triage 🚦 Someone needs to have a look at this issue and triage labels Apr 19, 2024
@krassowski krassowski self-assigned this Apr 19, 2024
@krassowski
Copy link
Member

Upstream PR betatim/vscode-binder#61 was merged. The icon should update next time the docker images are regenerated. There is still some follow-up work in JupyterLab because the icon currently is invisible in dark mode.

@krassowski
Copy link
Member

Closing - in addition to upstream providing the updated icon, we are also shipping jupyterlab-new-launcher which also has the updated icon (which works better with dark mode):

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: integration/code-server impact: low 🟩 Minimally disruptive or niche needs: discussion 💬 Needs discussion with the rest of the team type: enhancement 💅🏼 New feature or request
Projects
Development

No branches or pull requests

3 participants