fix(themes): style admin health alerts
Add custom-theme styles for the admin configuration alerts and bump built-in theme asset versions so existing installations refresh stale theme CSS.
This commit is contained in:
@@ -34,14 +34,14 @@ class WebappThemesConfigTests(unittest.TestCase):
|
||||
self.assertEqual(win95.tokens.style_preset, "win95")
|
||||
self.assertFalse(win95.use_primary_accent)
|
||||
self.assertTrue(win95.use_in_admin)
|
||||
self.assertEqual(win95.assets_version, 13)
|
||||
self.assertEqual(cfg.theme_by_key("light").assets_version, 5)
|
||||
self.assertEqual(win95.assets_version, 14)
|
||||
self.assertEqual(cfg.theme_by_key("light").assets_version, 6)
|
||||
ascii_theme = cfg.theme_by_key("ascii")
|
||||
self.assertIsNotNone(ascii_theme)
|
||||
self.assertEqual(ascii_theme.css_file, "style.css")
|
||||
self.assertFalse(ascii_theme.use_primary_accent)
|
||||
self.assertTrue(ascii_theme.use_in_admin)
|
||||
self.assertEqual(ascii_theme.assets_version, 6)
|
||||
self.assertEqual(ascii_theme.assets_version, 7)
|
||||
|
||||
def test_env_override_default_theme(self):
|
||||
cfg = builtin_webapp_themes_config("#00fe7a")
|
||||
@@ -430,7 +430,7 @@ class WebappThemesConfigTests(unittest.TestCase):
|
||||
descriptor["assets_version"],
|
||||
cfg.theme_by_key("windows95").assets_version,
|
||||
)
|
||||
self.assertEqual(descriptor["assets_version"], 13)
|
||||
self.assertEqual(descriptor["assets_version"], 14)
|
||||
self.assertIn("lucide-house", css)
|
||||
self.assertIn("lucide-earth", css)
|
||||
self.assertIn("lucide-circle-check", css)
|
||||
@@ -444,6 +444,7 @@ class WebappThemesConfigTests(unittest.TestCase):
|
||||
self.assertIn("New webapp surfaces: support, purchase info, password login", css)
|
||||
self.assertIn("Install guide theme surfaces", css)
|
||||
self.assertIn("Admin controls: range sliders and sortable rows", css)
|
||||
self.assertIn("Admin health config alerts", css)
|
||||
self.assertIn(
|
||||
(
|
||||
".theme-key-windows95 .support-list-card {\n"
|
||||
@@ -486,6 +487,7 @@ class WebappThemesConfigTests(unittest.TestCase):
|
||||
self.assertIn(".theme-key-light.app-shell", css)
|
||||
self.assertIn("Install guide theme surfaces", css)
|
||||
self.assertIn("Admin controls: range sliders and sortable rows", css)
|
||||
self.assertIn("Admin health config alerts", css)
|
||||
|
||||
def test_resolved_refreshes_stale_builtin_ascii_assets(self):
|
||||
with tempfile.TemporaryDirectory() as tmp:
|
||||
@@ -518,7 +520,8 @@ class WebappThemesConfigTests(unittest.TestCase):
|
||||
descriptor = json.loads((stale_theme_dir / "theme.json").read_text(encoding="utf-8"))
|
||||
css = (stale_theme_dir / "style.css").read_text(encoding="utf-8")
|
||||
self.assertEqual(descriptor["assets_version"], cfg.theme_by_key("ascii").assets_version)
|
||||
self.assertEqual(descriptor["assets_version"], 6)
|
||||
self.assertEqual(descriptor["assets_version"], 7)
|
||||
self.assertIn("Console-style tables", css)
|
||||
self.assertIn("Install guide theme surfaces", css)
|
||||
self.assertIn("Admin controls: range sliders and sortable rows", css)
|
||||
self.assertIn("Admin health config alerts", css)
|
||||
|
||||
Reference in New Issue
Block a user