Skip to content
This repository was archived by the owner on Feb 5, 2025. It is now read-only.

Add background view to the LoginPrologueViewController.swift#821

Merged
alpavanoglu merged 3 commits into
trunkfrom
cobranding-login-prolog-update
Jan 4, 2024
Merged

Add background view to the LoginPrologueViewController.swift#821
alpavanoglu merged 3 commits into
trunkfrom
cobranding-login-prolog-update

Conversation

@alpavanoglu

Copy link
Copy Markdown
Contributor

Refs: wordpress-mobile/WordPress-iOS#22296

Description

This PR fixes the issue where setting the background color for the buttons did not work in iPad as the background for the buttons did not cover the full width. Adding a view behind the buttons container and setting the background color to the same value, resolved the issue.

Testing Steps

Follow the steps on referenced PR and confirm the UI has no such aforementioned oddity on iPad.

  • I have considered if this change warrants release notes and have added them to the appropriate section in the CHANGELOG.md if necessary.

@alpavanoglu alpavanoglu added the bug Something isn't working label Jan 4, 2024

@salimbraksa salimbraksa left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! 🚀

@alpavanoglu
alpavanoglu merged commit 419cd5b into trunk Jan 4, 2024
@alpavanoglu
alpavanoglu deleted the cobranding-login-prolog-update branch January 4, 2024 14:39
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants