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

PLAT-322 video link #388

Merged
merged 13 commits into from
Aug 22, 2016
Merged

PLAT-322 video link #388

merged 13 commits into from
Aug 22, 2016

Conversation

gusliedke
Copy link
Contributor

@gusliedke gusliedke commented Jul 22, 2016

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

Changes proposed in this pull request

  • create instructions
  • override style with rnd look and feel
  • remove all unnecessary style and fonts from library
  • design review

Deployed at http://plat-322-tx3mbsqmxtu74.eu.platform.sh/

@Saphyel Saphyel changed the title Feature/plat 322 video link PLAT-322 video link Jul 26, 2016
@pvhee pvhee added this to the 1.7 milestone Aug 12, 2016
@gusliedke gusliedke modified the milestones: 1.8, 1.7 Aug 12, 2016
…o build so it can be refactored afterwards
@wagg-matt
Copy link
Contributor

wagg-matt commented Aug 19, 2016

@AndyEPhipps @gusliedke ready for review.

Have moved the lightcase CSS into a SASS file for us to use, removed their fonts, updated some styles for the close button and the spinner - nothing else. This is due to be refactired properly afterwards but has to go in today

See how it's looking here: http://pr-388-tx3mbsqmxtu74.eu.platform.sh/fundraise, top left card

@gusliedke
Copy link
Contributor Author

@wagg-matt can you replace /components/video/_lightcase.scss for /components/lightcase/...

@wagg-matt
Copy link
Contributor

@gusliedke that's been updated, are we good to merge it when it passes travis?

@gusliedke
Copy link
Contributor Author

yeah, passing we can merge it

@gusliedke
Copy link
Contributor Author

Assigning to QA

@gusliedke gusliedke merged commit c01aa8b into develop Aug 22, 2016
@gusliedke gusliedke deleted the feature/PLAT-322_video_link branch August 22, 2016 10:26
@Saphyel Saphyel mentioned this pull request Aug 22, 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

3 participants