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

Alena's Static-Site #25

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

Conversation

Spatterjaaay
Copy link

Static Site

Congratulations! You're submitting your assignment!

Comprehension Questions

Question Answer
Did you have to resolve any issues when running the HTML Validator? If so, what were they? The validator kept asking me about sections without headers, which helped me notice I had a couple of sections in my html files that had the same scope as articles, so I deleted those. Once I changed h1 heading to h2, because it was not on top of the page.
Why is it important to consider and use semantic HTML? It help make the code readable, both for human developers, who have to read it (and might not have written it), and for accessibility reasons, for example screen readers benefit form semantic tags a lot. Also it is useful to Google, which cans our page.
How did you decide to structure your CSS? I created main styles file for the home page and shared elements across all the sites. Then I created special stylesheet for each site that contained site specific elements. I heavily utilized classes and ids in order to make my css easily readable. (It really does help a lot.)
What was the most challenging piece of this assignment? Definitely design. It took a long time to come up with an idea I was happy with :) Designing 4 pages required a lot of ideas and lots of changes, before I came up with something that looked and worked okay. I did eventually run out of time and that's why there is a lot of lorem ipsum, instead of content. Coming up with what to say was also not easy, so I prioritized html and css.
Describe one area that you gained more clarity on when completing this assignment Using developer tools and containers (especially floating). At first I thought the tools were interesting, but not as useful, but then I discovered they are a lifesaver when it comes to inspecting boxes and inheritance. I can't imagine doing project this big without them! The box structure was initially a bit confusing, but I think I have a better grasp on how each box affect other boxes and so on.

…ain tag, header and footer, added lorem ipsum
… hand drawn path now! :D), added styling for paper airplane images
@CheezItMan
Copy link

Static Site

What We're Looking For

Feature Feedback
Baseline
Appropriate Git Usage Good regular commits and messages
Answered comprehension questions Check, I will say that it's often a very good idea to have one set of consistent style sheets for all the pages, that helps keep code dry and the site consistent. If each page is using a separate sheet it's easy to be duplicating effort and creating pages with wildly different styles.
Page fully loads Check
No broken links (regular or images) Check
Includes at least 4 pages and styling
HTML
Uses the high-level tags for organization: header, footer, main Check
Appropriately using semantic tags: section, article, etc. Yes, although I'm unclear why you used a div for the introduction.
All images include alternate text Check
CSS
Using class and ID names in style declarations Check
Style declarations are DRY Pretty dry here
Overall The site looks beautiful! You have a great eye for design. I would recommend in the future for the html to be in the root directory not a separate subdirectory. It's more traditional to do it that way. You have fairly DRY code and a great looking site. Make sure you get it published online!

Copy link

@LaurenSky LaurenSky left a comment

Choose a reason for hiding this comment

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

Great Job! The site looks amazing and speaks to your personality and design style. Feel free to reach out to me if you have any questions :)


<body>
<header class="header-and-footer">
<a id="home" href="index.html"> Home </a>

Choose a reason for hiding this comment

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

It's more semantic to not include spaces on either end of the text in between the html tags ... Ex: Home

</head>

<body>
<header class="header-and-footer">

Choose a reason for hiding this comment

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

I like how you used the same class name for the header and footer. Great Job keeping your css dry.

</nav>
</header>

<main>

Choose a reason for hiding this comment

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

I like your design decision to break up where you came from, where you are and where your going 👍

</header>

<main>
<section class="background" id="prague">

Choose a reason for hiding this comment

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

since the article and section here are both part of the same where I came from part of the page. How do you think you could show that semantically using html?


<main>
<section class="background" id="books">
<h1 class="banner" id="books-banner"> Book Adventures </h1>

Choose a reason for hiding this comment

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

I really like your background image and font selection. Do you think the contrast between them should be increased for user experience or accessibility reasons?

width: 100%;
top: 0;
z-index: 100;
display: flex;

Choose a reason for hiding this comment

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

great use of flex box!

color: #ffeecc;
}

a:hover {

Choose a reason for hiding this comment

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

always like hover overs on links so the user knows it's a link. good job.

}

#content {
animation: fadein 4s;

Choose a reason for hiding this comment

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

I like this, it's subtle but impactful.


#future {
background: url("../images/silicon-valley.jpg");
background-size: cover;

Choose a reason for hiding this comment

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

lines 29-32 are similiar to what's under #seattle.... Is there a way that you can think of to not duplicate code and dry this up?

padding-bottom: 1%;
}

article:nth-of-type(odd) .image-box {

Choose a reason for hiding this comment

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

great use of :nth-of-type

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