discoman/python-tdd
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Sea Select options: fixed-width chunks + 2-line spread names (add Rider-) — TDD

Browse Source 
- The option chunks' width tracked the select label length (jumped between the
  two spreads). Fix the `.sea-form-col` to 19rem (a touch wider) so all three
  chunks share a standard width.
- Split the spread label onto two lines around the comma + add the "Rider-"
  prefix now there's room: "Celtic Cross," / "Rider-Waite-Smith" and "Celtic
  Cross," / "Escape Velocity". combobox.js now writes the current label via
  `innerHTML` (not textContent) so the `<br>` survives a selection — plain-text
  options (sky / my_sea) are unaffected (their innerHTML == textContent).
- ITs updated for the new gameroom labels (my_sea's single-line names untouched).
  953 epic+gameboard ITs + Jasmine green.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
This commit is contained in:
Disco DeDisco
2026-06-08 00:25:11 -04:00
parent 2bf439eab5
commit 0a6bfcf6cc
4 changed files with 18 additions and 20 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
src/templates/apps/gameboard/_partials/_sea_overlay.html
View File

@@ -41,15 +41,15 @@ via epic:sea_save. `?seat` threads the CARTE-selected seat onto the action URLs.
aria-haspopup="listbox"
aria-labelledby="id_sea_spread_label"
tabindex="0">
<span class="sea-select-current">{% if sea_default_spread == "escape-velocity" %}Celtic Cross, Escape Velocity{% else %}Celtic Cross, Waite-Smith{% endif %}</span>
<span class="sea-select-current">{% if sea_default_spread == "escape-velocity" %}Celtic Cross,<br>Escape Velocity{% else %}Celtic Cross,<br>Rider-Waite-Smith{% endif %}</span>
<span class="sea-select-arrow" aria-hidden="true">▾</span>
<ul class="sea-select-list" role="listbox">
{% if user_polarity == "levity" %}
<li role="option" data-value="waite-smith" aria-selected="{% if sea_default_spread == 'escape-velocity' %}false{% else %}true{% endif %}">Celtic Cross, Waite-Smith</li>
<li role="option" data-value="escape-velocity" aria-selected="{% if sea_default_spread == 'escape-velocity' %}true{% else %}false{% endif %}">Celtic Cross, Escape Velocity</li>
<li role="option" data-value="waite-smith" aria-selected="{% if sea_default_spread == 'escape-velocity' %}false{% else %}true{% endif %}">Celtic Cross,<br>Rider-Waite-Smith</li>
<li role="option" data-value="escape-velocity" aria-selected="{% if sea_default_spread == 'escape-velocity' %}true{% else %}false{% endif %}">Celtic Cross,<br>Escape Velocity</li>
{% else %}
<li role="option" data-value="escape-velocity" aria-selected="{% if sea_default_spread == 'waite-smith' %}false{% else %}true{% endif %}">Celtic Cross, Escape Velocity</li>
<li role="option" data-value="waite-smith" aria-selected="{% if sea_default_spread == 'waite-smith' %}true{% else %}false{% endif %}">Celtic Cross, Waite-Smith</li>
<li role="option" data-value="escape-velocity" aria-selected="{% if sea_default_spread == 'waite-smith' %}false{% else %}true{% endif %}">Celtic Cross,<br>Escape Velocity</li>
<li role="option" data-value="waite-smith" aria-selected="{% if sea_default_spread == 'waite-smith' %}true{% else %}false{% endif %}">Celtic Cross,<br>Rider-Waite-Smith</li>
{% endif %}
</ul>
</div>