Skip to content

add task solution#788

Open
IvanRovenko wants to merge 1 commit into
mate-academy:masterfrom
IvanRovenko:develop
Open

add task solution#788
IvanRovenko wants to merge 1 commit into
mate-academy:masterfrom
IvanRovenko:develop

Conversation

@IvanRovenko

Copy link
Copy Markdown

No description provided.

Comment thread src/style.css
@@ -1 +1,17 @@
/* styles go here */
.body {

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Use 'body' instead of '.body' to select the body element.

Comment thread src/index.html
How old are You?
<input
type="number"
name="How old are You?"

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Use a more descriptive name for the 'name' attribute. For example, 'age'.

Comment thread src/index.html
<label>
<input
type="radio"
name="Yes"

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Radio buttons for the same question should have the same 'name' attribute. Change the 'name' attribute to 'loveCats' for both 'Yes' and 'No' options.

Comment thread src/index.html
Full date of birth:
<input
type="date"
name="Full date of birth"

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Use a more descriptive name for the 'name' attribute. For example, 'dateOfBirth'.

Comment thread src/index.html
How do you rate our work?
<input
type="range"
name="rate worl"

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Use a more descriptive name for the 'name' attribute. For example, 'workRating'.

Comment thread src/index.html
<select name="car" multiple>
<option value="BMW">BMW</option>
<option value="Audi">Audi</option>
<option value="Jeep">Lada</option>

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

The option text should match the value attribute. Change the option text to 'Lada' instead of 'Jeep'.

Comment thread src/index.html
I accept the term of the agreement
<input
type="checkbox"
name="I accept the term of the agreement"

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Use a more descriptive name for the 'name' attribute. For example, 'termsAgreement'.

Comment thread src/index.html
Comment on lines +19 to +26
<div class="form-field">
Surname:
<input
type="text"
name="Surname"
autocomplete="off"
>
</div>

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

  1. use label instead of div
  2. apply code styling recommendations
Suggested change
<div class="form-field">
Surname:
<input
type="text"
name="Surname"
autocomplete="off"
>
</div>
<label class="form-field">
Surname:
<input
type="text"
name="Surname"
autocomplete="off"
>
</label>

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.

Hi, i need chenge oll div to label ?

Comment thread src/style.css
Comment on lines +5 to +17
.form {
margin-bottom: 20px;

}

.form-field {

padding-bottom: 10px;
}

.form-field:last-child {
padding-bottom: 0;
}

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

no needs to add all of these classes.
You may only apply those to your filedset and form field (label or div)
Example:

.form-fieldset:not(:last-child) {
  margin-bottom: 20px
}

.form-field:not(:last-child) {
  margin-bottom: 10px
}

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