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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Some ideas #1

Open
7studio opened this issue Jun 12, 2017 · 1 comment
Open

Some ideas #1

7studio opened this issue Jun 12, 2017 · 1 comment
Assignees

Comments

@7studio
Copy link

7studio commented Jun 12, 2017

Hi,

Because your idea is brilliant, THANK YOU 馃槝

But it's a bit annoying that the plugin (through its only filter acf_svg_icon_filepath) doesn't allow us to specify an SVG file for a particular ACF field (by {$field_key} or {$field_name}) 馃槙
I don't know if it was a technical restriction or a personal choice (no need at first) but IMHO it could be very useful to be able to do it 馃槉

Second point a bit more specific 馃槈 It could be very useful to be able to change/translate the texts associated to the SVG icons. Most of the time, the SVG id names are a personal fight for the developper 馃槖 and don't represent exactly what they are. Because clients can manage/choose their icons in some cases, it's really important to give them a beautiful transcript.

For the fun, it could be interesting to improve the way the icons are used into Select2. Instead of using a symbol from in-document sprite, we could use the symbol directly from the external sprite (e.g.: <use xlink:href="/path/to/icons.svg#newspaper"></use>) :simple_smile:

If you continue to use this plugin and are interested in these changes, I will be happy to submit a PR :bowtie:

Best regards,
Xavier.

@MaximeCulea
Copy link
Contributor

Hi @7studio,

This plugin has been crafted once for a client, more like a snapshot, but knowing it will needs improvements.

We already thought about what you are speaking about, and you're totally right, it's a better implementation !
As you offer yourself for a PR, please let's go ahead ! 馃檶

So thank you for your interest and contribution. 馃挭
Waiting forward to read your PR.

@7studio 7studio mentioned this issue Jun 19, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants