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

Page | Case studies #116

Merged
merged 22 commits into from
Apr 30, 2020
Merged

Page | Case studies #116

merged 22 commits into from
Apr 30, 2020

Conversation

monsteronfire
Copy link
Contributor

@monsteronfire monsteronfire commented Apr 22, 2020

This PR includes:

  • a new Case Studies page design
  • case study content and illustrations
  • burger menu on the left
  • a downgrade of the Jekyll image version
  • updated hire us CTA design at bottom of case studies and services page
  • a fix for horizontal scroll bug on hire us page
  • a bunch of small UI fixes

To test:

  • In your browser, resize width (or use Responsive Design Mode)
  • make sure nothing is broken on small, medium, large screens

@monsteronfire
Copy link
Contributor Author

monsteronfire commented Apr 26, 2020

Blergh. Just leaving a note here for posterity.

The Jekyll image I was using has had a bunch of changes made to it. The latest image stopped working, suddenly, for some people.

There's an open issue on their github:
envygeeks/jekyll-docker#268

I'm going to downgrade and use a previous image.

@calumhalcrow
Copy link
Contributor

Looks great so far 😄

@monsteronfire monsteronfire removed do-not-merge wip work in progress labels Apr 28, 2020
@monsteronfire
Copy link
Contributor Author

@calumhalcrow I think this PR is ready to review. Just giving you a heads-up. Please take a look at it when you are able. Thanks!

@calumhalcrow
Copy link
Contributor

@calumhalcrow I think this PR is ready to review. Just giving you a heads-up. Please take a look at it when you are able. Thanks!

Will do soon...

Copy link
Contributor

@calumhalcrow calumhalcrow left a comment

Choose a reason for hiding this comment

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

The new Case Studies page looks amazing. The design plus the content makes a huge difference to our site. This is great @monsteronfire 👍

I did some UAT on the changes and they LGTM. Feel free to :shipit: 😄

@monsteronfire
Copy link
Contributor Author

Thanks for doing UAT, @calumhalcrow! Merging now. 👍

@monsteronfire monsteronfire merged commit e0a9847 into master Apr 30, 2020
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.

2 participants