Comments (7)
This was intended behavior. I wrote this plugin to solve performance issues in my tree regarding vim-devicons, but my tree only needs to pass the filename as it is so it does not really matter to me.
Thing is, the more we add more logic inside this plugin, the slower it'll get for every user of the library. A note in the readme might be more appropriate, although i'm not sure the performance implications would be tremendous adding some parsing logic on the pathnames. I think i would accept a PR :)
from nvim-web-devicons.
@saviocmc Do you know if your solution was adopted by Telescope team?
from nvim-web-devicons.
@gegoune Yes, it was! Telescope now uses only the file name (not the complete file path, as before) to call this plugins, this way the icons are always consistent. This issue can be closed. Sorry for the delay, appreciate your work.
from nvim-web-devicons.
For future reference, here is the helper function that seems to do the trick:
Perhaps it could be added to this plugin as an input sanitization step if it proves not to be a significant performance hit.
from nvim-web-devicons.
That's is a good news, thank you for getting back to us!
from nvim-web-devicons.
@alex-courtis Wouldn't #84 (comment) solve regex issues?
from nvim-web-devicons.
@alex-courtis Wouldn't #84 (comment) solve regex issues?
We could add an option like the very recently added strict https://github.com/nvim-tree/nvim-web-devicons#get-icon
I'm very hesitant to make any changes that might break existing behaviour, as the blast radius for this plugin is very large.
from nvim-web-devicons.
Related Issues (20)
- Add arduino (.ino) icon HOT 2
- Add TypoScript (TYPO3) icons
- compose.yml , should have the same icon as docker-compose.yml and Dockerfile HOT 1
- Is there a way to set default options (hidden by default) HOT 1
- Font
- bug: OptionSet autocommand is clearing the Neovim intro screen HOT 3
- Errors showing in nvim due to hex color with 7 characters HOT 2
- TOML icon broken HOT 7
- Are you open to a PR for adding operating system icons? HOT 5
- Package-lock.json file name causes error HOT 3
- Outdated Terraform file icon
- New JS icon is not legible/recognizable in small size HOT 11
- Question - Where to find / edit git icons. Maybe not part of this codebase?
- Icons for C++20 module files doent work HOT 1
- Consolidate Default Icon Set: Seti HOT 1
- Material Icon Set
- Highlight Error HOT 4
- Haxe icon not showing HOT 2
- Add mp3 file icons HOT 4
- No way to add/override icons for filetypes HOT 4
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from nvim-web-devicons.