@Arantor> I'm not getting the error but I suspect it's in /do/notification..? I'm not visiting the page, I'm focusing on the popup first
:PI tried fixing it, please tell me if it works.
The popup width will be increased soon enough, once I move the item to the header (instead of the sidebar). It will no longer be limited by an arbitrary width.
Isn't the font size okay..?
@Dragooon> There's a bug with
@Dismal Shadow :whistle: Also, you can only blame yourself for the bug you mention... Honestly, I spent my entire morning on the notification code, and all I got out of it was a headache. It's interesting to split tasks into several tasks (Notifier, Notification and weNotifs), but really -- there's no obvious role to any of these. It looks like one is just there to hold pointers to notification objects, another is there for actual notification handling, and another has some helper functions. To me, it sounds like these three objects could be reduced to one. There's a lot of time wasted in calling getters and setters here, too.
Currently, I have "6 notifications", and clicking them shows me 4. But from time to time (and persistently), it'll show me all of them.
My goal, as I said, is to use the Notification.template.php template for the popup. It's what I'll be working on next. I'll be showing all items (read and unread), with a class to differentiate them.
Well, I'm pretty much emulating Facebook here, but in my own style, hopefully...