Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Switch lua.lua to use extutils.pagedPrint() for Tables #422

Open
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
5 participants
@Wilma456
Copy link
Contributor

Wilma456 commented Aug 20, 2017

With this PR lua.lua can how the full table, if the table is bigger than your Screen.

Switch lua.lua to use extutils.pagedPrint() for Tables
With this PR lua.lua can how the full table, if the table is bigger than your Screen.
@SquidDev

This comment has been minimized.

Copy link
Contributor

SquidDev commented Aug 20, 2017

I feel we'd have to improve pagedPrint before doing this, maybe something like this. Otherwise you're going to be pressing keys a long time for big tables.

It might also be worth creating a modified serialise function, which can handle recursive structures and functions.

@Wilma456 Wilma456 closed this Sep 7, 2017

@Wilma456 Wilma456 deleted the Wilma456:luatable branch Sep 7, 2017

@Wilma456 Wilma456 restored the Wilma456:luatable branch Sep 7, 2017

@Wilma456

This comment has been minimized.

Copy link
Contributor Author

Wilma456 commented Sep 7, 2017

Sorry for closing

@Wilma456 Wilma456 reopened this Sep 7, 2017

@dmarcuse

This comment has been minimized.

Copy link
Contributor

dmarcuse commented Sep 7, 2017

Eh, I'm not really sure if this should be the default behavior. This is a REPL, you can always call textutils.pagedPrint yourself.

@lupus590

This comment has been minimized.

Copy link
Contributor

lupus590 commented Sep 7, 2017

Perhaps have this as an option in settings?

@BombBloke

This comment has been minimized.

Copy link
Contributor

BombBloke commented Sep 7, 2017

I'd rather it be the default behaviour, personally. It's hard to think of cases where I'd ever want part of a displayed table to be immediately hidden out the top of my screen (it's not like we can scroll back up), and if I do decide it'll take too long to scroll through a complete table, I can always just terminate the textutils call.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.