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

Max rows in Auto sequence can be bypassed when importing json #26

Open
CableGuy67 opened this issue Jun 23, 2019 · 0 comments
Open

Max rows in Auto sequence can be bypassed when importing json #26

CableGuy67 opened this issue Jun 23, 2019 · 0 comments

Comments

@CableGuy67
Copy link

When importing a json file in Twisted Threads format the max turning rows can be bypassed by manually adding the "F" and "B" sequence to the json file to the auto_turn_sequence array. Once the number of turns exceed 48 (the hard limit for the ui) trying to view the band horizontally won't display anything but a list of the card numbers. Trying to see the print friendly page won't display the band either as it is shown as a horizontal left to right band.
Oddly enough, the thumbnail on the patterns page (also horizontal left to right) will display correctly.

Some possibilities to consider. Keep the max 48 rows as the default but allow for the max to be adjusted somewhere deeper in the pattern UI in the same way that the "make public" is a click in so that increase requests could be dealt with on an as needed basis. I don't know what the overhead is for the rows but from what you have said it is in the way the database keeps the patterns? I do think 48 is too small overall. Somewhere around 60 or 80 is a more realistic baseline max imo.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant