Skip to content

EU FLEX - User Interface II - Zakaria TOUZRI 🇫🇷#1

Open
killinq-joke wants to merge 4 commits intomasterfrom
zakaria-touzri
Open

EU FLEX - User Interface II - Zakaria TOUZRI 🇫🇷#1
killinq-joke wants to merge 4 commits intomasterfrom
zakaria-touzri

Conversation

@killinq-joke
Copy link
Owner

No description provided.

@killinq-joke killinq-joke changed the title Add the h3's EU FLEX - User Interface II - Zakaria TOUZRI :dp-tn: Dec 4, 2019
@killinq-joke killinq-joke changed the title EU FLEX - User Interface II - Zakaria TOUZRI :dp-tn: EU FLEX - User Interface II - Zakaria TOUZRI :flag-fr: Dec 4, 2019
@killinq-joke killinq-joke changed the title EU FLEX - User Interface II - Zakaria TOUZRI :flag-fr: EU FLEX - User Interface II - Zakaria TOUZRI FR Dec 4, 2019
@NabeelahY NabeelahY changed the title EU FLEX - User Interface II - Zakaria TOUZRI FR EU FLEX - User Interface II - Zakaria TOUZRI 🇫🇷 Dec 5, 2019
@NabeelahY NabeelahY self-requested a review December 5, 2019 09:39
- [x] Go to your [index.html](great-idea-website/index.html) file and build out your HTML structure. You have been provided all the content and images you need to get started inside the `index.html` file. Start to code HTML around the content!

- [ ] Style your HTML using the [desktop design file](design-files/desktop.jpg) as a visual guide. Use any CSS properties you want for general styling but for layout, you can only use the box model (content, padding, margin, border) and `display:inline-block`.
- [x] Style your HTML using the [desktop design file](design-files/desktop.jpg) as a visual guide. Use any CSS properties you want for general styling but for layout, you can only use the box model (content, padding, margin, border) and `display:inline-block`.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Awesome 👌🏽

display:inline;
}

div{
Copy link
Collaborator

Choose a reason for hiding this comment

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

This div style might override other styles. I'll advise that you give each div their own class to make it easily selectable.

button{
border:1px solid black;
padding:10px 50px;
margin-bottom: 40%;
Copy link
Collaborator

Choose a reason for hiding this comment

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

I'll advise that you add bottom margin to the container wrapping the button and text. Easier to style

</section>
<section>
<h2 class="contact">Contact</h2>
<ul> 123 Way 456 Street</ul>
Copy link
Collaborator

Choose a reason for hiding this comment

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

Remeber <ul> is used to wrap <li> items. Like this

<ul>
  <li></li>
  <li></li>
</ul>

@NabeelahY
Copy link
Collaborator

The final webpage looks great! Just a few tweaks and its good to go. Great job 👍🏽

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