Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix: set display: block on plugin icon pseudo-element (#13096) #13101

Merged

Conversation

bvenreply
Copy link
Contributor

What it does

Fixes #13096

Fix the icon pseudo-element being displayed as inline by default. This meant that the width and height properties were being ignored and the icons displayed with 0 width.

Also fix css syntax for width and height properties.

How to test

The most mainstream vscode extension I could reproduce the issue with was ms-kubernetes-tools. If you have no Kubernetes clusters, you can have one displayed using this dummy kubeconfig:

apiVersion: v1
kind: Config
current-context: dev
preferences: {}
clusters:
  - cluster:
      server: https://fake-k8s.com.local
    name: dev
users:
  - name: dev
    user:
      token: dummy
contexts:
  - name: dev
    context:
      cluster: dev
      user: dev

Just save it somewhere, then add it with the command Kubernetes: Set Kubeconfig.

Then just open the Kubernetes plugin view and note the no longer missing icon to the left of the cluster name.

I also found the issue reproduced in the gitlens plugin list of commits, here's a before/after comparison:

image

To reproduce this you can just install gitlens and open any git repository.

All this works on the Theia browser example.

Follow-ups

None that I can see.

Review checklist

Reminder for reviewers

…a#13096)

Fix the icon pseudo-element being displayed as `inline` by default.
This meant that the `width` and `height` properties were being ignored
and the icons displayed with 0 width.

Also fix css syntax for `width` and `height` properties.

Signed-off-by: Beniamino Ventura <benia@protonmail.com>
@msujew msujew added plug-in system issues related to the plug-in system ui/ux issues related to user interface / user experience labels Nov 28, 2023
@JonasHelming JonasHelming requested a review from planger December 12, 2023 08:00
Copy link
Contributor

@planger planger left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, looks good to me! 👍

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
plug-in system issues related to the plug-in system ui/ux issues related to user interface / user experience
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

Custom icons are not displayed in plugin view
5 participants