Author Topic: Show reason for ban in banmenu  (Read 3415 times)

0 Members and 1 Guest are viewing this topic.

Offline blt950

  • Newbie
  • *
  • Posts: 7
  • Karma: 0
Show reason for ban in banmenu
« on: December 24, 2009, 03:10:40 AM »
It's a short suggestion, but is a little useful.
Its that if you ban somone, you can see the bans in !banmenu. You can see: who, when, and by who. Why not add reason of the ban there too? It would be useful for server owners :)

And Merry Christmas!  ;D

Offline JamminR

  • Ulysses Team Member
  • Hero Member
  • *****
  • Posts: 8096
  • Karma: 390
  • Sertafide Ulysses Jenius
    • Team Ulysses [ULib/ULX, other fine releases]
Re: Show reason for ban in banmenu
« Reply #1 on: December 24, 2009, 08:55:00 AM »
If you click the red X (Unban button), a window pops up asking confirmation of the unban... within that window is a 'reason' line.
Are you asking that be included as one of the columns instead of just in the unban confirmation?
I'm not sure why it wasn't already, but knowing Megiddo's coding style, I'd guess that due to the fact the reason could be long/typed out, it would mess up the tidyness of that first window.
"Though a program be but three lines long, someday it will have to be maintained." -- The Tao of Programming

Offline blt950

  • Newbie
  • *
  • Posts: 7
  • Karma: 0
Re: Show reason for ban in banmenu
« Reply #2 on: December 25, 2009, 08:46:14 AM »
So as I understood it will be added? Or it would up if it would?

Offline JamminR

  • Ulysses Team Member
  • Hero Member
  • *****
  • Posts: 8096
  • Karma: 390
  • Sertafide Ulysses Jenius
    • Team Ulysses [ULib/ULX, other fine releases]
Re: Show reason for ban in banmenu
« Reply #3 on: December 25, 2009, 10:35:55 PM »
No, I'm asking where you mean for the reason to be.
The unban button confirmation area already shows a reason.
The main list wouldn't display well unless we limit the ban reason, which in itself isn't a good idea.
"Though a program be but three lines long, someday it will have to be maintained." -- The Tao of Programming

Offline MrPresident

  • Ulysses Team Member
  • Hero Member
  • *****
  • Posts: 2728
  • Karma: 430
    • |G4P| Gman4President
Re: Show reason for ban in banmenu
« Reply #4 on: December 26, 2009, 07:10:15 AM »
Idea: What about making the unban menu like a scoreboard format. Have it show the person banned and some other basic information in the header (like time remaining/unban date and admin who banned) and then if you click on it, it'll expand down to show the reason they were banned, as well as the other information stored by ULX Ban. You could also have a button on there to unban the individual.

I dunno, just an idea, maybe I'll make a plugin ;)

Offline JamminR

  • Ulysses Team Member
  • Hero Member
  • *****
  • Posts: 8096
  • Karma: 390
  • Sertafide Ulysses Jenius
    • Team Ulysses [ULib/ULX, other fine releases]
Re: Show reason for ban in banmenu
« Reply #5 on: December 26, 2009, 12:35:14 PM »
Sure MrPresident... run with that. :P
"Though a program be but three lines long, someday it will have to be maintained." -- The Tao of Programming

Offline Megiddo

  • Ulysses Team Member
  • Hero Member
  • *****
  • Posts: 6214
  • Karma: 394
  • Project Lead
Re: Show reason for ban in banmenu
« Reply #6 on: December 26, 2009, 02:57:04 PM »
I didn't add it because I was worried about formatting. (JamminR knows me too well, apparently; get out of my head! :P)

As the design currently stands, it just wouldn't look good there. Gui's not really my cup of tea though, so anyone's welcome to improve on it. :)
Experiencing God's grace one day at a time.