• Resize fTelnet for msweeper to support level 5

    From Rob Swindell@1:103/705 to GitLab note in main/sbbs on Wed Nov 30 16:27:48 2022
    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. --- SBBSecho 3.20-Linux
    * Origin: Vertrauen - [vert/cvs/bbs].synchro.net (1:103/705)
  • From Rick Parrish@1:103/705 to GitLab note in main/sbbs on Thu Dec 1 06:20:35 2022
    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.
    --- SBBSecho 3.20-Linux
    * Origin: Vertrauen - [vert/cvs/bbs].synchro.net (1:103/705)
  • From Rob Swindell@1:103/705 to GitLab note in main/sbbs on Thu Dec 1 12:58:28 2022
    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.
    --- SBBSecho 3.20-Linux
    * Origin: Vertrauen - [vert/cvs/bbs].synchro.net (1:103/705)
  • From Rob Swindell@1:103/705 to GitLab note in main/sbbs on Thu Dec 1 12:59:22 2022
    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.
    --- SBBSecho 3.20-Linux
    * Origin: Vertrauen - [vert/cvs/bbs].synchro.net (1:103/705)
  • From Rick Parrish@1:103/705 to GitLab note in main/sbbs on Thu Dec 1 15:00:58 2022
    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?
    --- SBBSecho 3.20-Linux
    * Origin: Vertrauen - [vert/cvs/bbs].synchro.net (1:103/705)
  • From Rick Parrish@1:103/705 to GitLab note in main/sbbs on Thu Dec 1 15:03:49 2022
    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.
    --- SBBSecho 3.20-Linux
    * Origin: Vertrauen - [vert/cvs/bbs].synchro.net (1:103/705)
  • From Rob Swindell@1:103/705 to GitLab note in main/sbbs on Thu Dec 1 15:49:14 2022
    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.
    --- SBBSecho 3.20-Linux
    * Origin: Vertrauen - [vert/cvs/bbs].synchro.net (1:103/705)
  • From Rob Swindell@1:103/705 to GitLab note in main/sbbs on Thu Dec 1 15:52:16 2022
    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.
    --- SBBSecho 3.20-Linux
    * Origin: Vertrauen - [vert/cvs/bbs].synchro.net (1:103/705)