feat; add title system UI, glyph sheet #10

Open
planet wants to merge 1 commit from feature/title-update into main
Member
No description provided.
feat; add title system UI, glyph sheet
Some checks failed
Auto Assign / run (pull_request_target) Successful in 4s
Model Preview / model-preview (pull_request) Successful in 9s
CI / build-and-test (20.x) (pull_request) Failing after 19s
ffce79e69b
smell-of-curry was assigned by forgejo-actions 2026-08-15 23:11:19 +00:00
smell-of-curry left a comment

I dont think the form handling is correct

I dont think the form handling is correct
refactor: repack title glyph sheet gap-free
Some checks failed
Model Preview / model-preview (pull_request) Successful in 11s
CI / build-and-test (20.x) (pull_request) Failing after 17s
a88be0d166
feat: standalone json ui form
Some checks failed
Model Preview / model-preview (pull_request) Successful in 10s
CI / build-and-test (20.x) (pull_request) Failing after 17s
9d2fabb1f1
smell-of-curry left a comment

way better

way better
texts/es_ES.lang Outdated
@ -11648,3 +11648,10 @@ forms.emojiShop.success.purchased=§aYou unlocked the §e%s§a for §e%s§a Toke
forms.emojiShop.success.claimed=§aYou claimed the §e%s§a!
forms.emojiShop.error.insufficientTokens=§cYou need §e%s§c Tokens but only have §e%s§c.
forms.emojiShop.error.alreadyOwned=§cYou already own the §e%s§c.
## Titles

why did this one not get a space like the others?

why did this one not get a space like the others?
planet marked this conversation as resolved
fix: add blank line before ## Titles section in all lang files
Some checks failed
Model Preview / model-preview (pull_request) Successful in 10s
CI / build-and-test (20.x) (pull_request) Failing after 18s
0d8963072a
bot requested changes 2026-08-30 22:58:23 +00:00
Dismissed
bot left a comment

