New revs - Public comments

Arantor

  • As powerful as possible, as complex as necessary.
  • Posts: 14,278
Re: New revs - Public comments
« Reply #600, on May 18th, 2013, 10:26 PM »
Quote
I should be able to get a proper list of permissions, global and per board (and per album), for any given member, with a section in their profile area..
SMF has this already, so do we, I just moved it. In SMF, it's Profile > Profile Info > Show Permissions, in Wedge it's Profile > Manage Account > Show Permissions.

It gives you the general permissions and the 'all boards' permissions but lets you select different boards from a dropdown.
Quote
In the radio box list, I'd personally use dropdowns with an icon on each (green for enabled or enabled_own, red for disabled, another green or some blue for enabled_any). Sure, it's an extra click to change a permission, but then you can add deny permissions, right..? Unless you want to add deny for _own and _any separately, in which case, err...
This is what I've been wrestling with. You might want to deny an _any but allow the _own.

If deny permissions weren't an option, I'd do what I did in SimpleDesk which was dropdowns with nice icons.
Quote
- I'd like to see the membergroups ajaxified... i.e. when clicking the link to modify their permissions, have it load under the permission name instead, so that you can even load multiple membergroups on one page, and then change them all in a row...
I'm thinking about it. It gets complex with profiles and stuff, but something along those lines is sot of where I'm going. Still needs more work and I find I need to get a new mouse too >_<
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
  • Posts: 16,079
Re: New revs - Public comments
« Reply #601, on June 12th, 2013, 04:25 PM »
BTW Pete, are you fine with the tweaks I made to the permissions page..?
I only tried to improve their efficiency, at least for me it's working better that way, I guess...

Arantor

  • As powerful as possible, as complex as necessary.
  • Posts: 14,278

Nao

  • Dadman with a boy
  • Posts: 16,079
Re: New revs - Public comments
« Reply #603, on June 12th, 2013, 06:40 PM »
I only changed some colors, added title params everywhere (really!), and decided that the gzipped difference is too trivial to care about.
Re: New revs - Public comments
« Reply #604, on June 13th, 2013, 07:35 PM »
Okay, I was thinking about this, and decided to discuss it first...

How about, we change the permission page, at load time, to replace the radio buttons with a pseudo-select box?
I'm thinking along the lines of the post icon selector, i.e. a green/black/red icon that you can click, and it shows a small dropdown with all possible choices.

As you probably know, I already did something similar in Noisen with the privacy selector; it's re-using most of the IconList code, and it works very well; in fact, I was even able to add checkboxes to the popup, so it's really comfortable in terms of power, I'd say.

What I'm seeing, in the future, is an IconList that would be entirely configurable; it would rely on some custom JS to turn any kind of item into a dropdown list, along with a callback, to transform it internally back into whatever we want, so that we can apply it to radio buttons, checkboxes, select boxes and so on...

What do you think, guys..? Is it realistic, or even exciting, or just something that's not worth it..?

Arantor

  • As powerful as possible, as complex as necessary.
  • Posts: 14,278
Re: New revs - Public comments
« Reply #605, on June 14th, 2013, 01:28 AM »
Quote
How about, we change the permission page, at load time, to replace the radio buttons with a pseudo-select box?
I dunno. I did it with SimpleDesk and while I liked it (especially since I folded own/any into it), I don't think it would work for us. Partly because deny wasn't actually an option in the UI. Partly because we can't actually fold own/any into the dropdowns (since there are situations you'd need to give permissions separately and because there are some fubar moments in terms of any overriding own but the UI doesn't cover that aspect)

Though I don't know why we wouldn't just use a standard selectbox and be done with it if we're going to do it. The | option to add text after an item would cover that well enough.

Ultimately I don't think it's the solution to this problem.

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
  • Posts: 1,670
