Skip to content
This repository was archived by the owner on Oct 2, 2024. It is now read-only.

Emeka - HTML-CSS-Challenges - WM4#224

Open
Emeka1993 wants to merge 12 commits into
CodeYourFuture:mainfrom
Emeka1993:main
Open

Emeka - HTML-CSS-Challenges - WM4#224
Emeka1993 wants to merge 12 commits into
CodeYourFuture:mainfrom
Emeka1993:main

Conversation

@Emeka1993
Copy link
Copy Markdown

No description provided.

Copy link
Copy Markdown
Member

@SallyMcGrath SallyMcGrath left a comment

Choose a reason for hiding this comment

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

Thanks for this, @Emeka1993 ! I really appreciate the effort you've put in here.

Thinking a bit more strategically about using this exercise to develop your own learning, how might you go back now and revise your code?

Some things to consider:

Are you happy with this design? Would you put this in your professional portfolio? If not, why not, and what will you do about it?
What would you like to learn more about, of the things discussed in this review? What is your plan to develop that expertise?
What did you learn from this exercise already, and what part of your code are you particularly pleased with?
And lastly, what did you learn from reading Zaw's PR and review that you put into practice here?

Comment thread Form-Controls/index.html
</form>

<form>
<label for="Fullname">Full Name:</label>
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

When you ran Lighthouse, what did it say about this line?

https://dequeuniversity.com/rules/axe/4.4/label

Comment thread Form-Controls/index.html
</select>

<label for="date">Collection Date:</label>
<input type="date" id="date" name="date" min="<?php echo date('Y-m-d', strtotime('+4 weeks')); ?>" required>
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I appreciate the innovation @Emeka1993 ;)

But does this line meet all the requirements as stated in the readme?

Comment thread Form-Controls/index.html
<footer>
<!-- change to your name-->
<h2>By HOMEWORK SOLUTION</h2>
<h2>Emeka</h2>
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Comment thread Form-Controls/styles.css
border-radius: 10px;
background-color: white;
margin: 0 auto;
width: 50%;
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

50% of what?

What size would this be on a 340px iPhone 5?

Comment thread Form-Controls/styles.css
cursor: pointer;
}
input[type="submit"]:hover {
background-color: green;
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

What would happen if a user was accessing this without a mouse?

https://css-tricks.com/keyboard-only-focus-styles/

Comment thread Form-Controls/styles.css

input[type="submit"] {
width: 100%;
background-color: orange;
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

When you ran Lighthouse, what did it say about this line?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants