Skip to content
This repository has been archived by the owner on May 1, 2024. It is now read-only.

[iOS] Fix Page BackgroundColor issue using DarkTheme #12409

Merged
merged 2 commits into from
Feb 4, 2021
Merged

Conversation

jsuarezruiz
Copy link
Contributor

@jsuarezruiz jsuarezruiz commented Oct 8, 2020

Description of Change

Fix Page BackgroundColor issue using DarkTheme on iOS.

Issues Resolved

API Changes

None

Platforms Affected

  • iOS

Behavioral/Visual Changes

None

Before/After Screenshots

fix12406

Testing Procedure

Launch Core Gallery, then change the theme and verify that the page background color changed.

PR Checklist

  • Targets the correct branch
  • Tests are passing (or failures are unrelated)

@samhouts samhouts added this to In Review in vNext+1 (5.0.0) Nov 2, 2020
@PureWeen PureWeen added the blocker Issue blocks next stable release. Prioritize fixing and reviewing this issue. label Nov 5, 2020
@PureWeen PureWeen added this to In progress in v5.0.1 via automation Nov 5, 2020
@PureWeen PureWeen removed this from In Review in vNext+1 (5.0.0) Nov 5, 2020
@rachelkang rachelkang self-requested a review December 3, 2020 21:05
@PureWeen PureWeen added this to To do in vNext+1 (5.0.0) via automation Jan 15, 2021
@PureWeen PureWeen removed this from In progress in v5.0.1 Jan 15, 2021
@PureWeen PureWeen added this to the 5.0.0 milestone Jan 15, 2021
@PureWeen PureWeen moved this from To do to In Review in vNext+1 (5.0.0) Jan 15, 2021
Copy link
Contributor

@PureWeen PureWeen left a comment

Choose a reason for hiding this comment

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

@jsuarezruiz it looks like this is happening on android also

#13500

@hartez hartez added this to To Fix in 5.0.0 SR 3 via automation Feb 2, 2021
@hartez hartez removed this from In Review in vNext+1 (5.0.0) Feb 2, 2021
@hartez hartez moved this from To Fix to PR Changes Requested in 5.0.0 SR 3 Feb 2, 2021
@hartez hartez moved this from PR Changes Requested to PR Needs Review in 5.0.0 SR 3 Feb 2, 2021
@hartez
Copy link
Contributor

hartez commented Feb 3, 2021

iOS 12 failures are just the timeout problem, unrelated to this.

@PureWeen PureWeen merged commit 25b2aa4 into 5.0.0 Feb 4, 2021
5.0.0 SR 3 automation moved this from PR Needs Review to Done Feb 4, 2021
@PureWeen PureWeen deleted the fix-12406 branch February 4, 2021 16:40
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
a/darkmode blocker Issue blocks next stable release. Prioritize fixing and reviewing this issue. hacktoberfest 🍻 hacktoberfest-accepted p/iOS 🍎 t/bug 🐛
Projects
No open projects
5 participants