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 - Nao
7471
Features / Re: Template skeleton!
« on October 10th, 2011, 05:47 PM »
Quote from Arantor on October 10th, 2011, 05:32 PM
Quote
So... Who adds it to their to-do?
Well, refitting the API is on my todo list for the murky future... but for now, neither of us. There are bigger fish to fry - though it's certainly relevant in the integration of Aeva for attachments.
I'm still so wary of Aeva's bundling...
I mean, today I was thinking again about the comment system for AeMe and wondering whether we should beef it up, make it smaller (in UI at least), or simply integrate it into wedge_messages... Then I was wondering, is it necessary to implement floating topics before we can use wedge_messages to store posts that don't belong in a topic...? And then I thought, do we really need to put all 'floating topics' (i.e. topics, profile comments, media comments...) inside the topic table? Can't we just deal with these in the messages table, have a special 'wedge' field in wedge_messages that allows us to store a message origin, whether a topic, profile or media, I don't know, when I start thinking like that, I start to lose myself in thought...

Last week I was spending an afternoon with a friend who needed my help to configure .htaccess and .htpasswd features on her portfolio website. We spent about 3 hours on it, and in the end she commented to Milady, "I don't know how he does it. With all the music, and with his focus switching from one thing to another every other second..."
But, well, I did it :P And yes I'm probably a bit ADD.

Which brings me back to, ah, these annoying floating topics... I think I should just rename this to 'floating messages' and give up on the idea of storing the type in the topic table, and rather have it in the message table, with a proper index.
(I give you 5 minutes before you tell me it's the worst idea I had this week :P)
Quote
Except for the small detail that if it's not part of the object, it won't have access to the object's variables. Does it need access to the block list for any reason, for example?
Nope, it doesn't... Which is why I didn't push it into the object to begin with. It already has the block's name at this point, so it simply needs to call template_blockname().
Quote
Quote
But in the case of default, the list is never empty, is it..? It has the 'main' block by default in it...
But you're still adding to it, right?
Nope... load('block', 'default') will replace the 'main' block. load('block', 'sidebar') will add to the current blocks.
I think we already went through that... :P
7472
Features / Re: Template skeleton!
« on October 10th, 2011, 05:19 PM »
Quote from Arantor on October 10th, 2011, 03:02 PM
However you slice it, for something whose purpose is solely to authenticate the user, then serve a file, we should be doing absolutely as little as possible.
So... Who adds it to their to-do? :P
Quote
I have been wondering about a System class that handles stuff like loading sources and templates, which would give:
wesys::loadTemplate()
wesys::loadSource()
wesys is still a cool name... ;)
It's better than 'wetem', but it doesn't indicate what it's for.

I'm wondering whether execBlock() should be in wetem.
I tried moving it to a private method, but then I remembered that Load.php and other files will use it to run specific functions from template files, such as template_init(), so I made it public, but then it doesn't have much of a point being in the object... Except for being called through self::exec in the render method.
I try having as little data as possible in the object that is not directly related to outputting non-template data. I've been wondering about while_we_re_here(), for example...[1]

I've already sorted out the functions to push everything public to the beginning of the object, and everything private to the end. Seems cleaner to me that way, although on the other hand it also separates some functions from their twins, like render() is no longer followed by render_recursive(). I don't think it's that big a problem, as I'm not going to keep developing wetem forever... After a while, it'll be solid enough that no one will even bother to look at the code.

Oh, and I also turned the static variables into private, and the wetem class to final. I don't see any reason to inherit it... Might as well protect it entirely. If in the future a use is found in inheriting from wetem, then we can always remove the final keyword.
Quote
How is it different? You're still adding to a list, except the list was totally empty beforehand.
But in the case of default, the list is never empty, is it..? It has the 'main' block by default in it...
 1. For the curious, I name that function while listening to a certain Pink Floyd album... :P
7473
The Pub / [Archive] Re: Logo Madness
« on October 10th, 2011, 05:04 PM »
Quote from Arantor on October 10th, 2011, 02:57 PM
The real difference is whether the story is told through the music or not. RHPS, WWRY and Wicked, plus Sweeney Todd do. Enchanted sort of does, but not fully - it's more how they express themselves and their feelings, and more background stuff.
That's pretty much how I view it, too ;)

The South Park movie, for instance, is a musical because the lyrics make the story move forward.

Re: SVG, do you want a SVG version of the latest logos...?
7474
The Pub / Re: Spell checker
« on October 10th, 2011, 04:13 PM »
You can change your vote now.
7475
The Pub / Re: Spell checker
« on October 10th, 2011, 03:01 PM »
It's just that if a feature is vital to a minority, and it doesn't bother anyone else because it's disabled by default, it shouldn't be removed for the sake of removing it.
However, given that all participants to this topic said they weren't using it, I suspect we'll have a large majority in favor of removing it.
7476
The Pub / Re: PM popup
« on October 10th, 2011, 03:00 PM »
You aren't banned from anything (?), and you have demo access if you want...
7477
Features / Re: Template skeleton!
« on October 10th, 2011, 02:58 PM »
Quote from Arantor on October 10th, 2011, 01:45 PM
Oh, my bad. I was just trying to go through the circumstances in which you might want to but there aren't really any.
So, you agree we should be doing even less work for dlattach at this point...?
Quote
Works for me. I'm not in the middle of anything that will be broken if I sync, so I can readily wait until you're done before updating.
Well turns out I didn't leave it broken for long (a few minutes). I just split the stuff so I didn't have to mix 'proper changes' with 'site-wide search & replace changes' in the same commit. That way, you can check out my changes while skipping a specific commit so you don't have to go through a long list of boring changes...

