Skip to content

final#1

Open
cappers86 wants to merge 3 commits into
masterfrom
colin-toft
Open

final#1
cappers86 wants to merge 3 commits into
masterfrom
colin-toft

Conversation

@cappers86
Copy link
Copy Markdown
Owner

Colin Toft

Copy link
Copy Markdown
Collaborator

@ropeks ropeks left a comment

Choose a reason for hiding this comment

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

This looks good, still along way to go, but you're making good progress 🎉 🎉 . Just make sure you're comfortable with flex before Sprint challenge tomorrow @cappers86

Comment thread great-idea/css/index.css
}
ol, ul {
list-style: none;
list-style: ;
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

is this intentional? @cappers86

Comment thread great-idea/css/index.css
text-decoration: none;
}
.navigation {
display: inline-block;
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

forgot to add flex here :)

Comment thread great-idea/index.html
Comment on lines +19 to +103
<body>

<header id="nav-bar">
<nav class="navigation">
<a href="services.html">Services</a>
<a href="">Product</a>
<a href="">Vision</a>
<a href="">Features</a>
<a href="">About</a>
<a href="">Contact</a>
</nav>

<img class="logo" src="img/logo.png" alt="Great Idea! Company logo.">

</header>

<section id="innovation-demand">
<div id="innovatetext">
<h1 id="title">Innovation <br>
On<br>
Demand
</h1>
<button id="start-button"> Get Started</button>
</div>
<div id="innovate-imgage">
<img src="img/header-img.png" alt="Image of a code snippet.">
</div>
</section>

<hr>
<section class="upper-section">
<div id="features">
<h3 class="subsection1"> Features</h3>
<p>Aliquam elementum magna eros, ac posuere elvit tempus et. Suspendisse vel tempus odio, in interdutm nisi.
Suspendisse eu ornare nisl. Nullam convallis augue justo, at imperdiet metus scelerisque quis.</p>
</div>
<div id="about">
<h3 class="subsection1"> About</h3>
<p>Aliquam elementum magna eros, ac posuere elvit tempus et. Suspendisse vel tempus odio, in interdutm nisi.
Suspendisse eu ornare nisl. Nullam convallis augue justo, at imperdiet metus scelerisque quis.</p>
</div>
</section>

<img class="wide-img" src="img/mid-page-accent.jpg" alt="Image of code snippets across the screen">

<section class="lower-section">
<div id="services">
<h3 class="sub-section2">Services</h3>
<p>Aliquam elementum magna eros, ac posuere elvit tempus et. Suspendisse vel tempus odio, in interdutm nisi.
Suspendisse eu ornare nisl. Nullam convallis augue justo, at imperdiet metus scelerisque quis.
</p>
</div>

<div id="product">
<h3 class="sub-section2">Product</h3>
<p>Aliquam elementum magna eros, ac posuere elvit tempus et. Suspendisse vel tempus odio, in interdutm nisi.
Suspendisse eu ornare nisl. Nullam convallis augue justo, at imperdiet metus scelerisque quis.
</p>
</div>

<div id="vison">
<h3 class="sub-section2">Vision</h3>
<p>Aliquam elementum magna eros, ac posuere elvit tempus et. Suspendisse vel tempus odio, in interdutm nisi.
Suspendisse eu ornare nisl. Nullam convallis augue justo, at imperdiet metus scelerisque quis.

</p>
</div>
</section>

<hr>
<section class="contact-us">
<h3 class="contact">Contact</h3>
<p>Address: 123 Way 456 Street</p>
<p>Somewhere, USA</p>
<br>
<p>Phone: 1 (888) 888-8888</p>
<br>
<p>Email: sales@greatidea.io</p>
</section>

<div id="legal">
Copyright Great Idea! 2018
</div>

</body>
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

🔥

Comment thread great-idea/index.html
Comment on lines +92 to +93
<p>Somewhere, USA</p>
<br>
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

you can use just one of those (either <br> or wrap it in <p>) because <p> tag actually separates what's wrapped in it

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