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

Top bar notification #20

Closed
lonniebrodie opened this issue Jan 27, 2016 · 6 comments
Closed

Top bar notification #20

lonniebrodie opened this issue Jan 27, 2016 · 6 comments

Comments

@lonniebrodie
Copy link

No indication of message received when multiple recipients is utilized. It also does not indicate that the message is new (different background color). It works fine (i.e. number indictor) when single recipient is used.

Elgg 2.01

@daveonche
Copy link

How do I modify the Topbar Menu Icon in my own plugin so as not to mess with the plugins core files?? I tried to unregister the Hookhandler in my plugin start.php but the use of $this->hooks in my start.php returns fatal error. Would it be possible to override the setupTopbarMenu hookhandler with my plugin or I will just have to apply the modificaation directly from the plugin?? I want to use $text = "<i class="fa fa-envelope fa-lg">" and also 'priority' => 100, 'section' => 'alt', in my modification. Thank you.

@daveonche
Copy link

I mean $text = "< i class="fa fa-envelope fa-lg">"

@hypeJunction
Copy link
Owner

You can register a new hook for 'register','menu:topbar' and filter the menu items as you please.
You can unregister the hook by rewriting callback to hypeInbox()->hook->setupTopbarMenu()

@daveonche
Copy link

Thanks! I was able to unregister it using elgg_unregister_plugin_hook_handler('register', 'menu:topbar', '\hypeJunction\Inbox\HookHandlers::setupTopbarMenu'); Now I have registered a new hook and trying to filter the menu items but am still getting fatal error for using $count = $this->model->countUnreadMessages (); in my plugin's start.php

@hypeJunction
Copy link
Owner

use hypeInbox()->model->countUnreadMessages()

@daveonche
Copy link

Cool!! Its perfect now.............Thanks a million.

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