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

Cards and other changes #130

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Cards and other changes #130

wants to merge 3 commits into from

Conversation

maenad
Copy link

@maenad maenad commented Oct 4, 2019

Pull Request Template

Hey there,

Thank you for contribution and walking with crea8lab in this awesome journey. Here are a few tips to help you make successful PRs

  • Update your local copy after you've committed your changes.
  • If any conflicts, resolve them accordingly.
  • Take screenshots of your display... how does what you've done look in the browser?
  • Go to your github account and send a PR from there with the screenshot attached to it.

Happy hacking


Full list of changes

  • Added cards as per card issue.
  • Beautified the footer (more padding on the bottom so it wouldn't look weird)
  • Fixed the form-control dropdown since the words were getting cut off due to its height.
  • Updated readme to add MDN documentation for HTML, CSS and JavaScript (the link was broken).
  • Fixed font size
  • Fixed font background in carousel for better legibility.

hacktoberfest_capture
hacktoberfest_capture2

#64

@maenad
Copy link
Author

maenad commented Oct 4, 2019

Can we add the #hacktoberfest label to this, please?

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.

1 participant