Show Posts

This section allows you to view all posts made by this member. Note that you can only see posts made in areas you currently have access to.

Messages - Dragooon
166
Plugins / [Plugin] Re: Notifications system (1.0)
« on March 22nd, 2013, 03:52 AM »
Currently that's intended, I couldn't position it next to the cursor.
167
Features: Posts & Topics / Re: Automatic Quote splitter
« on March 21st, 2013, 11:56 AM »
@Nao: Perhaps an option?

EDIT: Okay, I can't mention people in this board or are my plugins really crappy?
168
Features: Posts & Topics / Re: Automatic Quote splitter
« on March 21st, 2013, 11:10 AM »
Thinking about it, is it possible to trigger it directly on enter? On mobile devices it doesn't work since I can't do a shift or control.
169
Off-topic / Re: HAPPY BIRTHDAY NAO!
« on March 21st, 2013, 11:07 AM »
Another happy birthday to @Nao!
170
Plugins / [Plugin] Re: Notifications system (1.0)
« on March 21st, 2013, 11:07 AM »
Quote
I'm not sure it works like this... It seems to me that as soon as you click the popup, it gets marked as read, even if not visiting the topic itself. I may be wrong, but this is what I gathered yesterday... And I haven't looked at the notification code too deeply for now.
That should not happen, some systems do this but I don't intend that.
Quote
Oh, I love the popup that comes up when I mention someone... It will need some refining, though! For instance, sometimes it doesn't close... (Like now. It's still opened.) Also, I tried locally to position the popup next to the name I entered, and while it works, which is great, it adds another gzipped kilobyte of JS, so I'll try to make it shorter or something, or maybe it's worth it I don't know...
I wanted it positioned next to the cursor but couldn't figure it out, so thanks! I'd like to see how you did it even if you don't commit it.
171
Plugins / [Plugin] Re: Notifications system (1.0)
« on March 21st, 2013, 10:19 AM »
Quote
We should also make sure that all unread notifs are sent by e-mail before they're pruned... Is it accounted for?
Unread notifications aren't pruned, only read ones. By default no emails are sent, you can change that in the profile area. The email area is perhaps one of the most unfinished part, it needs refinement and perhaps use of email templates.
Quote
More importantly -- I've (temporarily..?) disabled auto-markread when clicking the notification, so that I can actually have time to style the popup at all... :whistle:
Did you disable mark read on topic display (See Notification::markReadForNotifier, it handles all auto mark reads for areas other than the notifications action)? Because otherwise one way or the another it'll end up mark it read on click.
172
Off-topic / Re: HAPPY BIRTHDAY NAO!
« on March 21st, 2013, 06:04 AM »
Happy birthday!!
173
Plugins / [Plugin] Re: Notifications system (1.0)
« on March 19th, 2013, 10:16 AM »
Quote from Nao on March 19th, 2013, 09:58 AM
- Hmm, are you sure you can do it? Doesn't that bother you?
Not sure why it would bother me, I can definitely do it :P.
Quote from Nao on March 19th, 2013, 09:58 AM
- @if member is a trick I only recently added, you'd have surprised me if you'd implemented it that way, actually ;)

- I can do that, if you want... CSS/JS is my area of expertise these days, after all :P
Go ahead, it's a part of Wedge anyway
Quote from Nao on March 19th, 2013, 09:58 AM
Also... Where the heck is the admin page for setting up notifications...? I couldn't find it. The language files mention "Admin > Plugins > Notifications", so I'm trying to update them to the correct position...
There isn't one :P, there is only one setting for setting the number of days before which read notifications are pruned and that doesn't have an UI yet.
Quote from Nao on March 19th, 2013, 09:58 AM
Ah, and the notification popup is incorrectly positioned in my local install. I was wondering, could you test on your side with the latest revision? I'm wondering if it's a browser issue or a bug I introduced when optimizing your JS code... (I did some pretty big changes and didn't know *where* to test, so I wouldn't be surprised.)
I can have a look in the evening.
Quote from Nao on March 19th, 2013, 09:58 AM
I mixed things up in my head. Which reminds me: notifications.js was a stand-alone file because it was a plugin to begin with... Now that it's a core feature, we should be integrating it into script.js, all right? And add the infamous @if member to it, of course ;)
Yeah sure. The patch file was a quick job I did in the evening, it definitely isn't finished yet.
Posted: March 19th, 2013, 10:11 AM

Okay, quickly looking over the popup it seems to be correctly positioned in the latest revision.
174
Other software / Re: My review of customer service on SMF
« on March 18th, 2013, 10:01 PM »
Quote from emanuele on March 18th, 2013, 09:57 PM
SMF is a piece of software, it can treat anyone nor anything like shit. :P
can't*[1]
 1. Dragooon out
175
Plugins / [Plugin] Re: Notifications system (1.0)
« on March 18th, 2013, 06:05 PM »
Quote from Nao on March 18th, 2013, 05:59 PM
I'd really appreciate if Dragooon could rework these columns into data variables. If it's not too much to ask for.
Yeah sure, but you'll have to wait till weekend until my exams are through.
Quote from Nao on March 18th, 2013, 05:59 PM
Also, I'm looking at the CSS, and: (1) anything that's member-only should be within '@if member' blocks. This saves the extra load for guests... (2) What exactly is that ":parent" pseudo-class..? If you actually mean "select this selector's parent", this isn't defined anywhere in CSS, even in CSS4. (I should know, I read through their entire pseudo-class list for CSS4 no later than yesterday for some other reason.) At worst, you can use JavaScript for that, but...
1) Ah okay
2) Ugh...I always thought CSS has a :parent selector, I'll fix that as well.
176
Plugins / [Plugin] Re: Notifications system (1.0)
« on March 17th, 2013, 05:01 PM »
Quote from Nao on March 17th, 2013, 04:52 PM
- Does the mod really, really need so many new columns..? Technically, all new member data should go to the data column. If you need an explanation of how it works (really, it's just an array that's serialized and unserialized as needed), you can ask me.
notfy_email_last_sent and notify_email_period definitely need to be columns since they are searched for scheduled tasks, unread_notifications are also used (I forgot to define Keys, I'm not sure how that'll work for these). I guess disabled_notifiers, notifiers_prefs and email_notifiers can be consolidated into data since the latter two are json_encoded arrays anyway.
177
Features: Posts & Topics / Re: Automatic Quote splitter
« on March 17th, 2013, 03:24 PM »
Quote from Arantor on March 17th, 2013, 03:23 PM
Quote
shortcuts: hit Shift + Enter to use smart tag closer, Ctrl + B (bold), Ctrl + I (italics), Ctrl + U (underline), Alt + D (save draft), Alt + P (preview) or Alt + S to submit.
Full reply is a wonderful thing :P

Maybe we need this in the quick reply too.
You actually think I read all that? :P
178
Features: Posts & Topics / Re: Automatic Quote splitter
« on March 17th, 2013, 03:22 PM »
Quote from Arantor on March 17th, 2013, 03:20 PM
It's
Oh
Quote from Arantor on March 17th, 2013, 03:20 PM
shift-enter.
okay

I'm stupid :P
179
Features: Posts & Topics / Re: Automatic Quote splitter
« on March 17th, 2013, 03:18 PM »
This is still supposed to work right? I can't get my quotes to automatically split on enter.
180
Other software / Re: Fuck the SMF project. Fuck it up its stupid ass.
« on March 17th, 2013, 03:01 PM »
Quote from Yoshi2889 on March 17th, 2013, 02:59 PM
Does my post have an Android icon because I posted it from my phone? That's cool.
Yeah