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

RND-465 kids nav #579

Merged
merged 25 commits into from
Nov 11, 2016
Merged

RND-465 kids nav #579

merged 25 commits into from
Nov 11, 2016

Conversation

gusliedke
Copy link
Contributor

@gusliedke gusliedke commented Oct 25, 2016

https://jira.comicrelief.com/browse/RND-465

Manual deployment steps

See build instructions in JIRA

Todo

  • check ESU bg imgs .. conflict using img sprites

@pvhee
Copy link
Contributor

pvhee commented Oct 25, 2016

Weirdly article tests are broken for this PR - did you touch anything related to that @gusliedke ?

Other than that, http://pr-579-v7mya5y-tx3mbsqmxtu74.eu.platform.sh/kids looks really cool :)

@pvhee
Copy link
Contributor

pvhee commented Oct 28, 2016

@Saphyel can you review and help out here to fix the tests? maybe related to the issue you had yesterday

@pvhee pvhee added this to the 1.16.0 milestone Oct 28, 2016
@gusliedke
Copy link
Contributor Author

@AndyEPhipps can you review it please .. I am adding kids nav to look and behave like main nav

@pvhee pvhee removed this from the 1.16.0 milestone Oct 28, 2016
@AndyEPhipps
Copy link
Contributor

Code all looks good @gusliedke, would just be nice to get in some 2x images for retina devices?

@Saphyel
Copy link
Contributor

Saphyel commented Oct 28, 2016

@gusliedke travis is happy! ✅

@pvhee pvhee added this to the 1.17.0 milestone Nov 8, 2016
@bimsonz bimsonz changed the title Feature/rnd 465 kids nav RND-465 kids nav Nov 8, 2016
@gusliedke gusliedke modified the milestones: 1.18.0, 1.17.0 Nov 8, 2016
@gusliedke gusliedke removed the in QA label Nov 11, 2016
@AndyEPhipps AndyEPhipps merged commit 3800a91 into develop Nov 11, 2016
@AndyEPhipps AndyEPhipps deleted the feature/RND-465_kids_nav branch November 11, 2016 15:32
@bimsonz
Copy link
Contributor

bimsonz commented Nov 16, 2016

@gusliedke could you list the manual config please dude?

@bimsonz
Copy link
Contributor

bimsonz commented Nov 16, 2016

Chatted with Gus, the code will sit on the site but the menu needs building out by the content team!

@bimsonz bimsonz mentioned this pull request Nov 16, 2016
@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

5 participants