Re: New revs - Public comments
« Reply #606, on June 14th, 2013, 09:36 PM »
Quote
* PHP 5.2.3+ supports is_callable('class::method') (or even the equivalent function_exists) instead of is_callable(array('class', 'method')), so I figured it's best to use the string format, as it's a bit faster to handle internally.
The same can be applied elsewhere; eg. hooks, schd tasks,, etc.
A confident man keeps quiet.whereas a frightened man keeps talking, hiding his fear.

Arantor

  • As powerful as possible, as complex as necessary.
  • Posts: 14,278
Re: New revs - Public comments
« Reply #607, on June 14th, 2013, 09:39 PM »
For testing is-callable, sure. But for the actual call, call_user_func / call_user_func_array / the occasions we use variable functions ($function()), I'm not sure whether *they* support class::method or not.

Sara

  • Walking Contradiction
  • Posts: 41
Re: New revs - Public comments
« Reply #608, on November 11th, 2013, 09:16 PM »
Quote
* Updated Home template to be more... generic, really. It was tailored for wedge.org, and didn't have a reason to be that way in the repo. While the 'default' is still Welcome, I'd appreciate to get feedback from users -- which one should be the default? (If Home is, then Welcome will be completely removed.) (Home.template.php, Home.language.php)
Forgive my ignorance, regarding the final result, is the Home template the very front page I see when going to wedge.org?  With a blurb about the site, then recent posts, recent thoughts and then the board index?  If that is what you mean in regards to the underlying "Home.template.php" then I would say keep Home instead of Welcome. 

Nao

  • Dadman with a boy
  • Posts: 16,079
Re: New revs - Public comments
« Reply #609, on November 11th, 2013, 09:31 PM »
Yes, that's it.

The Welcome version has "Welcome, hacker!" or something along that line, it's a very limited file. I made Home for Wedge.org, and ended up straightening it out so that it'd remain 'usable' for a default homepage. But then I made a 'fork' of that file, and messed it up in here because I'd renamed it 'Home-Wedge', when I need to have a matching function name in the source files, and dashes aren't accepted in function names. Hence the 5-10 minutes of downtime in here earlier. ;)

Sara

  • Walking Contradiction
  • Posts: 41
Re: New revs - Public comments
« Reply #610, on November 11th, 2013, 09:33 PM »
Thanks for the explanation!  I stick with my original conclusion then!  I personally like the Home template better instead of Welcome. :)

Aaron Smith

  • The mini menus on Thoughts... Damn Sexy!
  • Posts: 39

Nao

  • Dadman with a boy
  • Posts: 16,079
Re: New revs - Public comments
« Reply #612, on December 4th, 2013, 12:24 AM »
Well, Home is the 'default' one now, and Welcome will only be there as a 'sample', although I could probably get it out of the main repo, and into a 'samples' repo.

Pandos

  • Living on the edge of Wedge
  • Posts: 635
Re: New revs - Public comments
« Reply #613, on December 4th, 2013, 04:15 PM »
Translation for new stats is comitted :)
By the way... could you please make the thoughts area "post field" on home a bit larger. Linebreaks (when using german language) are messing up the look&feel.
# dpkg-reconfigure brain
error: brain is not installed or configured

Nao

  • Dadman with a boy
  • Posts: 16,079
Re: New revs - Public comments
« Reply #614, on December 4th, 2013, 04:33 PM »
Done, but it also means the box is larger for all languages, and may wrap to the next line when replying to a thought, even if the thought isn't too long... :-/
Posted: December 4th, 2013, 04:21 PM

Did you check the $txt['charts'] array, BTW..? Because I *did* make it all up from your original strings, and the addition of 'Alles' and 'Ausses' or something, which I have no idea if they're even grammatically correct...
Posted: December 4th, 2013, 04:23 PM

Also, is there a reason you named your German version 'Deutsch (DE)'...? Why not just 'Deutsch'?
English (US) is only there because British people certainly have a case when it comes to British English being the 'proper one' and US English a derivative (lol). But I've settled on English (UK) because there are more Americans, and non-Americans non-British are probably more used to the US grammar, too, so it needed to be done. But German... It's only spoken *widely* in Germany, right..? So it doesn't need the DE, I'd say.