Polish title menu presentation
This commit is contained in:
@@ -76,10 +76,12 @@ func _check_title_menu_start_load_and_settings(failures: Array[String]) -> void:
|
||||
failures.append("title screen should use a dedicated cinematic background, not the first battle map backdrop")
|
||||
if scene.title_portrait_texture_rect == null or scene.title_portrait_texture_rect.texture == null:
|
||||
failures.append("title screen should show Cao Cao portrait art")
|
||||
elif scene.title_portrait_texture_rect.material == null or not bool(scene.title_portrait_texture_rect.material.get_meta("title_portrait_feather_material", false)):
|
||||
failures.append("title Cao Cao portrait should feather into the background instead of reading as a hard rectangle")
|
||||
var portrait_shadow := scene.title_screen_root.find_child("TitlePortraitLacquerField", true, false) as ColorRect if scene.title_screen_root != null else null
|
||||
if portrait_shadow == null:
|
||||
failures.append("title screen should keep a subtle portrait grounding field")
|
||||
elif portrait_shadow.color.a > 0.18:
|
||||
elif portrait_shadow.color.a > 0.08:
|
||||
failures.append("title portrait grounding field should not read as a black rectangle: %.2f alpha" % portrait_shadow.color.a)
|
||||
if scene.title_panel == null or scene.title_panel.position.x > 100.0:
|
||||
failures.append("title menu panel should sit on the left side of the cinematic title screen")
|
||||
@@ -426,6 +428,8 @@ func _check_title_menu_button_art(failures: Array[String], button: Button, expec
|
||||
failures.append("%s generated button surface is missing its texture" % context)
|
||||
elif style.texture.get_width() < 512 or style.texture.get_height() < 192:
|
||||
failures.append("%s generated button texture should keep high-resolution source pixels: %dx%d" % [context, style.texture.get_width(), style.texture.get_height()])
|
||||
if style.texture_margin_top > 24 or style.texture_margin_bottom > 24:
|
||||
failures.append("%s generated button slice should stay shallow enough for the 720p title menu: %d/%d" % [context, style.texture_margin_top, style.texture_margin_bottom])
|
||||
if str(button.tooltip_text).strip_edges().is_empty():
|
||||
failures.append("%s should describe its command through a tooltip" % context)
|
||||
|
||||
|
||||
Reference in New Issue
Block a user