Open PeterNerlich opened 3 years ago
I did not intend to address the same as #64, I wanted to start a conversation on whether to increase the number of characters displayed on the LCD screen. Only the first screenshot is what #64 looks like, and only changes the positioning for the current limits as well as making it properly generalized to enable changing those limits without it glitching out again. The subsequent screenshots are examples to demonstrate exactly that and give a basis to kickstart the conversation.
Thus, I kindly ask you to reopen the issue. And don't apologise, I firmly expect this to be an accidental thing because this looked the same and mentions #64. As does this comment now, thrice even... :sweat_smile:
Well, I'd say that using all available space is the desired behaviour - as long it's guaranteed that no letters overshoot (handled already).
Not necessarily with the line number though. While the individual lines are each centered, we do not vertically center by first seeing how many lines we end up with and then distributing the leftover space, so changing this would make many existing contraptions that just use a limited number of lines look ugly all of the sudden. I'm just speculating though, that's why I wanted to ask those who have observed this longer than me. If making designs ugly is of no concern and having as much space available as possible is much desirable, I could make that PR right away.
Currently the LCD screen has quite big borders and a very limited amount of text that can be displayed. I have come across intentions to connect multiple LCDs to one big display array. In that case, there is much less space to print characters than one would expect. Maybe it would be better to do this in a separate mod, probably there were some considerations why the limits were set as they are and I'd like to hear them and what your thoughts are about making more characters fit on screen. I fixed the calculations in #64 enabling me to play around with this a bit:
Old screen with 5 lines, 12 chars each (fixed positioning as of #64):
Max character cramming (6 lines by decreasing line height by one, 14 chars each):
Inbetween with 6 lines, 13 chars each: