Add Leaderboard modal UI, visual-state leaderboard projection, and wiring (#35)

* Review leaderboard modal behavior

* Fix leaderboard UI layering, scaling, and HUD placement

The leaderboard modal was rendering behind the HUD/map because its
UIDocument shared sortingOrder 0 with every other document and relied
on scene-authoring order to win ties. Give it an explicit sortingOrder
in code instead, per this project's own guidance not to rely on scene
ordering for stacking.

The table header wasn't scaling with the panel because .gs-panel uses
align-items: center, so header children shrank to content size instead
of stretching to the panel width — only the table header needed the
fix, the title bar stays centered by design.

Also moves the leaderboard button next to the player org panel,
fixes org/country flag aspect ratio, disables map zoom while the
window is open (full scroll-forwarding into the list is deferred),
and formats scores with thousands separators.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>

* Pin leaderboard close button to the panel's top-right corner

The close button lived inside the centered title row, so it sat next
to the title instead of at a fixed corner. Move it out to its own
absolutely-positioned element inset by the panel's own padding,
independent of the title, which stays centered.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>

* Replace InternalsVisibleTo with public API for test-only access

Game.Main and Game.Configs.Loader each carried an AssemblyInfo.cs
whose only purpose was granting Game.Tests blanket access to internal
members. That's an invisible, all-or-nothing escape hatch rather than
a deliberate API decision, and it hides which members tests actually
depend on.

Flip just the members Game.Tests calls (VisualStateConverter's
constructor and Update/UpdateCountryScore/UpdateLeaderboards; Program
.ApplyPreservedFields; ProvinceProcessor.Process, plus their
containing types) to public instead, and drop both AssemblyInfo.cs
files now that nothing needs InternalsVisibleTo. Document the
preference in code_style.md so future test-only access follows the
same pattern.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>

---------

Co-authored-by: Claude Sonnet 5 <noreply@anthropic.com>

Author: KonH

.claude/rules/csharp/code_style.md

@@ -20,3 +20,28 @@
   ```
 - Fail fast with descriptive, contextual error messages rather than continuing on bad state
 - Never silently swallow an exception — at minimum log it at error level
+
+## Prefer `public` over `InternalsVisibleTo` for test access
+
+When a `Game.Tests` test needs to call a class/method that's currently `internal` only
+because nothing outside its own assembly needs it *yet*, make the class and that member
+`public` instead of adding an `[assembly: InternalsVisibleTo("Game.Tests")]` escape hatch
+in an `AssemblyInfo.cs`. `InternalsVisibleTo` grants the test assembly access to
+*everything* internal, invisibly, forever — it decays into a blanket bypass of the
+access-modifier boundary rather than a deliberate API decision. A `public` member is
+visible in the type's own file, doesn't require readers to know a separate
+`AssemblyInfo.cs` exists, and if it turns out other assemblies want it too, no further
+change is needed.
+
+If an existing `AssemblyInfo.cs` only exists for `InternalsVisibleTo`, grep the target
+assembly's test project for which specific `internal` members it actually calls, make
+just those (and their containing types, if the type itself is `internal`/default-access)
+`public`, then delete the `AssemblyInfo.cs` file entirely. Example: `Game.Main`'s
+`VisualStateConverter` (constructor + `Update`/`UpdateCountryScore`/`UpdateLeaderboards`)
+and `Game.Configs.Loader`'s `Program.ApplyPreservedFields`/`ProvinceProcessor.Process`
+(plus their containing `static class`es) were flipped to `public` this way, removing
+both assemblies' `AssemblyInfo.cs`.
+
+This does not apply to genuine encapsulation boundaries (e.g. hiding a helper that would
+be actively misleading or unsafe to call from outside its owning system) — those should
+stay `internal` and get exercised only through the assembly's public entry points.

Assets/Localization/en.asset

@@ -14,6 +14,16 @@ MonoBehaviour:
   m_EditorClassIdentifier: GS.Unity.UI::LocaleConfig
   Locale: en
   Entries:
+  - Key: leaderboard.title
+    Value: Leaderboard
+  - Key: leaderboard.tab.organizations
+    Value: Organizations
+  - Key: leaderboard.tab.countries
+    Value: Countries
+  - Key: leaderboard.empty
+    Value: No leaderboard entries available
+  - Key: hud.leaderboard
+    Value: Leaderboard
   - Key: country_name.Afghanistan
     Value: Afghanistan
   - Key: country_name.Algeria

Assets/Localization/ru.asset

@@ -14,6 +14,16 @@ MonoBehaviour:
   m_EditorClassIdentifier: GS.Unity.UI::LocaleConfig
   Locale: ru
   Entries:
+  - Key: leaderboard.title
+    Value: Таблица лидеров
+  - Key: leaderboard.tab.organizations
+    Value: Организации
+  - Key: leaderboard.tab.countries
+    Value: Страны
+  - Key: leaderboard.empty
+    Value: Нет записей таблицы лидеров
+  - Key: hud.leaderboard
+    Value: Лидеры
   - Key: country_name.Afghanistan
     Value: "\u0410\u0444\u0433\u0430\u043D\u0438\u0441\u0442\u0430\u043D"
   - Key: country_name.Algeria