Skip to content
This repository has been archived by the owner on Dec 10, 2021. It is now read-only.

PLAT-386 promo design up #426

Merged
merged 38 commits into from
Aug 18, 2016
Merged

Conversation

gusliedke
Copy link
Contributor

@gusliedke gusliedke commented Aug 10, 2016

https://jira.comicrelief.com/browse/PLAT-386

http://pr-426-tx3mbsqmxtu74.eu.platform.sh/hidden-title

Changes proposed in this pull request

  • design review
  • move order bg colour

Manual deployment instructions

  • some config from cr profile needs to be added in manually. E.g. 443955c

@gusliedke gusliedke added this to the 1.8 milestone Aug 10, 2016
@gusliedke
Copy link
Contributor Author

@pvhee ok have pushed, watching if it will update in platsh env..

@pvhee
Copy link
Contributor

pvhee commented Aug 16, 2016

@gusliedke I've pushed this again - I think the branch naming might give problems (or the fact we have only 9 envs. on platform.sh currently paid for). Let's see what it says now.

@pvhee pvhee closed this Aug 16, 2016
@pvhee pvhee reopened this Aug 16, 2016
@Saphyel Saphyel changed the title Feature/plat 386 promo design up PLAT-386 promo design up Aug 17, 2016
&:hover {
color: $colour-white;
}
}
Copy link
Contributor

Choose a reason for hiding this comment

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

Do we need to update the mixin values here rather than just overriding it?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Moving it to buttons.scss but we still need it as you can't control hover colour in the mixin

@AndyEPhipps AndyEPhipps removed the in QA label Aug 18, 2016
@AndyEPhipps AndyEPhipps merged commit 0202e3b into develop Aug 18, 2016
@AndyEPhipps AndyEPhipps deleted the feature/PLAT-386_promo_design_up branch August 18, 2016 14:06
@Saphyel Saphyel mentioned this pull request Aug 22, 2016
@@ -91,4 +91,8 @@
</div>
</div>
</div>
<<<<<<< HEAD
</div>
=======
Copy link
Contributor

Choose a reason for hiding this comment

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

is ok having this git things here? cc// @gusliedke @AndyEPhipps

@pvhee pvhee mentioned this pull request Jan 5, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants