Multiverse / Multiverse-Portals

MV2's implementation of MV1 style portals.
BSD 3-Clause "New" or "Revised" License
122 stars 89 forks source link

Some changes for better clarity of the list command #622

Closed John-H-Smith closed 3 years ago

John-H-Smith commented 3 years ago

To make a better list command, I changed the default appended names to a list view with multiple pages. Also you can, instead of a filter, type the page you want to get the portals. As further little enhancement, I added the destination portal name into the line of the portal's entry if there's a destination set. Here some images as explanation:

Old: image

New: image image image

Sadly, you're not able to use a filter and group by pages at the same time.

benwoo1110 commented 3 years ago

Oh yea, another thing, maybe we should show the destination of all portals so something like:

test1 -> w:world
test2 -> p:test1
test3 -> e:nether:12,20,12

You can probably get the destination string with p.getDestination().toString()

John-H-Smith commented 3 years ago

I've changed all the brackets you mentioned. Also, I enhanced the destination view, just have a look: image

John-H-Smith commented 3 years ago

Ah, what do you think about giving the world behind the portal name for portal destinations? So you can see everywhere which world you would enter.

benwoo1110 commented 3 years ago

I've changed all the brackets you mentioned. Also, I enhanced the destination view, just have a look: image

Is there a reason why some portals listed doesn't have a destination shown?

Ah, what do you think about giving the world behind the portal name for portal destinations? So you can see everywhere which world you would enter.

Sure!

John-H-Smith commented 3 years ago

Is there a reason why some portals listed doesn't have a destination shown? Yeah, that's in case a portal has been created without a destination, just using /mvp create <name> and has not been modified either.

I just added the world's name after portal's name: image

benwoo1110 commented 3 years ago

Thanks for the PR! Really appreciate it!