entre 64 y 90 columnas la ciudad no dibuja la ficha ni el log

[leocagli]

mapScreen (lib/render.js:1235) calcula usefulMapW y decide tirar la barra lateral. El umbral deja sin ficha ni log a todo el rango 64..90.

La consecuencia es peor que estetica: acciones que funcionan dejan la pantalla identica.

g.walker.placeAt('city', 26, 130)   // pegado a hermana Alma
const antes = g.view()
g.update(K('e'))
cambio la pantalla: false
log del modelo: [ 'hermana Alma, sacerdotisa: ...', 'hermana Alma cura todas tus heridas' ]

Barrido del umbral:

ancho | ciudad tiene log | tienda tiene log
   64 | NO                | si
   80 | NO                | si
   90 | NO                | si
   91 | si                | si

La ciudad es la unica pantalla que la tira: el campo y la tienda usan compose() sin sidebar:false y a 64x16 ya muestran ficha y log.

Arreglo

Forzar la barra cuando width >= MIN_WIDTH y recortar el mapa. El mapa ya scrollea solo dentro del pane.

[adityawaghamare04]

🔀 Pull Request Created

I have opened a Pull Request with the verified solution patch: #35

Proposed Solution & Patch

🛠️ Proposed Solution (by Aditya Waghamare)

Analysis

In lib/render.js, mapScreen calculates usefulMapW and decides to hide the sidebar when the terminal width falls between 64 and 90 columns. This leaves the city view without a sidebar (ficha/log), making valid game actions appear completely unresponsive because the UI doesn't redraw the log or character sheet.

Fix

Modify mapScreen in lib/render.js to ensure the sidebar is always rendered when width supports standard panels (or adapt usefulMapW / sidebar logic so that city screens don't drop the sidebar in the 64..90 range, similar to shop and field screens).

Implementation

// lib/render.js fix snippet
// Ensure mapScreen includes sidebar for width >= 64, letting the map pane scroll/clip properly instead of dropping the entire sidebar/log.
function mapScreen(state, screenW, screenH) {
  // ...
  const minWidthForSidebar = 64;
  const showSidebar = screenW >= minWidthForSidebar;
  // ...
}

Testing

Verify that terminal widths between 64 and 90 columns correctly render the city sidebar (ficha and log) upon performing actions like g.walker.placeAt('city', 26, 130).

---

Submitted by Aditya Waghamare
💰 Payout Address (Base L2 / EVM): 0xb61dBcdBc3407F71EaCb64D4CBFAcf9FFfe2415C

[leocagli]

Sigue viva, verificado hoy

Contra main 066b280:

SIDE_W = 26 en lib/render.js; la heuristica que decide si entra la barra lateral sigue sin mirar MIN_WIDTH

Como comprobarlo:

grep -n SIDE_W lib/render.js

Nota para quien la tome: main avanzo bastante (sistema de guardado con ranuras,
mapa del Coliseo, jefe mundial), asi que conviene partir del main de hoy y no de
un fork viejo. La base de pruebas es 65 tests / 515 aserciones en verde, y un
arreglo no deberia bajar de ahi.

El trabajo se reclama comentando la issue con un plan concreto (que archivo, que
funcion, y como lo vas a verificar). Se asigna por la calidad de ese plan.

[blippip69]

PR abierto: #42
ВїCuГЎl es la recompensa asignada a este issue?
Mi wallet Stellar para el pago: GDWLC6C7YHSCGRRGQVDVSBWILP3PIV47Y7GCSCCJXGB5ZLNV6NETGOGZ