Skip to content

Exercise 8#14

Open
powlaa wants to merge 1 commit intomainfrom
exercise-8
Open

Exercise 8#14
powlaa wants to merge 1 commit intomainfrom
exercise-8

Conversation

@powlaa
Copy link
Contributor

@powlaa powlaa commented Feb 2, 2021

No description provided.

Copy link
Contributor

@roschaefer roschaefer left a comment

Choose a reason for hiding this comment

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

nice and easy

Hey @Systems-Development-and-Frameworks/lichtow your PR is nice and easy! We should have had more exercises without dependencies to the previous exercises.

I really like that you used TailwindCSS and I like the simplicity of the DOM compared to all the unnecessary markup that other groups have added, because they used bootstrap.

You recieved 5/5 ⭐ in this exercise:

⭐ For a header and it's responsiveness (direct links vs. hamburger menu)

⭐ For a content section and it's content items

⭐ For a responsive content section (3-column vs 1-column layout)

⭐ For responsive content items (1-column vs. 2-column layout)

⭐ For a footer

:img="require('../assets/img/face3.jpeg')"
title="Do"
>
Populo quaeque constituto ad pro, usu minimum fastidii oportere ne.
Copy link
Contributor

Choose a reason for hiding this comment

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

If it's just text, I would pass it as an object via props:

export default {
  data() {
    return { items: [{title: 'some title', text: 'lengthy text..'}, /* ... */] }
  }
}
<ContentItem v-for="item in items" :key="item.title" :item="item" />

Copy link
Contributor

Choose a reason for hiding this comment

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

Once you have arbitrary HTML it's time for a <slot>.

@roschaefer
Copy link
Contributor

@Systems-Development-and-Frameworks/lichtow your team has reached all 70/70 ⭐ of all homeworks. Amazing!

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