Xen Notices [Deleted]
- Thread starter Chris D
- Start date
R
ragtek
Guest
You're extending your controller via xfcp but you're not using the right classname for the parent class
should be
PHP:
public static function extendAccountController($class, array &$extend)
{
if ($class == 'XenForo_ControllerPublic_Account')
{
$extend[] = 'GritterNotifications_ControllerPublic_Account';
}
}
PHP:
class GritterNotifications_ControllerPublic_Account extends XenForo_ControllerPublic_Accountshould be
PHP:
class GritterNotifications_ControllerPublic_Account extends XFCP_GritterNotifications_ControllerPublic_Account
I know, thus the Homer SimpsonYou're extending your controller via xfcp but you're not using the right classname for the parent class
Thank you though (again!)
I love them, Shelley.
HenrikHansen
Well-known member
I have just tried to insert a youtube video in a notification. Looks cool 
.
However the notification containter cannot expand to full wide. Possible it can expand if content is wider?
.However the notification containter cannot expand to full wide. Possible it can expand if content is wider?
Shelley
Well-known member
I have just tried to insert a youtube video in a notification. Looks cool
However the notification containter cannot expand to full wide. Possible it can expand if content is wider?
Can't you just define a width? And extend the width in the extra.css this is what I have done for the fixed image notifications I'm sure it can be applied to the container.
R
ragtek
Guest
another suggestion:
IMO it's better to use
instead of
because:
is searching for <body and replacing it with <body class="foo" ?
Then your search string won't be found
EDIT: lol, i've miread the code, i thought it's FOR THE ACP area
change admin with template and then it should work but i think already you know what i mean^^
IMO it's better to use
PHP:
if ($templateName == 'PAGE_CONTAINER' && $template instanceof XenForo_Template_Admin) {
PHP:
public static function templatePostRender($templateName, &$content, array &$containerData, XenForo_Template_Abstract $template)
{
if ($templateName == 'PAGE_CONTAINER')
{
// <body data-controller=" only appears in the admin version of PAGE_CONTAINER.
// Without the following check, editing a notification that contains {name} causes
// a replacement. This ensures this replacement only happens outside of the Admin CP.
if (!preg_match('/<body data-controller="/', $content))
{- performance => you don't need to run preg_match with the whole output code (because all admin templates will be an instance of XenForo_Template_Admin it's fine to check only for this)
- you can't be sure, that the search string ( <body data-controller= ) is available.
is searching for <body and replacing it with <body class="foo" ?Then your search string won't be found
EDIT: lol, i've miread the code, i thought it's FOR THE ACP area
change admin with template and then it should work but i think already you know what i mean^^
yavuz
Well-known member
If you configure the width of your YouTube video in the embed settings appropriately, you can actually get away with the default notification width:I have just tried to insert a youtube video in a notification. Looks cool
However the notification containter cannot expand to full wide. Possible it can expand if content is wider?
Here's the YouTube settings - so aim for a width of about 275px:
It's ok, I got what you were saying.
I've gone with:
PHP:
public static function templatePostRender($templateName, &$content, array &$containerData, XenForo_Template_Abstract $template)
{
if ($templateName == 'PAGE_CONTAINER' && $template instanceof XenForo_Template_Public)I worry about the quality of the rest of my code now
Anthony Parsons
Well-known member
So is this being upgraded to not clash with Syndols mods? Asking, as I use his mods...
Yep, very soon actually.
HenrikHansen
Well-known member
Cool, thanks. Notifications can be used to a lot of different things. Advertising, tips & tricks, have you seen... pop ups, videos, sponsor blocks etc.
jadmperry
Well-known member
Purchased, Installed, +1 (actually +1000)...Here is what is would ideally have/want/need....notifications based on visiting certain pages (actually, what I really want is if someone visits my profile, then they get a notification).
Is this possible, if so, how do I do it?!?!?!?
Is this possible, if so, how do I do it?!?!?!?
This - I'm sorry to say - isn't possible.
The criteria capabilities are the same as Notices, and this isn't one of the possible ways to target a Notice so by proxy you also can't with Notifications.
You can target the XenForo_ControllerPublic_Member :: Member action, but this will show the notification on every member profile, not specific ones.
Yorick updated Notifications - powered by Gritter with a new update entry:
Bugs squashed, new colour picker (default XenForo), audible notifications and Admin CP preview
Read the rest of this update entry...
Bugs squashed, new colour picker (default XenForo), audible notifications and Admin CP preview
Read the rest of this update entry...
