[Plugin] Notifications system (1.0)
[Plugin] Re: Notifications system (1.0)
« Reply #60, on March 26th, 2013, 09:38 PM »
I think it's best to just have counts ,like you have and who, but not every time, maybe if some one dis likes the post, and can it be shown in the author/poster's profile, ( amount of likes only??) or is that too much too much?

being notified on a busy board could be annoying at times ( I'd think).

regards,
Maxx

Arantor

  • As powerful as possible, as complex as necessary.
  • Posts: 14,278
[Plugin] Re: Notifications system (1.0)
« Reply #61, on March 26th, 2013, 09:43 PM »
Quote
maybe if some one dis likes the post
Not an option.
Quote
and can it be shown in the author/poster's profile,
If you go to the profile > show posts area for a user, you will see their posts, if the posts received likes that will be shown there.

But I'm not talking about any of that. I'm talking about the notifications system that appears (currently) in the top of the sidebar, that is like XenForo's system.
When we unite against a common enemy that attacks our ethos, it nurtures group solidarity. Trolls are sensational, yes, but we keep everyone honest. | Game Memorial

[Plugin] Re: Notifications system (1.0)
« Reply #62, on March 26th, 2013, 09:51 PM »
The way it is now looks great! if they like would it show on drop down. or hit view all?
anyway it's cool the way it, and if you add on the likes that sounds cool!

was think of something else sorry!

regards,
Maxx

Dragooon

  • I can code! Really!
  • polygon.com has to be one of the best sites I've seen recently.
  • Posts: 1,841
[Plugin] Re: Notifications system (1.0)
« Reply #63, on March 26th, 2013, 10:01 PM »
Well sure, the plug in should work although I haven't extensively tested it. It should also show how multiple notifications are grouped together.
[Plugin] Re: Notifications system (1.0)
« Reply #64, on March 27th, 2013, 07:08 AM »
@Arantor: Yay your first notifier, a couple of points though
1) It can use multiple handling, something like Arantor and 4 others liked your post "Re:Like my post" which is pretty simple to implement (mostly Notifier::getText and Notifier::handleMultiple stuff). Currently it'll just stop the notification and bump it, you can add a count in data and increment it when handleMultiple is called and add some conditional checks on getText
2) It should be LikesPost since it'll currently only handle that

EDIT:
@Nao: Can you temporarily remove caching stuff from weNotif::get_quick_notifications on this site? I want to see if that fixes things.
The way it's meant to be

MultiformeIngegno

  • Posts: 1,337
[Plugin] Re: Notifications system (1.0)
« Reply #66, on March 27th, 2013, 12:40 PM »
Uhm, this is strange. I tried to notify Arantor, so I typed @ Aran (of course without the space) but Chrome throwed the popups attached, every time with a different number.

To reproduce: type @ Aran (without space) then close the first popup, press ESC and a new popup with a different number should appear, close it and press ESC again ...

 309_wedge.jpg - 47.96 kB, 873x494, viewed 125 times.

 315_wedge.jpg - 44.47 kB, 873x494, viewed 124 times.


Arantor

  • As powerful as possible, as complex as necessary.
  • Posts: 14,278
[Plugin] Re: Notifications system (1.0)
« Reply #67, on March 27th, 2013, 01:57 PM »
Quote from Dragooon on March 27th, 2013, 07:08 AM
@Arantor: Yay your first notifier, a couple of points though
1) It can use multiple handling, something like Arantor and 4 others liked your post "Re:Like my post" which is pretty simple to implement (mostly Notifier::getText and Notifier::handleMultiple stuff). Currently it'll just stop the notification and bump it, you can add a count in data and increment it when handleMultiple is called and add some conditional checks on getText
2) It should be LikesPost since it'll currently only handle that
It was a quick thing I did late last night ;) I did skim over the handleMultiple but I wasn't quite sure what I was supposed to do with it.

As far as the name goes, I did envision it supporting likes on thoughts too but I ran out of time last night to work on it.

Dragooon

  • I can code! Really!
  • polygon.com has to be one of the best sites I've seen recently.
  • Posts: 1,841
[Plugin] Re: Notifications system (1.0)
« Reply #68, on March 27th, 2013, 06:04 PM »
Quote from MultiformeIngegno on March 27th, 2013, 12:40 PM
Uhm, this is strange. I tried to notify Arantor, so I typed @ Aran (of course without the space) but Chrome throwed the popups attached, every time with a different number.

   To reproduce: type @ Aran (without space) then close the first popup, press ESC and a new popup with a different number should appear, close it and press ESC again ...
You can freely use @Aran, see it won't do anything. BTW @Nao, you forgot to remove debugging code :P

MultiformeIngegno

  • Posts: 1,337
[Plugin] Re: Notifications system (1.0)
« Reply #69, on March 27th, 2013, 06:06 PM »
No, I explained myself badly. When I try to type @Arantor, as long as I arrive at typing @Aran, those popups with the numbers appear.

EDIT: oh so they was for debug?

Dragooon

  • I can code! Really!
  • polygon.com has to be one of the best sites I've seen recently.
  • Posts: 1,841
[Plugin] Re: Notifications system (1.0)
« Reply #70, on March 27th, 2013, 06:12 PM »
Quote from MultiformeIngegno on March 27th, 2013, 06:06 PM
No, I explained myself badly. When I try to type @Arantor, as long as I arrive at typing @Aran, those popups with the numbers appear.

   EDIT: oh so they was for debug?
Yeah I got what you meant, what I meant was you didn't need to add a space after @ :P.

live627

  • Should five per cent appear too small / Be thankful I don't take it all / 'Cause I'm the taxman, yeah I'm the taxman
  • Posts: 1,667
[Plugin] Re: Notifications system (1.0)
« Reply #71, on March 27th, 2013, 11:32 PM »
Quote from MultiformeIngegno on March 27th, 2013, 12:40 PM
Uhm, this is strange. I tried to notify Arantor, so I typed @ Aran (of course without the space) but Chrome throwed the popups attached, every time with a different number.

To reproduce: type @ Aran (without space) then close the first popup, press ESC and a new popup with a different number should appear, close it and press ESC again ...
Quote from Dragooon on March 27th, 2013, 06:12 PM
Yeah I got what you meant, what I meant was you didn't need to add a space after @ :P.
Is someone confused? ::P
A confident man keeps quiet.whereas a frightened man keeps talking, hiding his fear.

Arantor

  • As powerful as possible, as complex as necessary.
  • Posts: 14,278

MultiformeIngegno

  • Posts: 1,337
[Plugin] Re: Notifications system (1.0)
« Reply #73, on March 28th, 2013, 12:13 AM »
Should I write again what I'm trying to say or is it clear enough? :P
Was someone able to reproduce that?

Arantor

  • As powerful as possible, as complex as necessary.
  • Posts: 14,278
[Plugin] Re: Notifications system (1.0)
« Reply #74, on March 28th, 2013, 12:14 AM »
Sure it was... it's debugging information in trying to figure out where the popup should be displayed.