Skip to content
This repository has been archived by the owner on Oct 12, 2023. It is now read-only.

Feature: Support outline icons #9

Closed
abennouna opened this issue Dec 29, 2016 · 10 comments
Closed

Feature: Support outline icons #9

abennouna opened this issue Dec 29, 2016 · 10 comments

Comments

@abennouna
Copy link
Contributor

Do you think it would be easy to add support for outline stroke icons? It would be a nice to have for iOS.

@GerritErpenstein
Copy link
Owner

Sounds like a reasonable feature. Let's see if I come up with a good idea...

@GerritErpenstein
Copy link
Owner

If all is going well, I will release a new version with the requested feature in the next few days.

@GerritErpenstein
Copy link
Owner

See https://github.com/GerritErpenstein/ionic2-custom-icons-example/tree/feature/platform-specific for an example of the (yet unstable) release of the feature.
Please let me know if you have any suggestions. The docs haven't been updated yet but I think it's pretty self-explaining how it works.

@mavogel
Copy link

mavogel commented Feb 24, 2017

@GerritErpenstein thx for this nice feature. Added it to my app and it works great (for tabs as well). It's nice with the (ion|md) suffix. At first I expected (ion|md) as prefix, like ionic internally does, but it's better your way, cuz then the icon are also sorted in my folder on the disc.

Took me a while to find out you changed the path config/sass.config.js

from: 'node_modules/ionic2-custom-icons/directive/scss/',
to: 'node_modules/ionic2-custom-icons/ionic/scss/',

but as you said: docs weren't updated yet from 0.3.x to 0.4.x.

One suggestion would be to be able to scale the custom icons of the tabs. They are really tiny, especially if I remove the text below the icon. Or is this possible with ionic2 out of the box?

@GerritErpenstein
Copy link
Owner

Hi @mavogel! Thanks for the kind feedback. I am still working on this feature. The next (beta) release will support inactive ("outline" for iOS) icons. After finishing that, I will update and rewrite the docs. Until then the feature branch in the ionic2-custom-icons-example contains a working example with the new features.

One suggestion would be to be able to scale the custom icons of the tabs. They are really tiny, especially if I remove the text below the icon. Or is this possible with ionic2 out of the box?

I will look into it. Ideally the size should be as close to the size of the original Ionic icon set as possible.

@GerritErpenstein
Copy link
Owner

GerritErpenstein commented Mar 12, 2017

I have just pushed a commit that should make the next release feature complete. Inactive ("outline") icons are fully supported now. The example app (in the platform-specific branch) has also been updated.
The next step is to update the docs and make a stable release.

Feel free to test the update and please let me know if you find any problems. Thanks!

@mavogel
Copy link

mavogel commented Mar 15, 2017

@GerritErpenstein thank for th update. Outline icons work fine for me now.
For me the tab icons are still really small for iOS. (or maybe it's meant to be this small by ios conventions. I'm new to app dev...)

screen shot 2017-03-15 at 5 33 19 pm

It tried it with your platform-logo. I also saw that it's possible to pass a small parameter to the <custom-icon> and probably something like large. Is this also possible in a way for tabs?

@GerritErpenstein
Copy link
Owner

@mavogel
The small icons issue for tabs should be fixed in ionic2-custom-icons 0.4.1-beta.3. Thanks for your feedback!

@sarahsga
Copy link

sarahsga commented May 9, 2017

@GerritErpenstein my custom outline icons still appear with the fill, I tried the heart icon, the outline works for that, but for some icons, it does not. Try the svg I've attached (inside the .zip because .svg was not allowed)
devices.svg.zip

@GerritErpenstein
Copy link
Owner

GerritErpenstein commented May 13, 2017

@sarahsga I have taken a look into your SVG. Please try to remove all SVG transforms. They have caused problems in the past.
Attached you can find an edited version of your SVG that works with this library.
devices_edited.zip

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants