Discourse Icon

:discourse2: Summary Discourse Icon allows you to use icons from your SVG icon subset in a post.
:hammer_and_wrench: Repository Link https://github.com/discourse/discourse-icon
:open_book: New to Discourse Themes? Beginner’s guide to using Discourse Themes

Install this theme component

Usage:

[wrap=icon id=square][/wrap]

Example:
This is the icon and the icon.

This is the [wrap=icon id=far-square][/wrap] icon and the [wrap=icon id=pencil-alt][/wrap] icon.

Note: if the icon doesn’t show it’s because it’s not in your subset. If you are admin you can add it in your site settings: svg icon subset.

31 Likes

Awesome! This is iconic!

(meta has many many themes, but luckily thanks to @kris.kotlarek we have an add to all themes button… got to sort out progress there, cause it takes a while to do its magic)

13 Likes

Yes didnt want to add to all for now, as I just finished coding it :grin: but will do now

Edit: oh already done

10 Likes

Thanks, that’s super fun! :sunglasses:

I never got into BB script tags, nor have I hand-coded FontAwesome, so the practice of using an empty tag pair has always seemed strange to me. Is that by popular convention, or is there a webcraft reason for using tag pairs? Also, can they have stuff between them, does that affect the text?

3 Likes

There are multiple causes here.

First when I wrote the wrap BBCode feature, @sam explicitly asked to me support 3 formats:

[wrap=foo][/wrap]
[wrap=foo]bar[/wrap]
[wrap=foo]
bar
[/wrap] 

This is done to be consistent with our existing BBCode usage in Discourse.

I could have a better syntax as: [icon=times] but then it would have to be a plugin and not a theme component.

As an experimentation, I think it’s fine as it is today, and if people were to use it a lot, we could think of having a plugin do this with a more concise syntax.

Another idea I have had is to add an API to add autocompletes symbol in the composer just like we have fore users/categories/emojis… But I’m afraid this would be a nightmare with multiple components choosing their own symbol…

5 Likes

Oh! I need to add support for another bbcode and those) thought that I’d use this as a starting point, but you’re saying that the wrap stuff is handled in core, which explains why I can’t find how it’s handled by your theme component.

My current solution is to use the linked words component and an ugly regex, but I’m still struggling to add the necessary classes to the links.

3 Likes

Yes read this topic to know more:

4 Likes

Right I’ve looked at it a dozen times. And printed out the code for your component and stared at it for hours, trying to understand if the degree to which it’s javascript or ember that I don’t understand. I can’t add a [foo] tag without a plugin? Or can I some how replace wrap in your code with foo to cover my new bbcode?

1 Like

No you can’t. Because part of it is server side. If you want your own custom BBCode you will have to write a plugin and look at how wrap is implemented for example:

https://github.com/discourse/discourse/blob/master/app/assets/javascripts/pretty-text/engines/discourse-markdown/d-wrap.js.es6

5 Likes

Thanks! It’s starting to make sense. I’ve not had this much trouble learning a new language since C++ and objective Pascal in 1990.

My current regex hack almost works. I’ll need to decide whether to go the plugin route or stick with my hack (and figure out how to add the classes).

At least now I understand why I can’t find that wrap in the theme component!

3 Likes

I can think of a few possible syntax layouts to add to the wrap plugin for this:

[wrap=foo /]
[wrap/=foo]

(mimicing XML’s self-closing tags) but those are both fairly ugly :confused:

Anyone got amazing ideas for something that looks okay?

3 Likes

yeh not a big fan of these :confused:

6 Likes

I’ve found a big bug…

If you use an icon wrap and then highlight quote it the resulting image is wildly embiggened:

  • Use an icon wrap in a sentence in a post. eg This is an icon [wrap=icon id=far-check-square][/wrap]
  • Highlight the relevant text and ‘Quote’
  • Icon goes super big in quoted excerpt

This is what the syntax looks like when highlight quoted:

[quote="JammyDodger, post:1, topic:294, full:true"]
This is an icon ![](/svg-sprite/grumpy.jammydodger.monster/icon/222222/far-check-square.svg)
[/quote]

This is a reply where it gets massive

Though quoting it using the speech bubble in the format bar works fine.

4 Likes

How are you able to highlight quote the icon? On my test site, the icon is skipped.

image

I am not sure? I just use the mouse.

image

What browser are you using?

Using the mouse, I tested with Chrome and Firefox, and I could not select the icon. :thinking:


EDIT:

If you wrap an icon unavailable in the current subset, it will create an SVG image instead.
Then, you can quote it.

1 Like

Can you provide an example? Your screenshot shows a ticked checkbox and I have no issue selecting it on my test instance (as well as other icons probably not used by Discourse like x-ray.

1 Like

I see the issue. The component searches by looking at whether there is a wrapper with d-wrap class; but there is no wrapper if the provided icon is not from the subset. It creates an image as SVG instead.

For example:

SVG, icon within the subset

Image as SVG, icon not within the subset

If you quote the images, the component does nothing, as there is no wrapper (and the image is converted to markdown resulting in an image without any class)

Does this mean I need to have an svg_icons field in settings.yml available in my theme? Or could I use any svg_icons field available from any enabled theme-components too? And if that is the case would adding this field to this theme-component make sense?

svg_icons:
  default: ''
  type: 'list'
  list_type: 'compact'

Because in my experience, the SVG icons are transformed to <img /> tags, and they do not inherit the currentColor when the color scheme is changed:

Please compare the Example paragraph: