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

Custom icons are not displayed in plugin view #13096

Closed
bvenreply opened this issue Nov 23, 2023 · 1 comment · Fixed by #13101
Closed

Custom icons are not displayed in plugin view #13096

bvenreply opened this issue Nov 23, 2023 · 1 comment · Fixed by #13101

Comments

@bvenreply
Copy link
Contributor

Bug Description:

Custom icons are not displayed in plugin views. It seems that (some?) custom icons are displayed as inline elements by default, thus ignoring the width property.

image

Steps to Reproduce:

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 missing icon to the left of the cluster name, as compared to vscode.

Additional Information

  • Theia Version: 1.43.1
  • Browser: Chromium version 119.0.6045.159

I believe the issue was introduced with this change, in this PR: /pull/12827 . I tested setting the display mode explicitly to something other than inline, the icons were displayed correctly in that case (see image above).

@bvenreply
Copy link
Contributor Author

setting the display mode explicitly to something other than inline, the icons were displayed correctly in that case (see image above).

I had another look, it seems that as of merging #13014 this is no longer true on master, as neither width nor height are being set on this pseudo-element anymore.

bvenreply added a commit to bvenreply/theia that referenced this issue Nov 26, 2023
…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>
JonasHelming pushed a commit that referenced this issue Dec 14, 2023
)

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>
Co-authored-by: Beniamino Ventura <benia@protonmail.com>
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

Successfully merging a pull request may close this issue.

1 participant