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

[theming] file-icon incorrectly displays name icon with a dot in name #7680

Closed
kesslerdev opened this issue Apr 28, 2020 · 1 comment
Closed
Labels
bug bugs found in the application help wanted issues meant to be picked up, require help theming issues related to theming vscode issues related to VSCode compatibility

Comments

@kesslerdev
Copy link
Contributor

Bug Description:

The file-icon is incorrectly displayed for files with a dot in name.
For instance, the icon for the following file package.json is not properly displayed (with: https://marketplace.visualstudio.com/items?itemName=PKief.material-icon-theme)

Theia
Sélection_035

VS Code
Sélection_034

Resolution

Theia
Sélection_036

when a class contain a dot in name, in html element don't need to escape dot but in css style definition yes!

.theia-plugin-package\.json-file-name-icon.theia-plugin-json-ext-file-icon.theia-plugin-ext-file-icon.theia-plugin-file-icon, .theia-plugin-package\.json-file-name-icon.theia-plugin-json-ext-file-icon.theia-plugin-ext-file-icon.theia-plugin-file-icon::before, .theia-plugin-package-lock\.json-file-name-icon.theia-plugin-json-ext-file-icon.theia-plugin-ext-file-icon.theia-plugin-file-icon, .theia-plugin-package-lock\.json-file-name-icon.theia-plugin-json-ext-file-icon.theia-plugin-ext-file-icon.theia-plugin-file-icon::before, .theia-plugin-\.nvmrc-file-name-icon.theia-plugin-nvmrc-ext-file-icon.theia-plugin-ext-file-icon.theia-plugin-file-icon, .theia-plugin-\.nvmrc-file-name-icon.theia-plugin-nvmrc-ext-file-icon.theia-plugin-ext-file-icon.theia-plugin-file-icon::before, .theia-plugin-\.esmrc-file-name-icon.theia-plugin-esmrc-ext-file-icon.theia-plugin-ext-file-icon.theia-plugin-file-icon, .theia-plugin-\.esmrc-file-name-icon.theia-plugin-esmrc-ext-file-icon.theia-plugin-ext-file-icon.theia-plugin-file-icon::before {
    content: ' ';
    background-image: url('http://127.0.0.1:3000/hostedPlugin/PKief_material_icon_theme/icons/nodejs.svg');
    background-size: 16px;
    background-position: left center;
    background-repeat: no-repeat;
}

Additional Information

  • Operating System: Linux / docker
  • Theia Version: next
@akosyakov akosyakov added bug bugs found in the application help wanted issues meant to be picked up, require help theming issues related to theming vscode issues related to VSCode compatibility labels Apr 28, 2020
@akosyakov
Copy link
Member

kesslerdev added a commit to kesslerdev/theia that referenced this issue Jun 24, 2020
kesslerdev added a commit to kesslerdev/theia that referenced this issue Jun 24, 2020
kesslerdev added a commit to kesslerdev/theia that referenced this issue Jun 24, 2020
kesslerdev added a commit to kesslerdev/theia that referenced this issue Jun 24, 2020
kesslerdev added a commit to kesslerdev/theia that referenced this issue Jun 30, 2020
kesslerdev added a commit to kesslerdev/theia that referenced this issue Jul 2, 2020
kesslerdev added a commit to kesslerdev/theia that referenced this issue Jul 2, 2020
Signed-off-by: Jean-François Monnier <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug bugs found in the application help wanted issues meant to be picked up, require help theming issues related to theming vscode issues related to VSCode compatibility
Projects
None yet
Development

No branches or pull requests

2 participants