Skip to content

Practice 015 - Carlos#61

Open
cadaar wants to merge 2 commits into
talosdigital:masterfrom
cadaar:carlos-015
Open

Practice 015 - Carlos#61
cadaar wants to merge 2 commits into
talosdigital:masterfrom
cadaar:carlos-015

Conversation

@cadaar
Copy link
Copy Markdown

@cadaar cadaar commented Jan 2, 2018

No description provided.

Comment thread src/015-flex/example/index.html Outdated
<footer>
<div class="dropdown">
<label for="about">Lorem ipsum dolor sit amet</label>
<input type="checkbox" id="about">
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I don't think any of this checkbox is need it

Comment thread src/015-flex/example/example.css Outdated
display: none;
}

footer input[type='checkbox'] {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I don't think this is need it

padding: 40px 50px;
}

footer .content {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This is not been used

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

It is, when the resolution is <= 768px the footer hides the content and only shows the headings

Comment thread src/015-flex/example/index.html Outdated
<div class="page-content">
<!-- Practice Content Goes Here -->
<section class="main-content">
<div class="card-wrapper">
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

this should be and < article >

Comment thread src/015-flex/example/index.html Outdated
</div>
</section>
<footer>
<div class="dropdown">
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

These div's should a < section >

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