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

website server enhancements #44

Merged
merged 21 commits into from
May 22, 2018
Merged

Conversation

PaulPaulDevelops
Copy link
Member

@PaulPaulDevelops PaulPaulDevelops commented May 21, 2018

@jedihacks @LizCottrell add any additional description of changes to the pull request if need be.

jedihacks and others added 20 commits May 18, 2018 17:48
Added server config and fixed issues caused by packages in index.html
In an effort to track down an ellusive server-side error committing in some light debugging language
to help identify the cause.  Also removed unused robotto condensed and made some small css fixes
that cause parsing errors

re #41
The import declaration causes some css syntax errors
Updated linter requirements; fixed a couple issues; and updated README
Fixes the flash issue; but causes error where going from About to Opportunities or Contact page
hides the background for some reason

BREAKING CHANGE: Causes issue with background image when navigating from About to Opportunities page
Forgot to include some files in the previous commit, so finishing them here.  Refactored the
navigation header to be a separate file
found issue with contact page background, will bring up to elizabeth tomorrow
The getElementsByClassName was causing elements in other components to become hidden.
broke out component into individual hero HTML + styles for each page (contact, about, opportunities)

#16
Copy link
Member

@jedihacks jedihacks left a comment

Choose a reason for hiding this comment

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

Just pushed a change with a couple cleanup items; after you pull that it in it looks good!

@PaulPaulDevelops PaulPaulDevelops merged commit 085d31b into develop May 22, 2018
@PaulPaulDevelops PaulPaulDevelops deleted the website-server-enhancements branch May 22, 2018 14:16
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