search

Phuks-co / throat

Open Source link aggregator and discussion platform powering Phuks
https://phuks.co
MIT License
73 stars 32 forks source link

Add in "speaking as" concept #201

Closed ferocious-ferret closed 3 years ago

ferocious-ferret commented 4 years ago

Allows Moderators and Admins to "distinguish" their posts and comments So that they show up with an alternate style, so that mods and admins stop looking like busy bodies when they tell people what to do.

Polsaker commented 4 years ago

Looks pretty good. An issue I found while testing is that I, being an admin cannot distinguish my comment as admin in a sub that I also moderate, and since admins are implicitly mods of all default subs it's not possible to distinguish a comment as admin in any default subs.

ferocious-ferret commented 4 years ago

Looks pretty good. An issue I found while testing is that I, being an admin cannot distinguish my comment as admin in a sub that I also moderate, and since admins are implicitly mods of all default subs it's not possible to distinguish a comment as admin in any default subs.

So, this was a design decision that was made, but I agree it has some issues.

I had noticed in other places we had people using the least power they had in that area to take an effect, such as deleting a post, and so I replicated that here...

The options for this are really add in another button or ???

I'm not sure what the right answer is to be honest, I guess it's another button, but it feels so ugh to have two buttons that do the same thing for admins...

Do you have any thoughts on the right answer here?

Polsaker commented 4 years ago

I had noticed in other places we had people using the least power they had in that area to take an effect, such as deleting a post, and so I replicated that here...

I can imagine some cases where it might be useful to speak as admin instead of as a mod

I'm not sure what the right answer is to be honest, I guess it's another button, but it feels so ugh to have two buttons that do the same thing for admins...

Yeah I agree there. Maybe an inline prompt similar to the "Are you sure? yes/no" ones. It'd be a trivial modification to the TextConfirm function to set the text for the yes/no part

ferocious-ferret commented 3 years ago

Take a look, let me know what you think

Added in the TextConfirm, and just made it so that it read in a mostly reasonable way rather than trying to get a different label in, but that is also a change that obviously could be made.