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

Display material_design_icons in IntelliJ #7

Closed
StefanLobbenmeier opened this issue Mar 19, 2019 · 5 comments
Closed

Display material_design_icons in IntelliJ #7

StefanLobbenmeier opened this issue Mar 19, 2019 · 5 comments

Comments

@StefanLobbenmeier
Copy link

Is there any way the icons could also be displayed in IntelliJ? The normal Icons are shown beside the corresponding line. We would probably have to ask the creators of the plugin and edit this generator:
grafik

https://github.com/flutter/flutter-intellij/blob/7f67fa1f4100ca09bebb49725069b1d3e007a748/tool/icon_generator/lib/main.dart

@ziofat
Copy link
Owner

ziofat commented Mar 19, 2019

I have never used IntelliJ but I will look into this.

@huobazi
Copy link

huobazi commented Jul 14, 2019

image
also android studio

@ziofat
Copy link
Owner

ziofat commented Jul 18, 2019

And VSCode as well.
image

@StefanLobbenmeier
Copy link
Author

@ziofat I found an IntelliJ Plugin that already supports Icons from https://pub.dev/packages/flutter_vector_icons - a library that looks like it is a superset of the icons here. Do you think it makes sense to fork https://github.com/marius-h/flutter_enhancement_suite and add support for this package - or would it make more sense to merge this package with the https://pub.dev/packages/flutter_vector_icons package.

Sorry for putting you in such a position. I really don't want to disrespect this package and the work you've put in it. On the other hand - if you're using this package only as a part of another project it would probably be nice to resolve this redundancy.
On the other hand it might also be worthwhile to keep this package as a more "lightweight" font for people that don't want to bloat their app with too many unused fonts.

What do you think?

@ziofat
Copy link
Owner

ziofat commented Nov 13, 2019

@StefanLobbenmeier Hi, sorry it takes long to reply you. I was kinda in a busy situation.

I am very glad to see flutter_enhancement_suit has already support this package.

flutter_vector_icons have multiple iconsets including mdi icons. I think it might result in heavier bundle weight in production because all font files were declared in pubspec.yml. It is nice to have multiple iconsets together if you actually using them, but I will keep maintaining this little project for who only use mdi icons.

I am closing this issue as flutter_enhancement_suit support this.

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

3 participants