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
8041
Features / Re: New revs - Public comments
« on August 20th, 2011, 10:30 PM »
Good. Because I could always add a shortcut to that somewhere but i prefer a menu link to begin with. :)
8042
Features / Re: New revs - Public comments
« on August 20th, 2011, 10:13 PM »
Well... I guess we can try.

Oh, I think I'm walking into your territory now. In the spirit of my latest commit. I'm planning to split the change theme page into its own menu entry. Are you okay with that?
8043
Features / Re: New revs
« on August 20th, 2011, 05:53 PM »
rev 958
(7 files, 12kb)

* Painstakingly rewrote the skin list function (in the Profile area) to fix a logic flaw. Add-type skins are now listed relative to their parent skin, and replace-type skins are listed on the top level, just like the root skin. (Themes.php, Themes.template.php)

* Tweaked language files to make a clearer statement about choosing the default skin when you choose a theme rather than a specific skin. (Themes.language.php)

- The skin list function was still avoiding the 'cache' folder, which was a remnant from the original skin system implementation... (Themes.php)

* Moved the menu toggler to the left of the menu -- where it makes a lot more sense having... (index.css, index.rtl.css)

* Number of theme users now uses number_context. (Themes.template.php, Themes.language.php)

* May not be necessary at all, but I ensured eAccelerator and other cache systems aren't detected if their 'put' function is detected but not callable. Consider it an extra safety. (ManageServer.php)

@ Minor issue with skins (as seen above): if the admin sets the guest skin (i.e. the default) to something else than the root skin, the thumbnail will still show the root skin. May have to change that, but there is only one default skin per website and there may be multiple themes... Dunno what's best. Any ideas...?
8044
Other software / Re: Fork discussion at SMF
« on August 20th, 2011, 05:08 PM »
Quote from Nightwish on August 18th, 2011, 08:42 PM
I'm going to do this as well in mine and it has little to do with copying, just with agreeing on the fact that it wasn't a very bright idea to include it in the first place. Seriously, how well are the PGSQL and SQLite layers tested in a real-world scenario?
I'm assuming it made sense back in 2005, when SMF2 was started. PHP5 was new, and at the time the MySQL license was seen as problematic when linked to PHP's, or something, and they decided to try and push for SQLite. Which didn't really work... I don't know about PG, but I suppose that because they made it possible to add other engines, they wanted to make space for a third one by default.
Quote
I'll also kill support for PHP4, caching support for eAccelerator (doesn't make sense, because eAccel has been stripped from the required functionality)
Not exactly, it was simply made disabled by default. Meaning someone using it can still force it to be enabled, so, because eAcc is still updated, I think it also makes it harder to remove support for it, unlike MMCache which was one of the first things to go in Wedge IIRC.
Quote
I hate legacy code that just sits there, making the code harder to read and maintain while doing absolutely nothing for 99% of all installations.
I think we 'managed' to remove several thousand lines of codes altogether. Still, it doesn't represent much when compared with the SMF codebase's size, but it did add a lot of overhead to functions that are central to SMF. Also, removing the first parameter in $smcFunc['db_query'] was heavenly. I've never liked the fact that 99% of SMF's queries had an empty string in front of them... It could have easily been put to the end of the list!
8045
Features / Re: New revs
« on August 20th, 2011, 02:08 PM »
rev 957
(1 file, 5kb)

! Although newer Firefox versions seem to force resizing handles on smileys in Wysiwyg mode, we can *at least* convert these back to real smileys when saving, regardless of whatever dimensions are forced in. (Class-Editor.php)

@ Note: this doesn't seem to happen in SMF, so it was a bug in my earlier smiley rewrite.
8046
Features / Re: Public & Private Groups?
« on August 20th, 2011, 01:08 PM »
I suppose that problem won't come up in Wedge, given that user boards are actual boards :P

(Oh, I should really get started and convert that code from noisen.com...)
8047
Features / Re: Login with eMail instead of username
« on August 20th, 2011, 12:29 PM »
Makes sense. Although the opposite does as well :P
8048
Features / Re: New revs - Public comments
« on August 20th, 2011, 12:17 PM »
I don't know, but I don't think we'll have CKEditor at all in v1.0... It'll probably take too long to implement. Or we'll have to postpone 1.0 until mid-2012... And I don't think people will like that. (If we're only doing Wedge for ourselves, okay, but I think we're past that aren't we...? :^^;:)
8049
Features / Re: Login with eMail instead of username
« on August 20th, 2011, 11:56 AM »
Logging in with e-mail addresses may feel slightly more 'natural' to people these days, e.g. your login form has 'e-mail address' and 'password', while you may not be sure whether 'user name' may refer to your actual user name or current display name...

Hmm well, I'm not sure anyone bothered until now, though... If it ain't broke...

It's just something about login scrapers.
8050
Features / Re: Login with eMail instead of username
« on August 20th, 2011, 11:24 AM »
What about existing users with dummy emails?
8051
Features / Re: Login with eMail instead of username
« on August 20th, 2011, 10:08 AM »
Hmm... Yes, I suppose it makes a lot of sense regarding forum username scrapers...
I think that'd be a definite yes, even if it adds an option to the profile area.

Unless we ask the user at registration time only...?

Like, we ask for an e-mail address, an account name and a display name. Then we ask the user what they want to use to login.... Errr.... Okay that's a bit overkill... :P

Most 'big' websites allow you to login with either email or username, your choice. How does that help them with scraping...? I suppose it doesn't.
8052
Features / Re: New revs - Public comments
« on August 20th, 2011, 09:53 AM »
So, I reverted your revert. ;)

Oh, I fucking hate IE... I'm looking at the post editor under IE7 and IE8. It's so incredibly broken, especially in Wysiwyg... There are glitches everywhere, it's comical. I wouldn't even know where to start.

And I fucking don't want to fix them!
8053
Features / Re: New revs
« on August 20th, 2011, 09:50 AM »
rev 956
(1 file, 3kb)

! Typo in one of the disable_feed replacements. Note for later: have someone actually actively look for issues in our commits, especially late at night, so that we don't have to do it in the morning :P (Media.template.php)
8054
Features / Re: New revs
« on August 20th, 2011, 12:30 AM »
rev 955
(7 files, 12kb)

* Replaced $amSettings['disable_feed'] with its opposite, $modSettings['xmlnews_enabled']. While it's nice to know you can remove feed links from your media pages and still have them work, I doubt there's any good reason for doing so... (install.sql, ManageMedia.php, ManageMedia.language.php, Media.template.php)

* Spacinazi and version numbers change. (SSI.php, ssi_examples.php)
8055
Features / Re: More stuff for the removal of
« on August 20th, 2011, 12:14 AM »
I already discussed that bug elsewhere... It's a good I won't care fixing because the forum will eventually run Wedge anyway...

Re: relative timestamps. I just think "3 days ago" is not enough, it should still add the hour and be done with it. Other than that, perhaps we can save performance by doing a $(window).blur(is_disabled = true) or something, and thus stop the update process when the tab is hidden... In which case I'm okay with having it show by default. (See, I always have at least 150 tabs open... Currently I'm closer to 300 but shh!)