Andreas/mtg_python_deckbuilder
1
0
Fork 0
mirror of https://github.com/mwisnowski/mtg_python_deckbuilder.git synced 2025-12-16 15:40:12 +01:00
Code Issues Projects Releases Packages Wiki Activity Actions

bugfix: removed owned bade on alternatives and added a specific close button to remove any confusion

Browse source
This commit is contained in:
matt 2025-10-13 16:02:12 -07:00
parent 575b85b254
commit bf60da89cd
3 changed files with 9 additions and 6 deletions
Download patch file Download diff file Expand all files Collapse all files

3
CHANGELOG.md
View file

@ -14,9 +14,10 @@ This format follows Keep a Changelog principles and aims for Semantic Versioning
### Added
- Non-basic land type tagging (i.e. Caves, Deserts, Gates, etc...) in the tagging module.
- Close button to alternatives panel header so it can be dismissed.
### Changed
- _No unreleased changes yet._
- Removed the owned badge from each alternative and moved owned metadata to a data attribute on the button.
### Fixed
- Custom theme fuzzy matching now accepts selection.

3
RELEASE_NOTES_TEMPLATE.md
View file

@ -6,9 +6,10 @@
### Added
- Non-basic land type tagging (i.e. Caves, Deserts, Gates, etc...) in the tagging module.
- Close button to alternatives panel header so it can be dismissed.
### Changed
- _No unreleased changes yet._
- Removed the owned badge from each alternative and moved owned metadata to a data attribute on the button.
### Fixed
- Custom theme fuzzy matching now accepts selection.

9
code/web/templates/build/_alternatives.html
View file

@ -5,7 +5,10 @@
#}
<div class="alts" style="margin-top:.35rem; padding:.5rem; border:1px solid var(--border); border-radius:8px; background:#0f1115;" data-skeleton data-skeleton-label="Pulling alternatives…" data-skeleton-delay="450">
<div style="display:flex;justify-content:space-between;align-items:center;margin-bottom:.25rem; gap:.5rem; flex-wrap:wrap;">
<strong>Alternatives</strong>
<div style="display:flex;align-items:center;gap:.5rem;">
<strong>Alternatives</strong>
<button type="button" onclick="this.closest('.alts').remove()" title="Close alternatives panel" aria-label="Close alternatives" style="background:transparent;border:1px solid var(--border);color:var(--text);border-radius:4px;font-size:14px;line-height:1;padding:2px 6px;cursor:pointer;">✕</button>
</div>
{% set toggle_q = '0' if require_owned else '1' %}
{% set toggle_label = 'Owned only: On' if require_owned else 'Owned only: Off' %}
<div style="display:flex; gap:.35rem; flex-wrap:wrap;">
@ -21,16 +24,14 @@
{% else %}
<ul class="alt-list">
{% for it in items %}
{% set badge = '✔' if it.owned else '✖' %}
{% set title = 'Owned' if it.owned else 'Not owned' %}
{% set tags = (it.tags or []) %}
<li>
<span class="owned-badge" title="{{ title }}">{{ badge }}</span>
<button class="btn alt-option" data-card-name="{{ it.name }}"
{% if it.role %}data-role="{{ it.role }}"{% endif %}
{% if it.mana %}data-mana="{{ it.mana }}"{% endif %}
{% if it.rarity %}data-rarity="{{ it.rarity }}"{% endif %}
{% if it.hover_simple %}data-hover-simple="1"{% endif %}
{% if it.owned %}data-owned="1"{% endif %}
data-tags="{{ tags|join(', ') }}" hx-post="/build/replace"
hx-vals='{"old":"{{ name }}", "new":"{{ it.name }}", "owned_only":"{{ 1 if require_owned else 0 }}"}'
hx-target="closest .alts" hx-swap="outerHTML" title="Lock this alternative and unlock the current pick">