• Resize fTelnet for msweeper to support level 5

    From Rob Swindell@VERT to GitLab note in main/sbbs on Wednesday, November 30, 2022 16:27:48
    https://gitlab.synchro.net/main/sbbs/-/merge_requests/225#note_2882

    Is this change intentional? Doesn't seem to be covered by the commit message.

    ---
    Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
  • From Rick Parrish@VERT to GitLab note in main/sbbs on Thursday, December 01, 2022 06:20:35
    https://gitlab.synchro.net/main/sbbs/-/merge_requests/225#note_2884

    Yes, sorry I didn't mention what that change was for.

    With #fTelnet as the target, the first two lines of the client window are above the scroll position after the client window resizes to 35 lines. This only happens in Firefox, in Chrome it works as expected.

    With # as the target, both Firefox and Chrome show the full client window.

    On a related note, is there an escape sequence that minesweeper could send to the client saying "would you mind resizing to 35 lines"? Would be nice if minesweeper could automatically resize clients that support such a sequence, because this change only helps people who launch minesweeper via the Games page. If they login via fTelnet on the homepage they'll still be stuck with 25 lines and a partial grid for higher minesweeper levels.

    ---
    Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
  • From Rob Swindell@VERT to GitLab note in main/sbbs on Thursday, December 01, 2022 12:58:28
    https://gitlab.synchro.net/main/sbbs/-/merge_requests/225#note_2888

    I'm not keen on hardcoding "minesweeper" here. What if the user just wants a bigger (wider and/or taller) terminal for any other reason?

    That said, I don't know of a standard terminal escape sequence to control the size of the user's terminal. Deuce might know, but I know that SyncTERM wouldn't support it if there was one.

    ---
    Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
  • From Rob Swindell@VERT to GitLab note in main/sbbs on Thursday, December 01, 2022 12:59:22
    https://gitlab.synchro.net/main/sbbs/-/merge_requests/225#note_2890

    Note: 80x35 is not big enough for a level 5.0 Synchronet Minesweeper map. That requires a wider *and* taller terminal.

    ---
    Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
  • From Rick Parrish@VERT to GitLab note in main/sbbs on Thursday, December 01, 2022 15:00:58
    https://gitlab.synchro.net/main/sbbs/-/merge_requests/225#note_2891

    Yeah I wasn't keen on hardcoding it either. Not so much for the case of a user wanting a different size, but presumably people could add minesweeper to an older install where they manually enter the internal code and enter something other than 'msweeper'.

    What about a settings page that allows the user to select their preferred fTelnet window size? Maybe it could have a few defaults, as well as the ability to specify a custom size. Then their preference could be stored in Window.localStorage, or a cookie.

    If that sounds good, do you think it makes sense to enforce 80x24 as a minimum size, or if a user wants 20x10 should they be allowed?

    ---
    Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
  • From Rick Parrish@VERT to GitLab note in main/sbbs on Thursday, December 01, 2022 15:03:49
    https://gitlab.synchro.net/main/sbbs/-/merge_requests/225#note_2892

    What size is needed for a level 5.0? I chose 80x35 based on the Winners screen, which indicates Nelgin's 30x30x180 game was a full 5.0 game, and 80x35 does support a 30x30 grid.

    ---
    Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
  • From Rob Swindell@VERT to GitLab note in main/sbbs on Thursday, December 01, 2022 15:49:14
    https://gitlab.synchro.net/main/sbbs/-/merge_requests/225#note_2893

    You're right. I was misremembering. An 80x30 is not big enough for a level 5.0 map, but 80x35 is.

    ---
    Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
  • From Rob Swindell@VERT to GitLab note in main/sbbs on Thursday, December 01, 2022 15:52:16
    https://gitlab.synchro.net/main/sbbs/-/merge_requests/225#note_2894

    I think user selectable terminal/window size would be a very nice feature to add. I'm not sure if that's a ecWeb feature or an fTelnet feature, but either way, it'd be cool.

    I think 40x24 is a sensible minimum if/when you support PETSCII charset/term-emulation. Otherwise, 80x24 is a fine minimum.

    ---
    Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net