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

Conversation

@nheagy
Copy link
Contributor

@nheagy nheagy commented Jun 28, 2019

This PR makes changes necessary for WPiOS to adopt a new color scheme:

  • change the prologue animations and colors
  • add new, optional, style properties for prologue text and bg colors
  • remove the default blue of the navigation bar bg color

Copy link
Contributor

@frosty frosty left a comment

Choose a reason for hiding this comment

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

Just a couple very small comments!

self.subheadlineColor = subheadlineColor
self.viewControllerBackgroundColor = viewControllerBackgroundColor
self.navBarImage = navBarImage
self.prologueBackgroundColor = prologueBackgroundColor ?? WPStyleGuide.wordPressBlue()
Copy link
Contributor

Choose a reason for hiding this comment

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

Should we just set these as default parameters?

Pod::Spec.new do |s|
s.name = "WordPressAuthenticator"
s.version = "1.5.4-beta.2"
s.version = "1.5.4-beta.3"
Copy link
Contributor

Choose a reason for hiding this comment

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

Do you think these changes warrant a bigger version bump?

@nheagy nheagy changed the title Issues/11683 new wpios colors Update for new WPiOS colors Jun 28, 2019
@nheagy nheagy requested a review from frosty June 28, 2019 21:02
@nheagy nheagy merged commit d4c6642 into develop Jun 28, 2019
@nheagy nheagy deleted the issues/11683-new-wpios-colors branch June 28, 2019 21:50
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants