Patrick Georgi has submitted this change. ( https://review.coreboot.org/c/edk2/+/58784 )
Change subject: MdeModulePkg/BootLogoLib: Center logo 38.2% from top of screen ......................................................................
MdeModulePkg/BootLogoLib: Center logo 38.2% from top of screen
Use Microsoft's recommended positioning [1] for the boot logo.
We recommend that the logo is placed with its center at 38.2% from the screen's top edge. This positioning is based on the golden ratio's visual aesthetics and matches the Windows 10 design proportions.
[1]: https://docs.microsoft.com/en-us/windows-hardware/drivers/bringup/boot-scree...
Signed-off-by: Matt DeVillier matt.devillier@gmail.com Change-Id: Ia357e59626dc8bec0827763e803ac0c8939f0acf Reviewed-on: https://review.coreboot.org/c/edk2/+/58784 Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Patrick Georgi pgeorgi@google.com --- M MdeModulePkg/Library/BootLogoLib/BootLogoLib.c 1 file changed, 1 insertion(+), 1 deletion(-)
Approvals: build bot (Jenkins): Verified Patrick Georgi: Looks good to me, approved
diff --git a/MdeModulePkg/Library/BootLogoLib/BootLogoLib.c b/MdeModulePkg/Library/BootLogoLib/BootLogoLib.c index 134660f..76255b6 100644 --- a/MdeModulePkg/Library/BootLogoLib/BootLogoLib.c +++ b/MdeModulePkg/Library/BootLogoLib/BootLogoLib.c @@ -175,7 +175,7 @@ break; case EdkiiPlatformLogoDisplayAttributeCenter: DestX = (SizeOfX - Image.Width) / 2; - DestY = (SizeOfY - Image.Height) / 2; + DestY = (SizeOfY * 382) / 1000 - Image.Height / 2; break; case EdkiiPlatformLogoDisplayAttributeCenterRight: DestX = SizeOfX - Image.Width;
1 is the latest approved patch-set. No files were changed between the latest approved patch-set and the submitted one.