search

MinetestForFun / server-minetestforfun-skyblock

Repository of the subgame and mods of "MinetestForFun Skyblock" server
https://www.xorhub.com
6 stars 3 forks source link

[xban2] Gui does not wordwrap #109

Closed aerozoic closed 8 years ago

aerozoic commented 8 years ago

The xban gui doesn't wordwrap so i can't see the entire ban report for a player. All i can see is the time and date, which is actually the CURRENT server time. Is this normal? Does it show the ban time and expire time?

clipboard01

BetterToAutomateTheWorld commented 8 years ago

Don't know if we can fix it without redo the entire formspec... Clearly a "xban" bad feature Maybe @crabman77 can help us ?

linked @kaeza just in case he wants to fix his mod :)

kaeza commented 8 years ago

@Ombridride Thank you for linking. The xban2 GUI needs some love. Expect something soon(tm).

kaeza commented 8 years ago

@cessna151 We have no way to know the client's timezone (it's actually possible, but out of scope for xban2), so the server time is the best we can do.

aerozoic commented 8 years ago

The server time is fine. But maybe i didn't explain well... it is showing live current time, if i keep clicking my name in the list i can watch the seconds tick by. It would be nice to know the time the ban was issued and the expire time. I can't see the end of the message, maybe this info is already there but at the end where i can't see it?

BetterToAutomateTheWorld commented 8 years ago

An other problem is we can't read more than ~80 letters, it cuts our full ban message

Lymkwi commented 8 years ago

An other problem is we can't read more than ~80 letters, it cuts our full ban message

I'm not sure you can have a ban reason longer than 80 characters.

BetterToAutomateTheWorld commented 8 years ago

@LeMagnesium look at the screenshot, in the ban message, near 60-65 characters are taken by the date/hour :)

kaeza commented 8 years ago

See minetest-mods/xban2@974b47f8f6f9f654e9ce3e0c2cb92e45ce743bcf.

screenshot_20160505_075050

BetterToAutomateTheWorld commented 8 years ago

It's a good step, thank you for your work @kaeza :) @LeMagnesium can you update the subtree please ?

Lymkwi commented 8 years ago

https://github.com/MinetestForFun/server-minetestforfun-skyblock/commit/af10e1164288bec00fa52135b06e74081cfd6db6

This seemingly helps for the issue. Thanks @kaeza . Should we close it or try to adapt the GUI to Sb a little bit more?

BetterToAutomateTheWorld commented 8 years ago

I think we can't do more with the GUI, i closed this issue.

Thank you again @kaeza !

kaeza commented 8 years ago

@Ombridride, @LeMagnesium, You are quite welcome.