Skip to content

Tylar Pierson#66

Open
tylarpierson wants to merge 13 commits into
bloominstituteoftechnology:masterfrom
tylarpierson:master
Open

Tylar Pierson#66
tylarpierson wants to merge 13 commits into
bloominstituteoftechnology:masterfrom
tylarpierson:master

Conversation

@tylarpierson

Copy link
Copy Markdown

No description provided.

@kiachoi kiachoi left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Great job Tylar! Even you work before Dan's solution. Love all of your commits and team work. Thank you for all your hard work! 🚀

@@ -0,0 +1,35 @@
// function myFunction() {
// document.getElementById('myDropdown').classList.toggle('show')

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Nice thought process here! I would have also probably used the toggle method as well.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Also, thank you for commenting out your prior work! I love seeing you progress and make changes. 🔥

Comment thread index.html
</div>
<div class="Box">
<div class="Box__header">
<h3 class="Box__headerHeadline">Placeholder</h2>

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

You can make thing look something like Box__header--Headline. Just a suggestion but if you will have anything nested, the third element will usually come after --.

Item1__Item2--Item3--Item4 etc

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