I'll do the same if/when I rename the load() method.

Oh, BTW, I've found out why I find it a bit ugly...

loadLanguage()
loadTemplate()
loadBlock()

We have a lot of these together. Now it's like:

loadLanguage()
loadTemplate()
wetem::load()

Meh... It's like I broke the pattern :P
Quote
Quote
In the meantime, I was thinking, maybe we shouldn't call the block loading code 'load' but use 'add' instead... Because that's precisely what it does. Except for the default layer, I know...
Well, technically it is still adding, even if there wasn't anything to add to before. It's essentially the same as saying x += y but dealing with the case of empty(x) first... no?
Hmm no...
I'll think about all of that when I have a moment to resume my work.
In the meantime, feel free to look at my object code and improve it (i.e. anything related to object optimization, not template manipulation, which I'll deal with later to merge the block and layer loading code.)
Quote
To me, calling it 'add' would make more sense.
It definitely makes more sense. Except for the default layer but shhh ;)
7478
The Pub / Re: PM popup
« on October 10th, 2011, 02:52 PM »
Quote from billy2 on October 10th, 2011, 01:21 PM
I quite like 2.0.1 with the opacity and popup going on..... but it is 'in yer face' as you rightly say. On the otherhand you have no excuses for missing one  :)
I also like the lower right window - classy  :cool:
My lower right popup code also has an opacity animation, you know... :P

Anyway, yeah Pete, that popup would best be using the help popup system.
Actually, any popup done with window.open() and things like that... I think there's a to-do list item about that in the list I posted a few days ago ;)

(Oh, and I forgot to reply to your comments on it... -_-)
7479
The Pub / [Archive] Re: Logo Madness
« on October 10th, 2011, 02:47 PM »
Some films aren't firmly set between one of the other genre. For instance, 'Enchanted' has, in homage to the Disney musicals, a few songs performed by the cast 'in situ', categorizing it as a musical, but 90% of the movie is spoken and not sung, so it's not really a musical either.
The Rocky Horror Picture Show is definitely a musical, while Phantom of the Paradise isn't. My 2 cents.
7480
The Pub / Re: Spell checker
« on October 10th, 2011, 02:44 PM »
(Added a poll about the spellchecker. Better than a long conversation I'd say...)

:edit: given the nature of the question, I don't think we should consider removing the spell-checker even if the 'no' has a minority, say 20%. Although 5% is a given :P
7481
The Pub / [Archive] Re: Logo Madness
« on October 10th, 2011, 02:27 PM »
Quote from billy2 on October 10th, 2011, 02:08 PM
Never watched a musical apart from about a quarter of Sweeney Todd: The Demon Barber of Fleet Street, which did absolutely zero for me. I am totally uncultured with regard to 'the arts'.
PotP is not a musical, it's a movie with a few songs performed by the main characters. Per se, a musical is supposed to be a story played through songs, while PotP's songs have pretty much nothing to do with the story, it's just that they're singers, and they perform in front of an audience... But when they're not on stage, they just speak normally. ;)
Quote
As far as the owl mask goes - I think I would have preferred seeing just the metal teeth. (I wiki'd PoftheP)
The guy wasn't very pretty to begin with. But his stay at Sing Sing didn't improve things ;)
7482
Features / Re: New revs
« on October 10th, 2011, 02:22 PM »
rev 1086
(2 files, 5kb)

! Fixed crash that occurred when saving a PM draft with an unknown recipient. Because getPmRecipients was called without going through MessagePost2 first, the Subs-Auth file wasn't loaded in the first place. (PersonalMessage.php)

! Layout issue in Personal Message area when using wrc class on non-PM elements, i.e. anything outside the PM list. (index.member.css)
7483
Features / Re: New revs
« on October 10th, 2011, 02:08 PM »
rev 1085
(79 files, 55kb)

* Converted remaining loadBlock calls to wetem::load(). (other/unittest/index.php, skins/Warm/skin.xml, 77 source files)
7484
Features / Re: New revs
« on October 10th, 2011, 02:01 PM »
rev 1084
(16 files, 33kb)

* Created a 'wetem' (Wedge Template) object, where the entire skeleton system has been moved. Among the functions that will be of interest to modders, loadBlock() is renamed to wetem::load, loadLayer() to wetem::layer, removeLayer() and removeBlock() to wetem::remove, and hideChrome() to wetem::hide. (Subs-Template.php)

+ Testing for a layer or block's presence in the skeleton is now done through wetem::has_block() or wetem::has_layer(), or generally through wetem::has() which will return true if it finds a block of layer with that name. (Profile.php, Security.php, Subs-Template.php)

* Started fixing files to use the object. (index.php, Boards.php, Display.php, ManageAttachments.php, ManageMedia.php, ManageMedia3.php, PersonalMessage.php, Profile.php, Subs-Menu.php, Welcome.php, SSI.php)

* Moved post list to a layer. The code looks complicated for now, but with the next rewrite it will be much simpler. (Display.php, Display.template.php)

@ Note: Wedge will BREAK on many pages with this release, as I need to finish converting all files to use the template object, as well as reprocess the object to use a more general method to handle both blocks and layers. Please bear with me.
7485
Features / Re: New revs
« on October 10th, 2011, 01:57 PM »
rev 1083
(2 files, 4kb -- larger commit coming up.)

* Minor optimization with URL matching. (Subs-Media.php)

! Suspecting a typo here... (other/plugin-info.rng)