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 - Arantor
31
Features / Re: New revs
« on August 23rd, 2013, 06:02 PM »
9 files, 26KB[1]

Revision: 2226
Author: arantor
Date: 23 August 2013 17:01:57
Message:
! PM menu now elevated out of the main menu and put up with Notifications. There's quite a bit to this commit so let me cover it off.
 * When doing the regular poll for notifications, also supply the number of unread PMs too. We already have this number calculated, just have to send it. (Notifications.php)
 * Don't have the PM item in the main menu. I removed the m_pm declaration but the mail icons (including the one I don't think I've ever seen) are still in the sprite. (Subs.php, index.template.php, index.css)
 * Add the PM popup area next to the notifications area. (skeleton.xml, index.css, index.template.php)
 * The PM area itself handles the popup, both for the main popup as well as the preview. Previewing immediately marks something read and this must only touch is_read; is_new is used for a separate purpose in the bowels of the PM code. (PersonalMessage.php, PersonalMessage.template.php, PersonalMessage language file)
 * Lastly, the JS has to make it all play nicely. Sorry that it's basically a duplicate of the notifications code but there are a few subtle differences; firstly the PM count and notifications count are kept separate. The PM count is never used to update the page title, it doesn't have to do separate callbacks to mark any read, and there's no delete function directly from the popup (intentionally). This just felt right to me but I'd suggest we try it for real to see how people feel about it. There's also differences of URLs and the need for each popup to close the other. (script.js)
----
Modified : /trunk/Sources/Notifications.php
Modified : /trunk/Sources/PersonalMessage.php
Modified : /trunk/Sources/Subs.php
Modified : /trunk/Themes/default/PersonalMessage.template.php
Modified : /trunk/Themes/default/index.template.php
Modified : /trunk/Themes/default/languages/PersonalMessage.english.php
Modified : /trunk/Themes/default/scripts/script.js
Modified : /trunk/Themes/default/skins/index.css
Modified : /trunk/Themes/default/skins/skeleton.xml

@ I *really* didn't know how to make the JS any leaner than it is :(
 1. Eek.
32
Bug reports / Re: req_win bug
« on August 23rd, 2013, 04:31 PM »
I've wanted to use reqwin for popping up modal forms (e.g. registration, login) for ages but I realised all the event handling was screwed anyway.
33
Archived fixes / Re: DB column size cannnot be comma seperated
« on August 23rd, 2013, 04:26 PM »
Then use a string column.

ETA: As a general rule, only store numbers when you need them to be numbers because you're doing work on them in some fashion. For example counts of things, or numeric ids (because they are still a mathematical concept at that point)

When I built it for Crossing Overland, that's exactly what I did, stored them as varchars. There's no risk of loss of accuracy.
34
Archived fixes / Re: DB column size cannnot be comma sepeerated
« on August 23rd, 2013, 04:16 PM »
There's a reason that such things are excluded.

What are you doing with floats of that precision? Are you doing maths on them? Do you worry about loss of precision?
35
Features / Re: Miscellaneous/WIP screenshots
« on August 23rd, 2013, 03:39 PM »
And it's getting closer.
36
Features / Re: Miscellaneous/WIP screenshots
« on August 23rd, 2013, 05:22 AM »
Something like that, yes :D

Now I just have to make it actually display unread messages and also do the preview thing.
37
Features / Re: Miscellaneous/WIP screenshots
« on August 23rd, 2013, 04:03 AM »
Oh, and by the way, now that that screenshot covers all the navigation items on the menu, guess what! The 'My Messages' menu item is now GONE.

<insert obligatory "Aaaand it's GONE" Joker quote>
38
Features / Re: Miscellaneous/WIP screenshots
« on August 23rd, 2013, 03:37 AM »
So I have a bit more done on it, giving all the navigation that is needed. (first screenshot)

The only problem is that in other languages, the longer terminology makes it wrap - see second screenshot. Personally I'm not *that* bothered but I thought I'd outline it anyway just in case anyone was curious what it looked like in that situation (and might have other ideas)
39
Features / Re: Miscellaneous/WIP screenshots
« on August 23rd, 2013, 03:21 AM »
That's looking a bit better.
40
Features / Re: Miscellaneous/WIP screenshots
« on August 23rd, 2013, 02:11 AM »
I've been slacking with Wedge stuff for various reasons. But here's what I'm doing this evening. It's broken, sure, but it's what I'm doing anyway.
41
Off-topic / Re: Is Wordpress actually Most SEO CMS?
« on August 23rd, 2013, 01:44 AM »
Here's the thing... Google changes its algorithm regularly. Far too regularly, including reranking, for anyone to be able to understand what's actually going on. What worked 6 months ago won't work so well today, and probably might work again better in 6 months time.

Google uses a lot of things for ranking. One of the key things is to do with link 'reputation'. Inbound links to a page, outbound links from that page, and so on - a lot of inbound links to a page that aren't site-wide navigation are important, and are a sign that a page is probably more authoritative about its content. (Not the only sign but a sign, sure)

If a page is new, it doesn't have that - but if a page is old and doesn't have many inbound links, odds are it's not that authoritative either. In absence of any other relevant criteria, Google seems to be taking the newer page as slightly more authoritative for simple virtue of the fact that it's newer and presumably therefore not outdated contextually.
42
Features / Re: Getting rid of zebra classes
« on August 22nd, 2013, 11:55 PM »
It's still in the source here... And actually... maybe you can.

As I understand it, and I might be wrong... parent > child:nth-of-type(even) would work and not match children or nesting or suchlike. I dunno. I'm not a CSS guru.

Heck, I managed to break notifications just by adding:
Code: [Select]
#pms extends #notifs
to the CSS (and it actively broke everything related to notifications, though I would add there was a suitable element with that in it, it all looked right but the popup wouldn't pop up)
43
Features / Re: Getting rid of zebra classes
« on August 22nd, 2013, 11:41 PM »
I think the only solution is to drop postbg vs postbg2, and use postbg:nth-of-type(even) to style alternately, so it's done based on groups of posts rather than individual posts within a group...?
44
Features / Re: Getting rid of zebra classes
« on August 22nd, 2013, 11:08 PM »
Looks good, I like it like that.
45
Features / Re: New revs
« on August 22nd, 2013, 09:33 PM »
5 modified, 5KB

Revision: 2220
Author: arantor
Date: 22 August 2013 20:33:13
Message:
! Simplified like tooltips as we talked about. Saves a query too. (Display.php, Like.php, Msg.template.php, Thoughts.template.php, index language file)
----
Modified : /trunk/Sources/Display.php
Modified : /trunk/Sources/Like.php
Modified : /trunk/Themes/default/Msg.template.php
Modified : /trunk/Themes/default/Thoughts.template.php
Modified : /trunk/Themes/default/languages/index.english.php