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

Site Structure #2

Open
wants to merge 21 commits into
base: materialize.html
Choose a base branch
from
Open

Site Structure #2

wants to merge 21 commits into from

Conversation

mallesea
Copy link
Collaborator

I have decided to re-structure my website and start over from scratch to fix any errors or bugs I created the first time (I still have that code copied and saved in case I need t refer to it later). The last page I had written was called materialize.html because that was the last page written, which is now compared to the updated new branch, which contains all of my documents and folders for my website.

Updated Site Structure
still needing header for css page, navigation, jaavscript, and materialize.html
This is my uploaded html pages, styles sheet with navigation, images, and javascript
@AlyssaKirstine
Copy link
Member

Hi @mallesea the base branch for your pull requests should always be set to main. It looks like you have it set to materialize.html for this pull request. You can take a look at the GitHub documentation here about updating the base branch. Please merge this pull request (you'll see the green button for doing this on this page), make sure to click the button to sync your changes in GitHub Desktop, and open a new PR/create a new branch for any future assignments! The way you did this is fine for now, but it makes it easier to review your code for each individual assignment if you have separate pull requests. It's a bit harder to review and give you good feedback since this PR has the code for multiple assignments.

mallesea and others added 15 commits November 4, 2024 16:51
Added link to css page in about.html and sisterlocation.html
Originally added to the wrong branch
Made some adjustments to main materialize.html, added materialize navigation, added materialize javascript onto pages
added materialize javascript to two pages and cleaned up website, added new image folder to website
Merge pull request #4 from Miami-University-ETBD/main
added jquery.html page with jquery intercation as well as navigation to the page
Created a bootstrap.html page with bootstrap interactions and corresponding context inside, including a new holiday menu
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