Plugin: add a menu icon (next to search)

Plugin
1

If you’ve always wanted to add an extra icon, next to the search, here’s your chance! :slight_smile:

screenshot.png1155×570 56.8 KB

Installation as usual, link:
https://github.com/VonLion/discourse-menu-icon.git

15 Likes
4

This is great ! Thanks!

If someone has a site using this, please tell me.

I’m translating this. :+1:

:sparkles: An idea @Bas, maybe besides “title”, you can add alt title to image too?
Same setting for menu_icon_title, I think it’s here: initializers/menu-icon.js.es6#L16

4 Likes
5

Hey @SidV,

Just merged the pull requests, thanks!

Alt text isn’t necessary I think; it’s a font icon and the link already has an aria-label which is enough for screenreaders :slight_smile:

1 Like
6

You’re right, I was confused!
Sorry for the idea, alt would not serve here. I agree with you.

1 Like
7

Excellent :slight_smile:

Feature Request: Multiple buttons!

1 Like
8

Yeah, figured it wouldn’t take long before that was requested, haha.

Thing is, I don’t know how to do this :smiley: The current plugin is a pretty straightforward modification of the excellent plugin tutorial by @eviltrout

I don’t know how to create an “add icon” button that spawns multiple fields; I know it can be done, since the navigation plugin does so, but I’m pretty new at this and I just created this one out of a personal need and to learn.

I might take a crack at this during a weekend somewhere, until then… PRs are welcomed :slight_smile:

4 Likes
9

For multiple buttons are some ideas here: Add new link on header beside search icon by @techAPJ :clap:

2 Likes
10

Heh, that thread is actually the other part of the code I stole :sushing_face:

11

Thanks for this plugin. Unfortunately, it doesn’t seem to work under 2.2.0.beta7. Despite enabling, no icon appears.

12

I don’t think a plugin should be necessary for this in our brave new world of theming…

1 Like
13

:+1: we have a theme component that does this now @allu. have a look here

@Bas Thanks for creating this plugin! Since the component essentially does the same thing that this plugin is supposed to do, I’m moving this plugin to the #plugin:broken-plugin category.

7 Likes