jccguimaraes / atom-project-viewer

Project Viewer - An Atom's package
MIT License
75 stars 15 forks source link

ISSUE - devicons not showing #135

Closed jccguimaraes closed 7 years ago

jccguimaraes commented 7 years ago

ISSUE

devicons not showing

Environment

Steps

Can't reproduce, just that some people don't have the devicons properly rendered

Hammster commented 7 years ago

I have the same issue again, 'devicons' as a font is loaded from another plugin in my case from toolbar. And sadly there are two diffrent devicons out there, my first idea would be to prefix our devicons font as file to prevent overriding and then also change the font name with the same prefix.

Hope this helps you, and keep up the great work πŸ’–

logo

jccguimaraes commented 7 years ago

this is awesome! Thanks. Let's pick each other's brain on this

Hammster commented 7 years ago

Preferable, we ditch the iconfonts in favor of SVG generally since we can have the advantage of electrons crossplatform consistant enviroment.

The main advantages are

The possible downside would be

But generally it only has advantages, we can even get the src icons of devicons which are SVGs already, same applies to octicons btw.

I'll try to provide a prototype implementation on my fork in the next days. πŸ”¨

jccguimaraes commented 7 years ago

cool work!

just one quick note, can we take them without breaking their license?

Hammster commented 7 years ago

MIT, we have to credit them and add a copy of the license which should not be a problem. Same for octicons ;)

jccguimaraes commented 7 years ago

@Hammster any updates on this ? :)

jccguimaraes commented 7 years ago

https://github.com/file-icons/DevOpicons

Hammster commented 7 years ago

@jccguimaraes jep, i am working on the changes just had to do it in small bits. I hope i'll be done in the next days. Also the icons look sharper, I can give you a screenshot tomorrow.

Also thanks for the link, this was a issue on my Todo πŸ‘

ninthart commented 7 years ago

Just to let you know, that while I have this issue too, it only happens in Atom Beta (1.17.0-beta5), and not in the stable channel…

Hammster commented 7 years ago

@ninthart could you tell me the plugins you use? on stable and beta

ninthart commented 7 years ago

@Hammster Sure. It's the same on both:

Community Packages (132) /Users/bradbrooks/.atom/packages β”œβ”€β”€ Altered@1.0.1 β”œβ”€β”€ atom-autocomplete-php@0.22.2 β”œβ”€β”€ atom-beautify@0.29.24 β”œβ”€β”€ atom-bootstrap4@1.4.0 β”œβ”€β”€ atom-commander@0.9.0 β”œβ”€β”€ atom-css-comb@3.1.0 β”œβ”€β”€ atom-fix-path@0.1.0 β”œβ”€β”€ atom-html-preview@0.1.22 β”œβ”€β”€ atom-image-resize@0.3.0 β”œβ”€β”€ atom-message-panel-service@0.0.1-1 β”œβ”€β”€ atom-minify@0.8.0 β”œβ”€β”€ atom-shortcuts@0.0.2 β”œβ”€β”€ atom-slack-snippets@0.2.0 β”œβ”€β”€ atom-ternjs@0.18.3 β”œβ”€β”€ atom-trello@0.7.1 β”œβ”€β”€ atom-zurb-foundation@2.0.0 β”œβ”€β”€ atomizr@0.23.1 β”œβ”€β”€ autocomplete-emojis@2.5.0 β”œβ”€β”€ autocomplete-font-awesome@0.1.0 β”œβ”€β”€ autocomplete-html-entities@0.1.0 β”œβ”€β”€ autocomplete-modules@1.6.10 β”œβ”€β”€ autocomplete-nunjucks@0.5.0 β”œβ”€β”€ autocomplete-paths@1.0.5 β”œβ”€β”€ autocomplete-project-paths@2.2.0 β”œβ”€β”€ autocomplete-wordpress-hooks@0.2.2 β”œβ”€β”€ autoprefixer@3.7.0 β”œβ”€β”€ browser-plus@0.0.87 β”œβ”€β”€ browser-refresh@0.9.0 β”œβ”€β”€ build@0.67.0 β”œβ”€β”€ build-grunt@0.4.0 β”œβ”€β”€ busy@0.7.0 β”œβ”€β”€ busy-signal@1.4.1 β”œβ”€β”€ caniuse-css@1.1.5 β”œβ”€β”€ chrome-color-picker@0.8.0 β”œβ”€β”€ clone-cursor@1.1.2 β”œβ”€β”€ color-picker@2.2.5 β”œβ”€β”€ config-import-export@0.3.1 β”œβ”€β”€ csslint@1.1.5 β”œβ”€β”€ csstree-validator@0.0.12 β”œβ”€β”€ dash@1.7.1 β”œβ”€β”€ data-atom@0.25.1 β”œβ”€β”€ docblockr@0.9.4 β”œβ”€β”€ emmet@2.4.3 β”œβ”€β”€ emmet-snippets-compatibility@1.0.6 β”œβ”€β”€ expose@0.14.0 β”œβ”€β”€ file-icons@2.1.5 β”œβ”€β”€ filecolor@0.4.0 β”œβ”€β”€ flexbox-snippets@0.3.0 β”œβ”€β”€ git-control@0.9.0 β”œβ”€β”€ git-plus@7.8.0 β”œβ”€β”€ git-time-machine@1.5.9 β”œβ”€β”€ grunt-runner@0.14.0 β”œβ”€β”€ highlight-column@0.5.2 β”œβ”€β”€ highlight-selected@0.13.1 β”œβ”€β”€ hosts-edit@1.0.0 β”œβ”€β”€ html-entities@0.4.0 β”œβ”€β”€ imagemin@1.5.1 β”œβ”€β”€ improved-autocomplete-php@1.1.1 β”œβ”€β”€ intentions@1.1.2 β”œβ”€β”€ language-jsont@0.3.0 β”œβ”€β”€ language-markdown@0.22.0 β”œβ”€β”€ language-nunjucks@4.1.0 β”œβ”€β”€ language-postcss@1.3.1 β”œβ”€β”€ language-pug@0.0.21 β”œβ”€β”€ language-sql-mysql@0.3.1 β”œβ”€β”€ language-svg@0.9.2 β”œβ”€β”€ less-than-slash@0.16.0 β”œβ”€β”€ linter@2.1.4 β”œβ”€β”€ linter-csslint@1.3.4 β”œβ”€β”€ linter-htmlhint@1.3.3 β”œβ”€β”€ linter-jshint@3.1.2 β”œβ”€β”€ linter-jsonlint@1.3.0 β”œβ”€β”€ linter-markdown@4.0.1 β”œβ”€β”€ linter-php@1.3.2 β”œβ”€β”€ linter-phpmd@1.5.3 β”œβ”€β”€ linter-pug@1.3.1 β”œβ”€β”€ linter-scss-lint@3.1.0 β”œβ”€β”€ linter-stylelint@3.10.0 β”œβ”€β”€ linter-tidy@2.3.1 β”œβ”€β”€ linter-ui-default@1.2.4 β”œβ”€β”€ lipsum@0.1.0 β”œβ”€β”€ livestyle-atom@0.2.9 β”œβ”€β”€ markdown-preview-enhanced@0.12.3 β”œβ”€β”€ markdown-preview-plus@2.4.9 β”œβ”€β”€ markdown-toolbar@0.1.2 β”œβ”€β”€ markdown-writer@2.6.5 β”œβ”€β”€ merge-conflicts@1.4.4 β”œβ”€β”€ minimap@4.28.2 β”œβ”€β”€ minimap-bookmarks@0.4.2 β”œβ”€β”€ minimap-codeglance@0.4.6 β”œβ”€β”€ minimap-cursorline@0.2.0 β”œβ”€β”€ minimap-find-and-replace@4.5.2 β”œβ”€β”€ minimap-highlight-selected@4.6.1 β”œβ”€β”€ minimap-linter@2.0.0 β”œβ”€β”€ minimap-pigments@0.2.2 β”œβ”€β”€ minimap-selection@4.4.0 β”œβ”€β”€ minimap-split-diff@0.3.7 β”œβ”€β”€ monokai-php-css-syntax@0.2.2 β”œβ”€β”€ navigate@0.1.17 β”œβ”€β”€ open-in-browser@0.5.0 β”œβ”€β”€ php-cs-fixer@4.1.0 β”œβ”€β”€ pigments@0.39.1 β”œβ”€β”€ platformio-ide-terminal@2.5.0 β”œβ”€β”€ project-viewer@1.0.14 β”œβ”€β”€ quick-editor@0.4.3 β”œβ”€β”€ scratch@1.2.0 β”œβ”€β”€ script@3.14.1 β”œβ”€β”€ simple-drag-drop-text@0.3.4 β”œβ”€β”€ snippet-generator@0.3.2 β”œβ”€β”€ source-preview@0.5.0 β”œβ”€β”€ source-preview-pug@0.2.0 β”œβ”€β”€ source-preview-sass@0.1.6 β”œβ”€β”€ split-diff@1.3.1 β”œβ”€β”€ stylefmt@4.1.1 β”œβ”€β”€ svg-preview@0.11.0 β”œβ”€β”€ svgo@2.8.5 β”œβ”€β”€ sync-settings@0.8.1 β”œβ”€β”€ toggle-quotes@1.0.1 β”œβ”€β”€ tool-bar@1.1.0 β”œβ”€β”€ tool-bar-almighty@0.8.2 β”œβ”€β”€ transform@0.4.0 β”œβ”€β”€ tree-view-autoresize@1.6.0 β”œβ”€β”€ tree-view-copy-relative-path@1.1.0 β”œβ”€β”€ tree-view-git-branch@0.1.2 β”œβ”€β”€ tree-view-git-status@1.3.1 β”œβ”€β”€ tree-view-open-files@0.3.0 β”œβ”€β”€ twilight-updated-ninthart-syntax@0.5.3 β”œβ”€β”€ twilight-updated-syntax@0.5.5 β”œβ”€β”€ wordcount@2.10.4 β”œβ”€β”€ wordpress-api@1.2.2 β”œβ”€β”€ wordpress-dictionary@4.4.0 └── xeta-syntax@0.5.0