Resource-pack review of head 0d8963072a8d (companion to beh#722). mergeable=false vs current main — rebase before merge.

UI wiring is the right pattern: $flag_titles = §t§i§t§l, subtracted from the default long-form visible/enabled bindings, titles@titles.main_panel gated the same way. _ui_defs.json + titles_gui.png + glyph_E3.png are present. Human reviews are stale (ffce79e REQUEST_CHANGES, 9d2fabb APPROVED).

Blocker: events.beforeChat.format is changed in every locale to strip the [rank] brackets. That is global chat, not title-UI scope. If titles are meant to replace the rank prefix, say so and keep it; otherwise revert the format key so this PR stays title-system only.

Title strings (forms.titles.*, commands.tags.*) are English in all 7 locales. Fine as a first pass, but not localized.

Resource-pack review of head `0d8963072a8d` (companion to beh#722). mergeable=false vs current main — rebase before merge. UI wiring is the right pattern: `$flag_titles` = `§t§i§t§l`, subtracted from the default long-form visible/enabled bindings, `titles@titles.main_panel` gated the same way. `_ui_defs.json` + `titles_gui.png` + `glyph_E3.png` are present. Human reviews are stale (`ffce79e` REQUEST_CHANGES, `9d2fabb` APPROVED). **Blocker:** `events.beforeChat.format` is changed in every locale to strip the `[rank]` brackets. That is global chat, not title-UI scope. If titles are meant to replace the rank prefix, say so and keep it; otherwise revert the format key so this PR stays title-system only. Title strings (`forms.titles.*`, `commands.tags.*`) are English in all 7 locales. Fine as a first pass, but not localized.
texts/en_US.lang Outdated
@ -3974,3 +3974,3 @@
events.beforeChat.error.tooManyEmojis=§cA maximum of %s emojis can be sent in a single message.
events.beforeChat.error.zalgo=§cYour message contains zalgo and cannot be sent!
events.beforeChat.format=§r§l§8[§r%s§r§l§8]§r§7 %s:§r %s
events.beforeChat.format=§r%s§r§7 %s:§r %s
Member

This is the live chat format for every message, not a titles-form string. It drops §l§8[§r%s§l§8] so ranks no longer render in brackets. Unrelated to "add title system UI, glyph sheet" unless beh#722 is deliberately replacing the prefix with a glyph title — if so, call that out; if not, revert this key.

This is the live chat format for every message, not a titles-form string. It drops `§l§8[§r%s§l§8]` so ranks no longer render in brackets. Unrelated to "add title system UI, glyph sheet" unless beh#722 is deliberately replacing the prefix with a glyph title — if so, call that out; if not, revert this key.
@ -0,0 +40,4 @@
"anchor_from": "top_left",
"anchor_to": "top_left",
"grid_dimensions": [2, 14],
"collection_name": "form_buttons",
Member

Hard cap of 28 slots ([2, 14]). #form_button_length can grow past that and the extra titles never get a cell. Either document the 28-title max with BEH or bind grid rows from the collection length so new titles do not clip.

Hard cap of 28 slots (`[2, 14]`). `#form_button_length` can grow past that and the extra titles never get a cell. Either document the 28-title max with BEH or bind grid rows from the collection length so new titles do not clip.
fix: drop unused commands.tags.error.unknownTitle key
Some checks failed
Model Preview / model-preview (pull_request) Successful in 10s
CI / build-and-test (20.x) (pull_request) Failing after 19s
dfb0c8f251
bot requested changes 2026-09-05 07:25:36 +00:00
Dismissed
bot left a comment

Re-review head dfb0c8f25188 (was 0d8963072a8d). Companion beh#722.

This push: drops unused commands.tags.error.unknownTitle across all 7 langs. Correct — beh review already marked that key unused. Note beh langKeys.ts still lists it until generate:langKeys runs against this RP.

Cleared: events.beforeChat.format bracket strip. Intentional with beh beforeChat.ts — title glyph + ranks are built into the first %s; the lang string must not wrap it again. Prior inline on that line is obsolete context.

Still blocking: mergeable=false vs current main. Rebase before merge. Do not merge res ahead of beh#722.

Still open (no new inline): titles_server_form.json grid_dimensions: [2, 14] hard-caps 28 cells. Catalog is 25 titles today, so it fits, but anything past 28 silently clips. Prefer binding rows from collection length or document the cap with BEH.

Nit: title strings remain English in non-EN locales.

Human reviews still stale (ffce79e REQUEST_CHANGES, 9d2fabb APPROVE).

Re-review head `dfb0c8f25188` (was `0d8963072a8d`). Companion [beh#722](https://git.pokebedrock.com/pokebedrock/pokebedrock-beh/pulls/722). **This push:** drops unused `commands.tags.error.unknownTitle` across all 7 langs. Correct — beh review already marked that key unused. Note beh `langKeys.ts` still lists it until `generate:langKeys` runs against this RP. **Cleared:** `events.beforeChat.format` bracket strip. Intentional with beh `beforeChat.ts` — title glyph + ranks are built into the first `%s`; the lang string must not wrap it again. Prior inline on that line is obsolete context. **Still blocking:** `mergeable=false` vs current `main`. Rebase before merge. Do not merge res ahead of beh#722. **Still open (no new inline):** `titles_server_form.json` `grid_dimensions: [2, 14]` hard-caps 28 cells. Catalog is 25 titles today, so it fits, but anything past 28 silently clips. Prefer binding rows from collection length or document the cap with BEH. **Nit:** title strings remain English in non-EN locales. Human reviews still stale (`ffce79e` REQUEST_CHANGES, `9d2fabb` APPROVE).
Member

CI build-and-test failed on head dfb0c8f25188 (run 142).

Likely cause: npm run syncLangFiles -- --check. Locale files are not en_US-ordered/byte-sync (e.g. es_ES key order around events.beforeChat.*; other langs trailing blank-line drift). Fix: run npm run syncLangFiles on the branch and push. Do not merge until green + rebase onto current main (still mergeable=false).

**CI** `build-and-test` failed on head `dfb0c8f25188` ([run 142](https://git.pokebedrock.com/pokebedrock/pokebedrock-res/actions/runs/142)). Likely cause: `npm run syncLangFiles -- --check`. Locale files are not en_US-ordered/byte-sync (e.g. `es_ES` key order around `events.beforeChat.*`; other langs trailing blank-line drift). Fix: run `npm run syncLangFiles` on the branch and push. Do not merge until green + rebase onto current `main` (still `mergeable=false`).
planet force-pushed feature/title-update from dfb0c8f251
Some checks failed
Model Preview / model-preview (pull_request) Successful in 10s
CI / build-and-test (20.x) (pull_request) Failing after 19s
to c539085c39
Some checks failed
Model Preview / model-preview (pull_request) Successful in 10s
CI / build-and-test (20.x) (pull_request) Failing after 19s
2026-09-06 22:35:30 +00:00
Compare
bot requested changes 2026-09-06 22:39:33 +00:00
Dismissed
bot left a comment

Re-review head c539085c3980 (was dfb0c8f25188). Companion beh#722 still open — do not merge res alone.

This push: adds commands.tags.error.alreadyGranted / success.revoked / error.notGranted across all 7 langs (matches commit message). mergeable=true vs current main — prior unmergeable hold is cleared.

Still blocking — CI: ci.yml run 144 failure on this head (model-preview #145 success is not the gate). Same class of failure as dfb0c8f: locale files are not syncLangFiles --check clean.

Still blocking — lang hygiene: non-EN files still append a second ## Emoji Update block + mojibake (┬º / broken UTF-8) after PHUD. pt_BR still has two events.beforeChat.format lines; first is the old bracketed form. Repo keeps first value on duplicates, so Portuguese chat format will not pick up the title-aware string. Fix: npm run syncLangFiles on the branch (drop tails, restore §, en_US order) and push.

Still open: titles_server_form.json grid_dimensions: [2, 14] hard-caps 28 cells.

Nit: title strings remain English in non-EN locales; tip also dropped a few unrelated crate/tera keys into en_US — keep the PR title-scoped.

Human reviews still stale (ffce79e REQUEST_CHANGES, 9d2fabb APPROVE). Open res issues: 0 (no Fixes links).

Re-review head `c539085c3980` (was `dfb0c8f25188`). Companion [beh#722](https://git.pokebedrock.com/pokebedrock/pokebedrock-beh/pulls/722) still open — do not merge res alone. **This push:** adds `commands.tags.error.alreadyGranted` / `success.revoked` / `error.notGranted` across all 7 langs (matches commit message). `mergeable=true` vs current `main` — prior unmergeable hold is cleared. **Still blocking — CI:** `ci.yml` [run 144](https://git.pokebedrock.com/pokebedrock/pokebedrock-res/actions/runs/144) **failure** on this head (model-preview #145 success is not the gate). Same class of failure as dfb0c8f: locale files are not `syncLangFiles --check` clean. **Still blocking — lang hygiene:** non-EN files still append a second `## Emoji Update` block + mojibake (`┬º` / broken UTF-8) after PHUD. `pt_BR` still has **two** `events.beforeChat.format` lines; first is the old bracketed form. Repo keeps first value on duplicates, so Portuguese chat format will not pick up the title-aware string. Fix: `npm run syncLangFiles` on the branch (drop tails, restore `§`, en_US order) and push. **Still open:** `titles_server_form.json` `grid_dimensions: [2, 14]` hard-caps 28 cells. **Nit:** title strings remain English in non-EN locales; tip also dropped a few unrelated crate/tera keys into `en_US` — keep the PR title-scoped. Human reviews still stale (`ffce79e` REQUEST_CHANGES, `9d2fabb` APPROVE). Open res issues: 0 (no Fixes links).
texts/cs_CZ.lang Outdated
@ -11657,3 +11657,36 @@ commands.spawnrate.success=Nastavení výskytu aktualizováno.
## PHUD
phud.playerPing.label=Aktuální ping:
## Emoji Update
Member

Trailing re-append after PHUD: second ## Emoji Update block + mojibake (┬º / mangled Czech). Same pattern in de/es/ru/zh/pt. This is what keeps syncLangFiles --check / CI red. Run npm run syncLangFiles and push — do not hand-append keys.

Trailing re-append after PHUD: second `## Emoji Update` block + mojibake (`┬º` / mangled Czech). Same pattern in de/es/ru/zh/pt. This is what keeps `syncLangFiles --check` / CI red. Run `npm run syncLangFiles` and push — do not hand-append keys.
texts/pt_BR.lang Outdated
@ -3974,6 +3974,26 @@ events.beforeChat.error.tooFast=§cVocê está enviando mensagens muito rapidame
events.beforeChat.error.tooManyEmojis=§cA maximum of %s emojis can be sent in a single message.
events.beforeChat.error.zalgo=§cSua mensagem contém zalgo e não pode ser enviada!
events.beforeChat.format=§r§l§8[§r%s§r§l§8]§r§7 %s:§r %s
Member

First events.beforeChat.format is still the old bracketed string; a second corrected copy appears ~20 lines later. This repo keeps the first value on duplicate keys, so pt_BR will not use the title-aware format. Delete the old line (and the mid-file mojibake restart block) — prefer npm run syncLangFiles.

First `events.beforeChat.format` is still the old bracketed string; a second corrected copy appears ~20 lines later. This repo keeps the **first** value on duplicate keys, so pt_BR will not use the title-aware format. Delete the old line (and the mid-file mojibake restart block) — prefer `npm run syncLangFiles`.
@ -0,0 +38,4 @@
"size": ["100%", 182],
"offset": [2, 0],
"anchor_from": "top_left",
"anchor_to": "top_left",
Member

Still open: grid_dimensions: [2, 14] hard-caps 28 cells. Catalog fits today (~25), but growth past 28 silently clips. Prefer binding rows from collection length or document the cap with beh#722.

Still open: `grid_dimensions: [2, 14]` hard-caps 28 cells. Catalog fits today (~25), but growth past 28 silently clips. Prefer binding rows from collection length or document the cap with beh#722.
feat: add still-owned title revoke warning key
Some checks failed
Model Preview / model-preview (pull_request) Successful in 10s
CI / build-and-test (20.x) (pull_request) Failing after 20s
225f5df337
bot requested changes 2026-09-07 01:59:50 +00:00
Dismissed
bot left a comment

Re-review head 225f5df337ab (was c539085c3980). Companion beh#722 still open — do not merge res alone.

This push: adds commands.tags.warning.stillOwned across all 7 langs. Correct pairing for the still-owned revoke path; leave it.

Still blocking — CI: ci.yml run 146 failure on this head (CI / build-and-test). Model-preview #147 success is not the gate. Same syncLangFiles --check class as c539085c.

Still blocking — lang hygiene (existing threads; unchanged by this push):

  • Non-EN files still append a second ## Emoji Update block + mojibake (┬º / §) after PHUD.
  • pt_BR still has two events.beforeChat.format lines; first (bracketed) wins.
  • Fix: npm run syncLangFiles on the branch and push — do not hand-append more keys.

Still open: titles_server_form.json grid_dimensions: [2, 14] hard-caps 28 cells.

Nit: title strings remain English in non-EN locales.

mergeable=true vs current main. Open res issues: 0 (no Fixes links). Human reviews still stale.

Re-review head `225f5df337ab` (was `c539085c3980`). Companion [beh#722](https://git.pokebedrock.com/pokebedrock/pokebedrock-beh/pulls/722) still open — do not merge res alone. **This push:** adds `commands.tags.warning.stillOwned` across all 7 langs. Correct pairing for the still-owned revoke path; leave it. **Still blocking — CI:** `ci.yml` [run 146](https://git.pokebedrock.com/pokebedrock/pokebedrock-res/actions/runs/146) **failure** on this head (`CI / build-and-test`). Model-preview #147 success is not the gate. Same `syncLangFiles --check` class as c539085c. **Still blocking — lang hygiene** (existing threads; unchanged by this push): - Non-EN files still append a second `## Emoji Update` block + mojibake (`┬º` / `§`) after PHUD. - `pt_BR` still has two `events.beforeChat.format` lines; first (bracketed) wins. - Fix: `npm run syncLangFiles` on the branch and push — do not hand-append more keys. **Still open:** `titles_server_form.json` `grid_dimensions: [2, 14]` hard-caps 28 cells. **Nit:** title strings remain English in non-EN locales. `mergeable=true` vs current `main`. Open res issues: 0 (no Fixes links). Human reviews still stale.
planet force-pushed feature/title-update from 225f5df337
Some checks failed
Model Preview / model-preview (pull_request) Successful in 10s
CI / build-and-test (20.x) (pull_request) Failing after 20s
to 801551c0f1
Some checks failed
Model Preview / model-preview (pull_request) Successful in 14s
CI / build-and-test (20.x) (pull_request) Failing after 27s
2026-09-09 06:24:20 +00:00
Compare
bot requested changes 2026-09-09 06:33:50 +00:00
Dismissed
bot left a comment

Re-review head 801551c0f1b1 (was 225f5df337ab). Looks like a force-push/rebase rewrite of the same tip series — tip still “stillOwned” key. Companion beh#722 still open — do not merge res alone.

This push: no material content fix vs prior RC. UI wiring ($flag_titles / titles@titles.main_panel), glyph sheet, and title keys are still the right shape.

Still blocking — CI: ci.yml run 166 failure on this head (CI / build-and-test). Model-preview #167 success is not the gate. Same syncLangFiles --check class as 225f5df.

Still blocking — lang hygiene (unchanged by this rewrite):

  • Non-EN files still append a second ## Emoji Update block + mojibake (┬º / §) after PHUD.
  • pt_BR still has two events.beforeChat.format lines; first (bracketed) wins, so Portuguese chat won’t pick up the title-aware string.
  • Fix: npm run syncLangFiles on the branch and push — do not hand-append more keys.

Still open: titles_server_form.json grid_dimensions: [2, 14] hard-caps 28 cells.

Nit: title strings remain English in non-EN locales; en_US still picks up unrelated serverDrop / chunkUnloadDrop keys.

mergeable=true vs current main. Open res issues: 0 (no Fixes links). Human reviews still stale.

Re-review head `801551c0f1b1` (was `225f5df337ab`). Looks like a force-push/rebase rewrite of the same tip series — tip still “stillOwned” key. Companion [beh#722](https://git.pokebedrock.com/pokebedrock/pokebedrock-beh/pulls/722) still open — do not merge res alone. **This push:** no material content fix vs prior RC. UI wiring (`$flag_titles` / `titles@titles.main_panel`), glyph sheet, and title keys are still the right shape. **Still blocking — CI:** `ci.yml` [run 166](https://git.pokebedrock.com/pokebedrock/pokebedrock-res/actions/runs/166) **failure** on this head (`CI / build-and-test`). Model-preview #167 success is not the gate. Same `syncLangFiles --check` class as 225f5df. **Still blocking — lang hygiene** (unchanged by this rewrite): - Non-EN files still append a second `## Emoji Update` block + mojibake (`┬º` / `§`) after PHUD. - `pt_BR` still has two `events.beforeChat.format` lines; first (bracketed) wins, so Portuguese chat won’t pick up the title-aware string. - Fix: `npm run syncLangFiles` on the branch and push — do not hand-append more keys. **Still open:** `titles_server_form.json` `grid_dimensions: [2, 14]` hard-caps 28 cells. **Nit:** title strings remain English in non-EN locales; `en_US` still picks up unrelated `serverDrop` / `chunkUnloadDrop` keys. `mergeable=true` vs current `main`. Open res issues: 0 (no Fixes links). Human reviews still stale.
texts/cs_CZ.lang Outdated
@ -11658,3 +11658,37 @@ commands.spawnrate.success=Nastavení výskytu aktualizováno.
## PHUD
phud.playerPing.label=Aktuální ping:
## Emoji Update
Member

Blocker: trailing second ## Emoji Update dump + mojibake (┬º). Same in de/es/ru/zh. npm run syncLangFiles — don’t append tails by hand.

**Blocker:** trailing second `## Emoji Update` dump + mojibake (`┬º`). Same in de/es/ru/zh. `npm run syncLangFiles` — don’t append tails by hand.
texts/pt_BR.lang Outdated
@ -3978,0 +3994,4 @@
events.beforeChat.error.parseFailed=§cFalha ao analisar seus dados!
events.beforeChat.error.tooFast=§cVocê está enviando mensagens muito rapidamente. Por favor, diminua a velocidade e tente novamente.
events.beforeChat.error.zalgo=§cSua mensagem contém zalgo e não pode ser enviada!
events.beforeChat.format=§r%s§r§7 %s:§r %s
Member

Blocker: second events.beforeChat.format — the bracketed line above still wins. Also the trailing second ## Emoji Update + § / ┬º mojibake will fail syncLangFiles --check. Run npm run syncLangFiles and push; don’t hand-append.

**Blocker:** second `events.beforeChat.format` — the bracketed line above still wins. Also the trailing second `## Emoji Update` + `§` / `┬º` mojibake will fail `syncLangFiles --check`. Run `npm run syncLangFiles` and push; don’t hand-append.
@ -0,0 +39,4 @@
"offset": [2, 0],
"anchor_from": "top_left",
"anchor_to": "top_left",
"grid_dimensions": [2, 14],
Member

Still open: grid_dimensions: [2, 14] hard-caps 28 cells. Catalog is ~25 today so it fits, but anything past 28 silently clips. Prefer binding rows from collection length, or document the cap with BEH.

**Still open:** `grid_dimensions: [2, 14]` hard-caps 28 cells. Catalog is ~25 today so it fits, but anything past 28 silently clips. Prefer binding rows from collection length, or document the cap with BEH.
planet force-pushed feature/title-update from 801551c0f1
Some checks failed
Model Preview / model-preview (pull_request) Successful in 14s
CI / build-and-test (20.x) (pull_request) Failing after 27s
to 12734a7962
Some checks failed
Model Preview / model-preview (pull_request) Successful in 12s
CI / build-and-test (20.x) (pull_request) Failing after 19s
2026-09-10 00:28:07 +00:00
Compare
bot requested changes 2026-09-10 00:32:13 +00:00
Dismissed
bot left a comment

Re-review head 12734a796237 (was 801551c0f1b1). Force-push/rebase onto main e906936b (post findable-blocks). Companion beh#722 still open — do not merge res alone.

This push: tip still “stillOwned” key series, but branch is now rebased cleanly onto current main. mergeable=true.

Cleared — lang hygiene: all 7 locales now match en_US key order (11571 keys), single events.beforeChat.format, single ## Emoji Update / ## Titles, no mojibake / duplicate-format tails. Prior syncLangFiles-class blockers from older heads look fixed on this tip. Unrelated serverDrop / chunkUnloadDrop drift is no longer part of the Titles append.

Still blocking — companion: keep res behind beh#722.

Still blocking — CI: ci.yml run 179 failure on this head (Validate format versions; lang-sync + tests skipped). Model-preview #180 success is not the gate. Same step is red on main itself (run 176 after #4 merge) — PR does not touch models/ / entity/ / other format-version scan dirs, so this looks inherited from base, not introduced here. Still: do not claim merge-ready while the gate is red.

Still open: titles_server_form.json grid_dimensions: [2, 14] hard-caps 28 cells (prior threads).

Nit: title strings remain English in non-EN locales.

UI wiring ($flag_titles / titles@titles.main_panel), glyph sheet, and title keys remain the right shape. Open res issues: 0 (no Fixes links). Human reviews still stale.

Re-review head `12734a796237` (was `801551c0f1b1`). Force-push/rebase onto main `e906936b` (post findable-blocks). Companion [beh#722](https://git.pokebedrock.com/pokebedrock/pokebedrock-beh/pulls/722) still open — do not merge res alone. **This push:** tip still “stillOwned” key series, but branch is now rebased cleanly onto current `main`. `mergeable=true`. **Cleared — lang hygiene:** all 7 locales now match `en_US` key order (11571 keys), single `events.beforeChat.format`, single `## Emoji Update` / `## Titles`, no mojibake / duplicate-format tails. Prior syncLangFiles-class blockers from older heads look fixed on this tip. Unrelated `serverDrop` / `chunkUnloadDrop` drift is no longer part of the Titles append. **Still blocking — companion:** keep res behind beh#722. **Still blocking — CI:** `ci.yml` [run 179](https://git.pokebedrock.com/pokebedrock/pokebedrock-res/actions/runs/179) **failure** on this head (`Validate format versions`; lang-sync + tests skipped). Model-preview #180 success is not the gate. Same step is red on `main` itself ([run 176](https://git.pokebedrock.com/pokebedrock/pokebedrock-res/actions/runs/176) after #4 merge) — PR does not touch `models/` / `entity/` / other format-version scan dirs, so this looks inherited from base, not introduced here. Still: do not claim merge-ready while the gate is red. **Still open:** `titles_server_form.json` `grid_dimensions: [2, 14]` hard-caps 28 cells (prior threads). **Nit:** title strings remain English in non-EN locales. UI wiring (`$flag_titles` / `titles@titles.main_panel`), glyph sheet, and title keys remain the right shape. Open res issues: 0 (no Fixes links). Human reviews still stale.
@ -0,0 +39,4 @@
"offset": [2, 0],
"anchor_from": "top_left",
"anchor_to": "top_left",
"grid_dimensions": [2, 14],
Member

Still open from prior reviews: grid_dimensions: [2, 14] hard-caps 28 cells. Catalog fits today (~25), but anything past 28 silently clips. Prefer binding rows from collection length, or document the cap with BEH.

Still open from prior reviews: `grid_dimensions: [2, 14]` hard-caps 28 cells. Catalog fits today (~25), but anything past 28 silently clips. Prefer binding rows from collection length, or document the cap with BEH.
fix: raise titles grid cap to 40 cells
Some checks failed
Model Preview / model-preview (pull_request) Successful in 10s
CI / build-and-test (20.x) (pull_request) Failing after 20s
fbee22adfc
bot requested changes 2026-09-10 01:25:25 +00:00
Dismissed
bot left a comment

Re-review head fbee22adfc4b (was 12734a796237). Tip: fix: raise titles grid cap to 40 cells. Companion beh#722 still open — do not merge res alone.

This push: titles_server_form.json grid_dimensions [2, 14] → [2, 20] (28 → 40 cells) and scroll content height 182 → 260. Clears the prior hard-cap finding for the current ~25-title catalog with headroom. Still a fixed cap (not collection-length binding); fine for now — document or raise again if the catalog grows past 40.

Still blocking — companion: keep res behind beh#722.

Still blocking — CI: ci.yml run 181 failure on this head (model-preview #182 success is not the gate). Same class as prior tip / main itself (run 176 after #4 merge) — PR does not touch format-version scan dirs, so inherited from base. Do not claim merge-ready while the gate is red.

Nit: title strings remain English in non-EN locales.

UI wiring / glyph sheet / lang hygiene from prior tip unchanged and still look right. mergeable=true. Open res issues: 0 (no Fixes links). Human reviews still stale.

Re-review head `fbee22adfc4b` (was `12734a796237`). Tip: `fix: raise titles grid cap to 40 cells`. Companion [beh#722](https://git.pokebedrock.com/pokebedrock/pokebedrock-beh/pulls/722) still open — do not merge res alone. **This push:** `titles_server_form.json` `grid_dimensions` `[2, 14]` → `[2, 20]` (28 → 40 cells) and scroll content height `182` → `260`. Clears the prior hard-cap finding for the current ~25-title catalog with headroom. Still a fixed cap (not collection-length binding); fine for now — document or raise again if the catalog grows past 40. **Still blocking — companion:** keep res behind beh#722. **Still blocking — CI:** `ci.yml` [run 181](https://git.pokebedrock.com/pokebedrock/pokebedrock-res/actions/runs/181) **failure** on this head (model-preview #182 success is not the gate). Same class as prior tip / `main` itself ([run 176](https://git.pokebedrock.com/pokebedrock/pokebedrock-res/actions/runs/176) after #4 merge) — PR does not touch format-version scan dirs, so inherited from base. Do not claim merge-ready while the gate is red. **Nit:** title strings remain English in non-EN locales. UI wiring / glyph sheet / lang hygiene from prior tip unchanged and still look right. `mergeable=true`. Open res issues: 0 (no Fixes links). Human reviews still stale.
@ -0,0 +39,4 @@
"offset": [2, 0],
"anchor_from": "top_left",
"anchor_to": "top_left",
"grid_dimensions": [2, 20],
Member

Cleared prior 28-cell hard-cap note: [2, 20] = 40 cells + taller scroll (260). Good for ~25 titles today. Still a fixed grid — bump again or bind to collection length if the catalog outgrows 40.

Cleared prior 28-cell hard-cap note: `[2, 20]` = 40 cells + taller scroll (`260`). Good for ~25 titles today. Still a fixed grid — bump again or bind to collection length if the catalog outgrows 40.
planet marked this conversation as resolved
blocks.json Outdated
@ -429,6 +429,9 @@
"pokeb:yago_berry_plant": {
"sound": "grass"
},
"pokeb:findable_block": {
Member

??? why here

??? why here
planet marked this conversation as resolved
@ -0,0 +1,250 @@
{
Member

-_-

-_-
planet marked this conversation as resolved
planet force-pushed feature/title-update from fbee22adfc
Some checks failed
Model Preview / model-preview (pull_request) Successful in 10s
CI / build-and-test (20.x) (pull_request) Failing after 20s
to 0b88af267f
Some checks failed
Model Preview / model-preview (pull_request) Successful in 11s
CI / build-and-test (20.x) (pull_request) Failing after 21s
2026-09-17 17:26:34 +00:00
Compare
add title slot lang key
Some checks failed
Model Preview / model-preview (pull_request) Successful in 14s
CI / build-and-test (20.x) (pull_request) Failing after 22s
997437ac56
planet force-pushed feature/title-update from 997437ac56
Some checks failed
Model Preview / model-preview (pull_request) Successful in 14s
CI / build-and-test (20.x) (pull_request) Failing after 22s
to caa3d77719
Some checks failed
Model Preview / model-preview (pull_request) Successful in 12s
CI / build-and-test (20.x) (pull_request) Failing after 20s
2026-09-25 00:14:25 +00:00
Compare
Merge branch 'main' into feature/title-update
All checks were successful
CI / build-and-test (20.x) (pull_request) Successful in 28s
Model Preview / model-preview (pull_request) Successful in 3m26s
7758c17bea
bot requested changes 2026-10-01 18:52:36 +00:00
Dismissed
bot left a comment

Re-review head 7758c17bead8 (was fbee22adfc4b / prior tip before main merge). Tip: Merge branch 'main' into feature/title-update (brings in 222b71bf v2.22.5 + 5a2ff211 format-version / Chrome runner fix). Companion beh#722 still open — do not merge res alone.

This push: merge-from-main only. Title scope unchanged vs prior tip — same UI wiring ($flag_titles / titles@titles.main_panel), glyph sheet, lang Titles block, grid_dimensions: [2, 20] (40 cells). No new title-feature delta to re-litigate.

Cleared — CI format-version gate: ci.yml run 321 success on this head (inherited main failure from older tips is gone after 5a2ff211). Model-preview run 322 was still running at review time and is not the merge gate.

Still blocking — companion: keep res behind beh#722.

Still clear (no new threads): lang hygiene remains clean (single Titles append, single events.beforeChat.format, no mojibake / duplicate-format tails). Grid cap finding stays cleared at 40 cells.

Nit: title strings remain English in non-EN locales.

mergeable=true vs current main (5a2ff211). Open res issues: 0 (no Fixes links). Human reviews still stale on older SHAs. Not merge-ready until companion lands.

Re-review head `7758c17bead8` (was `fbee22adfc4b` / prior tip before main merge). Tip: `Merge branch 'main' into feature/title-update` (brings in `222b71bf` v2.22.5 + `5a2ff211` format-version / Chrome runner fix). Companion [beh#722](https://git.pokebedrock.com/pokebedrock/pokebedrock-beh/pulls/722) still **open** — do not merge res alone. **This push:** merge-from-main only. Title scope unchanged vs prior tip — same UI wiring (`$flag_titles` / `titles@titles.main_panel`), glyph sheet, lang Titles block, `grid_dimensions: [2, 20]` (40 cells). No new title-feature delta to re-litigate. **Cleared — CI format-version gate:** `ci.yml` [run 321](https://git.pokebedrock.com/pokebedrock/pokebedrock-res/actions/runs/321) **success** on this head (inherited main failure from older tips is gone after `5a2ff211`). Model-preview [run 322](https://git.pokebedrock.com/pokebedrock/pokebedrock-res/actions/runs/322) was still running at review time and is not the merge gate. **Still blocking — companion:** keep res behind beh#722. **Still clear (no new threads):** lang hygiene remains clean (single Titles append, single `events.beforeChat.format`, no mojibake / duplicate-format tails). Grid cap finding stays cleared at 40 cells. **Nit:** title strings remain English in non-EN locales. `mergeable=true` vs current `main` (`5a2ff211`). Open res issues: 0 (no Fixes links). Human reviews still stale on older SHAs. Not merge-ready until companion lands.
bot requested changes 2026-10-01 19:00:06 +00:00
Dismissed
bot left a comment

Re-review head 7758c17bead8 (was fbee22adfc4b). Tip: merge main (5a2ff211 format-version fix) + prior commands.titles.* / worldchampion / noFreeSlot commits. Companion beh#722 still open — do not merge res alone.

This push: merges current main onto feature/title-update. Material vs last bot tip also includes (already on branch before this merge): commands.tags.* → commands.titles.*, world_champion.png → worldchampion.png, forms.titles.error.noFreeSlot. Aligns with beh#722 langKeys + title id worldchampion.

Cleared — CI: ci.yml run 321 success on this head (CI / build-and-test). Prior format-version gate inherited from main is green after 5a2ff211. Model-preview #322 still running — not the gate.

Cleared (unchanged): lang hygiene (all 7 locales key-order match en_US, 11630 keys, single events.beforeChat.format, no mojibake / leftover commands.tags.*); grid [2, 20] / height 260; textures cover all 25 beh title ids.

Still blocking — companion: keep res behind beh#722.

Nit: title strings remain English in non-EN locales.

UI wiring ($flag_titles / titles@titles.main_panel), glyph sheet, and title keys remain the right shape. mergeable=true. Open res issues: 0 (no Fixes links). Human reviews still stale.

Re-review head `7758c17bead8` (was `fbee22adfc4b`). Tip: merge `main` (`5a2ff211` format-version fix) + prior `commands.titles.*` / `worldchampion` / `noFreeSlot` commits. Companion [beh#722](https://git.pokebedrock.com/pokebedrock/pokebedrock-beh/pulls/722) still open — do not merge res alone. **This push:** merges current `main` onto `feature/title-update`. Material vs last bot tip also includes (already on branch before this merge): `commands.tags.*` → `commands.titles.*`, `world_champion.png` → `worldchampion.png`, `forms.titles.error.noFreeSlot`. Aligns with beh#722 `langKeys` + title id `worldchampion`. **Cleared — CI:** `ci.yml` [run 321](https://git.pokebedrock.com/pokebedrock/pokebedrock-res/actions/runs/321) **success** on this head (`CI / build-and-test`). Prior format-version gate inherited from `main` is green after `5a2ff211`. Model-preview #322 still running — not the gate. **Cleared (unchanged):** lang hygiene (all 7 locales key-order match `en_US`, 11630 keys, single `events.beforeChat.format`, no mojibake / leftover `commands.tags.*`); grid `[2, 20]` / height 260; textures cover all 25 beh title ids. **Still blocking — companion:** keep res behind beh#722. **Nit:** title strings remain English in non-EN locales. UI wiring (`$flag_titles` / `titles@titles.main_panel`), glyph sheet, and title keys remain the right shape. `mergeable=true`. Open res issues: 0 (no Fixes links). Human reviews still stale.
Point model preview at the Forgejo copy of the action.
All checks were successful
CI / build-and-test (20.x) (pull_request) Successful in 32s
Model Preview / model-preview (pull_request) Successful in 1m20s
b7d06fdf66
bot requested changes 2026-10-01 19:10:08 +00:00
Dismissed
bot left a comment

Re-review head b7d06fdf66a7 (was 7758c17bead8). Tip: Point model preview at the Forgejo copy of the action. Companion beh#722 still open — do not merge res alone.

This push: .github/workflows/model-preview.yml only — uses: smell-of-curry/mc-model-preview@latest → https://git.pokebedrock.com/pokebedrock/mc-model-preview@b10a8ffcf825203aa537c56ce48531249bf1ca6e (Forgejo copy, SHA-pinned). Correct for self-hosted Actions; @latest on the old GitHub path was fragile here. Verified pokebedrock/mc-model-preview + commit b10a8ffc exist. No title-UI / lang / texture delta.

Still blocking — companion: keep res behind beh#722.

CI: ci.yml + model-preview.yml both pending on this head at review time (not settled yet). Prior tip 7758c17b had ci.yml green after main 5a2ff211; this one-line uses: swap should not regress build-and-test — confirm green before merge. Model-preview is not the merge gate.

Still clear (no new threads): lang hygiene, grid_dimensions: [2, 20] / height 260, UI wiring ($flag_titles / titles@titles.main_panel), glyph sheet / title keys.

Nit: title strings remain English in non-EN locales.

mergeable=true vs current main. Open res issues: 0 (no Fixes links). Human reviews still stale on older SHAs. Not merge-ready until companion lands.

Re-review head `b7d06fdf66a7` (was `7758c17bead8`). Tip: `Point model preview at the Forgejo copy of the action.` Companion [beh#722](https://git.pokebedrock.com/pokebedrock/pokebedrock-beh/pulls/722) still **open** — do not merge res alone. **This push:** `.github/workflows/model-preview.yml` only — `uses:` `smell-of-curry/mc-model-preview@latest` → `https://git.pokebedrock.com/pokebedrock/mc-model-preview@b10a8ffcf825203aa537c56ce48531249bf1ca6e` (Forgejo copy, SHA-pinned). Correct for self-hosted Actions; `@latest` on the old GitHub path was fragile here. Verified `pokebedrock/mc-model-preview` + commit `b10a8ffc` exist. No title-UI / lang / texture delta. **Still blocking — companion:** keep res behind beh#722. **CI:** `ci.yml` + `model-preview.yml` both **pending** on this head at review time (not settled yet). Prior tip `7758c17b` had `ci.yml` green after main `5a2ff211`; this one-line `uses:` swap should not regress build-and-test — confirm green before merge. Model-preview is not the merge gate. **Still clear (no new threads):** lang hygiene, `grid_dimensions: [2, 20]` / height 260, UI wiring (`$flag_titles` / `titles@titles.main_panel`), glyph sheet / title keys. **Nit:** title strings remain English in non-EN locales. `mergeable=true` vs current `main`. Open res issues: 0 (no Fixes links). Human reviews still stale on older SHAs. Not merge-ready until companion lands.
bot requested changes 2026-10-01 19:11:03 +00:00
Dismissed
bot left a comment

Re-review head b7d06fdf66a7 (was 7758c17bead8). Tip: Point model preview at the Forgejo copy of the action. Companion beh#722 still open — do not merge res alone.

This push: only .github/workflows/model-preview.yml (smell-of-curry/mc-model-preview@latest → Forgejo pokebedrock/mc-model-preview@b10a8ffc…). Blob matches current main — no title-feature delta vs prior tip. UI wiring ($flag_titles / titles@titles.main_panel), glyph sheet, commands.titles.* / worldchampion.png, grid [2, 20] / height 260 unchanged.

Still blocking — companion: keep res behind beh#722.

CI: ci.yml run 353 + model-preview run 354 still pending (waiting) on this head. Prior tip had build-and-test green after 5a2ff211; not calling merge-ready while the new gate is queued.

Cleared (unchanged): lang hygiene (Titles append, single events.beforeChat.format, no mojibake / leftover commands.tags.*); grid cap at 40 cells.

Nits: PR body empty; title uses feat; (semicolon). Title strings still English in non-EN locales.

mergeable=true vs main (9a5af785). Open res issues: 0 (no Fixes links). Human reviews still stale on older SHAs.

Re-review head `b7d06fdf66a7` (was `7758c17bead8`). Tip: `Point model preview at the Forgejo copy of the action.` Companion [beh#722](https://git.pokebedrock.com/pokebedrock/pokebedrock-beh/pulls/722) still **open** — do not merge res alone. **This push:** only `.github/workflows/model-preview.yml` (`smell-of-curry/mc-model-preview@latest` → Forgejo `pokebedrock/mc-model-preview@b10a8ffc…`). Blob matches current `main` — no title-feature delta vs prior tip. UI wiring (`$flag_titles` / `titles@titles.main_panel`), glyph sheet, `commands.titles.*` / `worldchampion.png`, grid `[2, 20]` / height 260 unchanged. **Still blocking — companion:** keep res behind beh#722. **CI:** `ci.yml` [run 353](https://git.pokebedrock.com/pokebedrock/pokebedrock-res/actions/runs/353) + model-preview [run 354](https://git.pokebedrock.com/pokebedrock/pokebedrock-res/actions/runs/354) still **pending** (waiting) on this head. Prior tip had build-and-test green after `5a2ff211`; not calling merge-ready while the new gate is queued. **Cleared (unchanged):** lang hygiene (Titles append, single `events.beforeChat.format`, no mojibake / leftover `commands.tags.*`); grid cap at 40 cells. **Nits:** PR body empty; title uses `feat;` (semicolon). Title strings still English in non-EN locales. `mergeable=true` vs `main` (`9a5af785`). Open res issues: 0 (no Fixes links). Human reviews still stale on older SHAs.
revert: restore original titles json ui and gui texture
All checks were successful
CI / build-and-test (20.x) (pull_request) Successful in 30s
Model Preview / model-preview (pull_request) Successful in 1m3s
6940f6d0eb
bot requested changes 2026-10-05 09:24:30 +00:00
Dismissed
bot left a comment

Re-review head 6940f6d0eb58 (was b7d06fdf66a7). Tip: revert: restore original titles json ui and gui texture. Companion beh#722 still open (head 12b5ec76) — do not merge res alone.

This push: ui/titles_server_form.json back to grid_dimensions: [2, 14] / scroll height 182 (reverts b165fd43), and textures/ui/titles/titles_gui.png swapped back to the original asset (same 206×186, different pixels). Reads as an intentional author revert; no lang / glyph / title-texture delta.

Non-blocking — grid cap is back to 28 cells. beh#722 TITLES has 25 ids today and the form only lists owned titles, so it fits with 3 to spare. A 29th title will silently clip. Fine to ship; whoever adds titles past 28 must bump this (inline).

Still blocking — companion: keep res behind beh#722.

CI: CI / build-and-test run 368 success on this head. Model-preview run 369 pending, not the gate.

Cleared (unchanged): lang hygiene, UI wiring ($flag_titles / titles@titles.main_panel), glyph sheet, commands.titles.* / worldchampion.png.

Nits: PR body empty; title uses feat; (semicolon); title strings still English in non-EN locales.

mergeable=true vs main (9a5af785). Open res issues: 0 (no Fixes links).

Re-review head `6940f6d0eb58` (was `b7d06fdf66a7`). Tip: `revert: restore original titles json ui and gui texture`. Companion [beh#722](https://git.pokebedrock.com/pokebedrock/pokebedrock-beh/pulls/722) still **open** (head `12b5ec76`) — do not merge res alone. **This push:** `ui/titles_server_form.json` back to `grid_dimensions: [2, 14]` / scroll height `182` (reverts `b165fd43`), and `textures/ui/titles/titles_gui.png` swapped back to the original asset (same 206×186, different pixels). Reads as an intentional author revert; no lang / glyph / title-texture delta. **Non-blocking — grid cap is back to 28 cells.** beh#722 `TITLES` has 25 ids today and the form only lists owned titles, so it fits with 3 to spare. A 29th title will silently clip. Fine to ship; whoever adds titles past 28 must bump this (inline). **Still blocking — companion:** keep res behind beh#722. **CI:** `CI / build-and-test` [run 368](https://git.pokebedrock.com/pokebedrock/pokebedrock-res/actions/runs/368) **success** on this head. Model-preview [run 369](https://git.pokebedrock.com/pokebedrock/pokebedrock-res/actions/runs/369) pending, not the gate. **Cleared (unchanged):** lang hygiene, UI wiring (`$flag_titles` / `titles@titles.main_panel`), glyph sheet, `commands.titles.*` / `worldchampion.png`. **Nits:** PR body empty; title uses `feat;` (semicolon); title strings still English in non-EN locales. `mergeable=true` vs `main` (`9a5af785`). Open res issues: 0 (no Fixes links).
@ -0,0 +39,4 @@
"offset": [2, 0],
"anchor_from": "top_left",
"anchor_to": "top_left",
"grid_dimensions": [2, 14],
Member

Back to a 28-cell cap (2 × 14). 25 titles in beh#722 today, so it fits — but title #29 clips silently with no error. Not blocking; leave a note for whoever extends TITLES, or bump rows + size height together when that happens.

Back to a 28-cell cap (`2 × 14`). 25 titles in beh#722 today, so it fits — but title #29 clips silently with no error. Not blocking; leave a note for whoever extends `TITLES`, or bump rows + `size` height together when that happens.
planet marked this conversation as resolved
bot requested changes 2026-10-05 09:25:08 +00:00
Dismissed
bot left a comment

Re-review head 6940f6d0eb58 (was b7d06fdf66a7). Tip: revert: restore original titles json ui and gui texture. Companion beh#722 still open (head 12b5ec762f) — do not merge res alone.

This push: ui/titles_server_form.json back to grid_dimensions: [2, 14] / scroll height 182 (40 → 28 cells), plus textures/ui/titles/titles_gui.png swapped back to the original art. Reverts the fbee22ad cap raise.

Re-opened (inline): 28-cell hard cap again. Catalog today is 25 title textures, so it fits with 3 slots of headroom, but title #29+ silently clips. If the revert is intentional (layout/art reasons), fine — say so in the PR and coordinate the cap with BEH so nobody adds titles past 28 without bumping this.

CI: CI / build-and-test success and Model Preview success on this head.

Still clear (unchanged): lang hygiene, UI wiring ($flag_titles / titles@titles.main_panel), glyph sheet, commands.titles.* / worldchampion.png.

Still blocking — companion: keep res behind beh#722.

Nits: PR body empty; title uses feat; (semicolon). Title strings still English in non-EN locales.

mergeable=true vs main (9a5af785). Open res issues: 0 (no Fixes links).

Re-review head `6940f6d0eb58` (was `b7d06fdf66a7`). Tip: `revert: restore original titles json ui and gui texture`. Companion [beh#722](https://git.pokebedrock.com/pokebedrock/pokebedrock-beh/pulls/722) still **open** (head `12b5ec762f`) — do not merge res alone. **This push:** `ui/titles_server_form.json` back to `grid_dimensions: [2, 14]` / scroll height `182` (40 → 28 cells), plus `textures/ui/titles/titles_gui.png` swapped back to the original art. Reverts the `fbee22ad` cap raise. **Re-opened (inline):** 28-cell hard cap again. Catalog today is 25 title textures, so it fits with 3 slots of headroom, but title #29+ silently clips. If the revert is intentional (layout/art reasons), fine — say so in the PR and coordinate the cap with BEH so nobody adds titles past 28 without bumping this. **CI:** `CI / build-and-test` **success** and Model Preview success on this head. **Still clear (unchanged):** lang hygiene, UI wiring (`$flag_titles` / `titles@titles.main_panel`), glyph sheet, `commands.titles.*` / `worldchampion.png`. **Still blocking — companion:** keep res behind beh#722. **Nits:** PR body empty; title uses `feat;` (semicolon). Title strings still English in non-EN locales. `mergeable=true` vs `main` (`9a5af785`). Open res issues: 0 (no Fixes links).
@ -0,0 +39,4 @@
"offset": [2, 0],
"anchor_from": "top_left",
"anchor_to": "top_left",
"grid_dimensions": [2, 14],
Member

Back to [2, 14] = 28 cells (scroll height 182). Fits the current 25 titles, but anything past 28 silently clips. If this revert is intentional, document the cap or bind rows from collection length.

Back to `[2, 14]` = 28 cells (scroll height 182). Fits the current 25 titles, but anything past 28 silently clips. If this revert is intentional, document the cap or bind rows from collection length.
planet marked this conversation as resolved
fix titles menu: hide empty grid cells
All checks were successful
CI / build-and-test (20.x) (pull_request) Successful in 35s
Model Preview / model-preview (pull_request) Successful in 1m27s
598e9e2211
bot requested changes 2026-10-08 18:14:51 +00:00
Dismissed
bot left a comment

Re-review head 598e9e22 (was 6940f6d0). Tip: fix titles menu: hide empty grid cells.

This push: item_button in ui/titles_server_form.json gets a view binding (not (#form_button_text = '')) → #visible. Looks right: every real title button from beh#722's TitlesForm.main has non-empty text (§f<name>\n§7<status>), so only the padding cells disappear. No problems with the change itself.

CI: CI / build-and-test and Model Preview are still pending on this head.

Still blocking — companion: beh#722 is still open (head 12b5ec762f). Don't merge res on its own.

The earlier open threads are unchanged and I'm not repeating them here. mergeable=true vs main (7e6ccf54). Open res issues: 0, so no Fixes links.

Re-review head `598e9e22` (was `6940f6d0`). Tip: `fix titles menu: hide empty grid cells`. **This push:** `item_button` in `ui/titles_server_form.json` gets a view binding `(not (#form_button_text = ''))` → `#visible`. Looks right: every real title button from beh#722's `TitlesForm.main` has non-empty text (`§f<name>\n§7<status>`), so only the padding cells disappear. No problems with the change itself. **CI:** `CI / build-and-test` and Model Preview are still **pending** on this head. **Still blocking — companion:** [beh#722](https://git.pokebedrock.com/pokebedrock/pokebedrock-beh/pulls/722) is still open (head `12b5ec762f`). Don't merge res on its own. The earlier open threads are unchanged and I'm not repeating them here. `mergeable=true` vs `main` (`7e6ccf54`). Open res issues: 0, so no Fixes links.
bot requested changes 2026-10-08 18:15:15 +00:00
Dismissed
bot left a comment

Re-review head 598e9e221181 (was 6940f6d0eb58). Tip: fix titles menu: hide empty grid cells. Companion beh#722 still open (head 70a29448) — do not merge res alone.

This push: one view binding on titles.item_button in ui/titles_server_form.json: #visible = (not (#form_button_text = '')). Fix is correct. beh#722 TitlesForm.main always sends §f<name>\n§7<status> text per owned title, so real buttons never match ''; only the unfilled cells of the fixed [2, 14] grid get hidden, which also kills their stray hover/focus border. banner_image already hid the empty texture, so this was the missing half.

Nit (inline): every other view binding in this file carries "binding_name": "#null". Add it here for consistency.

Still open (non-blocking, prior thread): grid is still a 28-cell hard cap with scroll height 182. Now that empty cells hide, raising grid_dimensions later costs nothing visually, but the fixed height would still clip — bump both together when the catalog passes 28.

Still blocking — companion: keep res behind beh#722.

CI: CI / build-and-test run 386 + model-preview run 387 pending on this head. Not calling merge-ready until the gate settles green.

Unchanged / still clear: lang hygiene, UI wiring ($flag_titles / titles@titles.main_panel), glyph sheet, commands.titles.* / worldchampion.png.

Nits: PR body empty; title uses feat; (semicolon); title strings still English in non-EN locales.

mergeable=true vs main (7e6ccf54). Open res issues: 0 (no Fixes links).

Re-review head `598e9e221181` (was `6940f6d0eb58`). Tip: `fix titles menu: hide empty grid cells`. Companion [beh#722](https://git.pokebedrock.com/pokebedrock/pokebedrock-beh/pulls/722) still **open** (head `70a29448`) — do not merge res alone. **This push:** one view binding on `titles.item_button` in `ui/titles_server_form.json`: `#visible` = `(not (#form_button_text = ''))`. Fix is correct. beh#722 `TitlesForm.main` always sends `§f<name>\n§7<status>` text per owned title, so real buttons never match `''`; only the unfilled cells of the fixed `[2, 14]` grid get hidden, which also kills their stray hover/focus border. `banner_image` already hid the empty texture, so this was the missing half. **Nit (inline):** every other view binding in this file carries `"binding_name": "#null"`. Add it here for consistency. **Still open (non-blocking, prior thread):** grid is still a 28-cell hard cap with scroll height `182`. Now that empty cells hide, raising `grid_dimensions` later costs nothing visually, but the fixed height would still clip — bump both together when the catalog passes 28. **Still blocking — companion:** keep res behind beh#722. **CI:** `CI / build-and-test` [run 386](https://git.pokebedrock.com/pokebedrock/pokebedrock-res/actions/runs/386) + model-preview [run 387](https://git.pokebedrock.com/pokebedrock/pokebedrock-res/actions/runs/387) **pending** on this head. Not calling merge-ready until the gate settles green. **Unchanged / still clear:** lang hygiene, UI wiring (`$flag_titles` / `titles@titles.main_panel`), glyph sheet, `commands.titles.*` / `worldchampion.png`. **Nits:** PR body empty; title uses `feat;` (semicolon); title strings still English in non-EN locales. `mergeable=true` vs `main` (`7e6ccf54`). Open res issues: 0 (no Fixes links).
@ -0,0 +81,4 @@
"binding_collection_name": "form_buttons"
},
{
"binding_type": "view",
Member

Nit: match the rest of this file and add "binding_name": "#null" to this view binding (see banner_image and hover_control). Logic is fine — real title buttons always have text from beh#722, so only filler cells hide.

Nit: match the rest of this file and add `"binding_name": "#null"` to this view binding (see `banner_image` and `hover_control`). Logic is fine — real title buttons always have text from beh#722, so only filler cells hide.
planet marked this conversation as resolved
Member

Follow-up on RC 3099/3100 (head 598e9e22 hasn't changed). This isn't a new review. The earlier findings still stand.

New conflict: against res main b4a8343a (v3.1.1), this branch now conflicts in all 7 texts/*.lang files. It was mergeable when it was reviewed. The cause is an end-of-file append clash: main appended keys after message.rewards.grantError, and this branch appends its ## Titles block there too. I checked a union resolution (keep both blocks): no duplicate keys in en_US or pt_BR. Keep the same block order in every lang so the lang-sync check passes. The split-pack build (#20) has a catch-all, so the new glyph/UI files need no pack mapping.

This still has to merge together with BEH #722, whose data pin (3183 / 13084) needs to land first.

Follow-up on RC 3099/3100 (head `598e9e22` hasn't changed). This isn't a new review. The earlier findings still stand. **New conflict:** against res main `b4a8343a` (v3.1.1), this branch now conflicts in all 7 `texts/*.lang` files. It was mergeable when it was reviewed. The cause is an end-of-file append clash: main appended keys after `message.rewards.grantError`, and this branch appends its `## Titles` block there too. I checked a union resolution (keep both blocks): no duplicate keys in en_US or pt_BR. Keep the same block order in every lang so the lang-sync check passes. The split-pack build (#20) has a catch-all, so the new glyph/UI files need no pack mapping. This still has to merge together with BEH #722, whose data pin (3183 / 13084) needs to land first.
planet force-pushed feature/title-update from 598e9e2211
All checks were successful
CI / build-and-test (20.x) (pull_request) Successful in 35s
Model Preview / model-preview (pull_request) Successful in 1m27s
to eed96697f8
All checks were successful
CI / build-and-test (20.x) (pull_request) Successful in 33s
Model Preview / model-preview (pull_request) Successful in 1m6s
2026-10-17 23:32:02 +00:00
Compare
bot requested changes 2026-10-17 23:33:50 +00:00
bot left a comment

Re-review head eed96697 (was 598e9e22). This push squashes the branch into one commit, feat: title system ui, glyphs and lang, and rebases it onto main b4a8343a (v3.1.1). mergeable=true.

This push: no content change. I checked every non-image file in the PR (7 lang files, ui/_ui_defs.json, ui/server_form.json, ui/titles_server_form.json) against the old head and they're the same. The 27 textures/ui/titles/*.png and font/glyph_E3.png blobs are the same too. The PR diff is now just the title system, so the scope is clean.

Still open from before (not repeating): the binding_name: "#null" nit on the item_button view binding in ui/titles_server_form.json wasn't addressed, and the earlier threads still stand.

Still blocking, companion: beh#722 is still open (head 5f5ed83c, changes requested). The chat format change here (events.beforeChat.format drops the [...] rank brackets) only looks right with #722's title prefix, so don't merge res on its own. The rank titles in data#22 (artist, developer, game designer, assistant manager, project manager) already have matching textures here.

CI: CI / build-and-test passed on this head. Model Preview is still pending, so this isn't merge-ready yet.

Open res issues: 0, so there are no Fixes links.

Re-review head `eed96697` (was `598e9e22`). This push squashes the branch into one commit, `feat: title system ui, glyphs and lang`, and rebases it onto `main` `b4a8343a` (v3.1.1). `mergeable=true`. **This push:** no content change. I checked every non-image file in the PR (7 lang files, `ui/_ui_defs.json`, `ui/server_form.json`, `ui/titles_server_form.json`) against the old head and they're the same. The 27 `textures/ui/titles/*.png` and `font/glyph_E3.png` blobs are the same too. The PR diff is now just the title system, so the scope is clean. **Still open from before (not repeating):** the `binding_name: "#null"` nit on the `item_button` view binding in `ui/titles_server_form.json` wasn't addressed, and the earlier threads still stand. **Still blocking, companion:** [beh#722](https://git.pokebedrock.com/pokebedrock/pokebedrock-beh/pulls/722) is still open (head `5f5ed83c`, changes requested). The chat format change here (`events.beforeChat.format` drops the `[...]` rank brackets) only looks right with #722's title prefix, so don't merge res on its own. The rank titles in [data#22](https://git.pokebedrock.com/pokebedrock/pokebedrock-data/pulls/22) (artist, developer, game designer, assistant manager, project manager) already have matching textures here. **CI:** `CI / build-and-test` passed on this head. Model Preview is still **pending**, so this isn't merge-ready yet. Open res issues: 0, so there are no Fixes links.
All checks were successful
CI / build-and-test (20.x) (pull_request) Successful in 33s
Model Preview / model-preview (pull_request) Successful in 1m6s
This pull request can be merged automatically.
This branch is out-of-date with the base branch
You are not authorized to merge this pull request.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin feature/title-update:feature/title-update
git switch feature/title-update

Merge

Merge the changes and update on Forgejo.

Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.

git switch main
git merge --no-ff feature/title-update
git switch feature/title-update
git rebase main
git switch main
git merge --ff-only feature/title-update
git switch feature/title-update
git rebase main
git switch main
git merge --no-ff feature/title-update
git switch main
git merge --squash feature/title-update
git switch main
git merge --ff-only feature/title-update
git switch main
git merge feature/title-update
git push origin main
Sign in to join this conversation.
No reviewers
No labels
No milestone
No project
No assignees
4 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
pokebedrock/pokebedrock-res!10
No description provided.