LibertyBans icon indicating copy to clipboard operation
LibertyBans copied to clipboard

Allow /alts command to use pagination (and maybe horizontality)

Open RealTriassic opened this issue 1 year ago • 1 comments

I have confirmed that ...

  • [X] There is no duplicate feature request.

Description

Currently, you can only have the /alts output display as a list. This is okay, but not really compact and if a player has a lot of alts, it can get really annoying, really fast.

My suggestion is to allow the /alts output to be displayed horizontally to allow users to configure the message format themselves to be more compact if they'd like to.

Image from LiteBans' /alts command as an example: image

RealTriassic avatar Aug 25 '24 17:08 RealTriassic

Marking this as a bug since, according to devjakob on Discord, a sufficiently long chat message will cause the recipient to be kicked. This means we need pagination. When pagination is designed, you will be able to configure the per-page message with either new-lines or commas as needed.

A248 avatar Sep 28 '24 10:09 A248

Hi there, as I mentioned on Discord, I'm looking for testers of this feature. You can become a tester either to help from goodwill, or to get early access to the bug-fix. I apologize this bug took a while to solve until now.

A248 avatar Mar 28 '25 19:03 A248

Basic manual testing suggests alt pagination is working, and integration testing suggests pagination is fully working. However, I don't have a large enough dataset to comprehensively test this feature.

Here is a test jar: http://www.mediafire.com/file/74acq9hvm338h7u/LibertyBans_1.1.0-SNAPSHOT.jar

A248 avatar Mar 29 '25 22:03 A248

I solved an edge case where if two alts were registered at the exact same second, one of them would not appear if they occur at the page boundary. That's basically impossible, but good to have fixed.

Otherwise, this feature is implemented.

A248 avatar Jul 03 '25 13:07 A248