Git Packages (1) /Users/bradbrooks/.atom/packages └── emmet-atom-plugin@0.0.1 (emmetio/atom-plugin#45a43c4b)

Hammster commented 7 years ago

Thx for the fast reply, i hope after the icon system rewrite this will no longer happen. btw @jccguimaraes current state is that i still need to fix some things on mac and linux but it works on windows.

I will give you some insight once i am home

swensorm commented 7 years ago

Some of my colleagues have had this happen over the last few months and it just happened to me after Atom updated to 1.17. Once gone, we haven't found a way to get them back, even after wiping out all traces of Atom and re-installing.

Somehow the styling is disappearing. In the below image, the icon works but the devicon does not. It's missing the css ::before for displaying the image altogether. image

The stylesheet project-viewer.less is loading and it contains all the .devicons styles, but for some reason they're not being used and this is instead: image

I searched .atom for BlinkMacSystemFont and only found it as part of linter-ui-default, and the compile-cache folder for project-viewer. I have been unable to find where that's being pulled from and included in the less and style-manager compile caches.

mdeuerlein commented 7 years ago

FIX: Just move the inline import of devicons (line 199) outside the class .pv-has-icons and to the top (line 3)

File: atom-project-viewer/styles/project-viewer.less Line199: @import (inline) "../node_modules/devicons/css/devicons.min.css";

see commit: 18b8bb44648df15951c68837f89a47ad64c94296

jccguimaraes commented 7 years ago

:scream: