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

Added Favicon and Assets Folder #38

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

Added Favicon and Assets Folder #38

wants to merge 6 commits into from

Conversation

bmento
Copy link

@bmento bmento commented Nov 17, 2021

Added favicon and assets folder

I added the favicon to the tab as well as added it on the top left of the page. I also created an assets folder for the images and audio used in our site.
ignore .DS_Store
Copy link
Owner

@lee2sman lee2sman left a comment

Choose a reason for hiding this comment

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

I'm almost ready to merge. Can you move the height and width if needed to the stylesheet so we are only looking in one place?

bmento and others added 2 commits November 17, 2021 16:56
updated the style of the logo to the css file
@bmento
Copy link
Author

bmento commented Nov 17, 2021

fixed the height and width by putting them into the css file

@bmento bmento closed this Nov 17, 2021
@bmento bmento reopened this Nov 17, 2021
@lee2sman
Copy link
Owner

Can you resolve conflicts?

updated the html and css conflicts
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.

2 participants