Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature/improvements #101

Open
wants to merge 15 commits into
base: src
Choose a base branch
from
Open

Feature/improvements #101

wants to merge 15 commits into from

Conversation

Farid-Karimli
Copy link

Fiddled with the design. Changed the HeaderPhoto, removed it from every page except the landing page as well. Changed the font, moved some things around and overall just tweaked the looks of the website.

package.json Outdated Show resolved Hide resolved
src/components/main-site/Footer.js Outdated Show resolved Hide resolved
src/components/main-site/Header.js Outdated Show resolved Hide resolved
src/components/main-site/Header.js Outdated Show resolved Hide resolved
src/components/main-site/Header.js Outdated Show resolved Hide resolved
src/pages/main-site/Contact.js Outdated Show resolved Hide resolved
src/pages/main-site/Events.js Outdated Show resolved Hide resolved
src/pages/main-site/Landing.js Outdated Show resolved Hide resolved
src/pages/main-site/Landing.js Outdated Show resolved Hide resolved
src/pages/main-site/Members.js Outdated Show resolved Hide resolved
@ROODAY
Copy link
Member

ROODAY commented Apr 28, 2021

@Farid-Karimli I made a lot of comments regarding whitespace but I realized after submitting the review that this repo doesn't run prettier as a pre commit hook. So a quick way to fix almost all of the white space issues is to run yarn format. Also added this issue to track the pre-commit hook.

dependabot bot and others added 13 commits May 7, 2021 22:58
…js-0.7.28

Bump ua-parser-js from 0.7.22 to 0.7.28
…1.5.1

Bump url-parse from 1.4.7 to 1.5.1
…-info-2.8.9

Bump hosted-git-info from 2.8.8 to 2.8.9
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants