live627

  • Should five per cent appear too small / Be thankful I don't take it all / 'Cause I'm the taxman, yeah I'm the taxman
  • Messages: 1 670
Re: Users Online Today
« Réponse #15, le 15 Juin 2012 à 19:10 »
Oh, and it doesn't show in Wine (hey, where's the quick edit button and the action menu? Now I can only quote myself, odd...)
A confident man keeps quiet.whereas a frightened man keeps talking, hiding his fear.

Nao

  • Dadman with a boy
  • Messages: 16 082
Re: Users Online Today
« Réponse #16, le 15 Juin 2012 à 19:46 »
Citation de live627 le 15 Juin 2012 à 18:41
Shows for me just fine, error notwithstanding. Mind if I attempt to commit my change?
Hey... I *was* working on that!! :lol:

Now I'll have to compare the changes... :^^;:

Arantor

  • As powerful as possible, as complex as necessary.
  • Messages: 14 278
Re: Users Online Today
« Réponse #17, le 15 Juin 2012 à 19:49 »
Oh crap something else I got wrong :lol:
When we unite against a common enemy that attacks our ethos, it nurtures group solidarity. Trolls are sensational, yes, but we keep everyone honest. | Game Memorial

Nao

  • Dadman with a boy
  • Messages: 16 082
Re: Users Online Today
« Réponse #18, le 15 Juin 2012 à 20:00 »
Well, up until today I'd never touched the plugin code anyway... ;)

I made a couple more fixes, but your JS code is complicated...
It's not like I can easily reuse weUrl() (which in addition doesn't support anchors...), and it doesn't support index.php-free URLs like here.

I also moved the code to the sidebar, isn't that what your description mentions...?
And added language files.

Arantor

  • As powerful as possible, as complex as necessary.
  • Messages: 14 278
Re: Users Online Today
« Réponse #19, le 15 Juin 2012 à 20:02 »
I honestly don't remember. It was made a long time ago and even though I gave everyone access I half thought no-one would ever look, heh.

Nao

  • Dadman with a boy
  • Messages: 16 082
Re: Users Online Today
« Réponse #21, le 15 Juin 2012 à 20:16 »
rev 36
(3 files +2, 3kb)

! Fixed language files to say "including X hidden" instead of "plus X hidden", which was misleading... (OnlineToday.language.php)

! Fixed theme selector. Added language files. Yeah, I named them after skins... We're not going to call it a theme selector, are we...? (ThemeSelector.php, SkinSelector.language.php)

Arantor

  • As powerful as possible, as complex as necessary.
  • Messages: 14 278
Re: Users Online Today
« Réponse #22, le 15 Juin 2012 à 20:23 »
Citation
which was misleading...
Is it? I thought they were counted separately? They should have been...
Citation
We're not going to call it a theme selector, are we...?
Some of the code is derived from a theme selector I wrote for SMF, the name just sort of stuck really.

Nao

  • Dadman with a boy
  • Messages: 16 082
Re: Users Online Today
« Réponse #23, le 15 Juin 2012 à 20:34 »
Ah, yes, now we have a problem... It's "XX including X" for moderators, and "XX plus X" for visitors...
I think it'd be faster to just have a separate counter for all users, instead of a count().

Arantor

  • As powerful as possible, as complex as necessary.
  • Messages: 14 278
Re: Users Online Today
« Réponse #24, le 15 Juin 2012 à 20:40 »
I seem to recall that's why I kept it separate (that's certainly why SMF keeps it separate)

Nao

  • Dadman with a boy
  • Messages: 16 082

Arantor

  • As powerful as possible, as complex as necessary.
  • Messages: 14 278
Re: Users Online Today
« Réponse #26, le 15 Juin 2012 à 21:50 »
Is it not? It should be...

Oh, I don't remember, it was months ago :P

Nao

  • Dadman with a boy
  • Messages: 16 082
Re: Users Online Today
« Réponse #27, le 15 Juin 2012 à 22:11 »
You can fix it yourself if you want. I'm not on my computer for now ;)