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

Pull request 1287 makes PluginSharedStyle.toIconClass() not work in some cases #13310

Closed
jcortell68 opened this issue Jan 24, 2024 · 3 comments
Closed

Comments

@jcortell68
Copy link
Contributor

@msujew Are you aware of any regression caused by this pull request of yours, 12827 ?

The addition of ::before to the selector name causes the resulting icon class to not have effect in some cases. The icon isn't shown. I ran into this with my app. Creating a standalone reproducible case will take some effort. Before I dive into that, I want to make sure that you're not already aware of the issue.

Bug Description:

TBD

Steps to Reproduce:

TBD

Additional Information

  • Operating System: Windows
  • Theia Version: 1.43.0
@msujew
Copy link
Member

msujew commented Jan 24, 2024

Hey @jcortell68, yes I am aware of that and this issue has been resolved by #13101 which is included in Theia 1.45.0. Can you try using that to see whether it resolves the issues you were experiencing?

@jcortell68
Copy link
Contributor Author

jcortell68 commented Jan 24, 2024

Ahh. Glad I asked. Let me see if the change fixes it in my situation.

@jcortell68
Copy link
Contributor Author

Bingo! It sure does. Thanks!

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

2 participants