Skip to content

add forms#783

Open
AnnBusya wants to merge 2 commits into
mate-academy:masterfrom
AnnBusya:develop
Open

add forms#783
AnnBusya wants to merge 2 commits into
mate-academy:masterfrom
AnnBusya:develop

Conversation

@AnnBusya

Copy link
Copy Markdown

Comment thread src/index.html Outdated
<fieldset class="rows">
<legend>An interesting fact about you!</legend>

<label class="form-field distance">

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 don't need this label

Comment thread src/index.html Outdated
type="radio"
id="No"
name="loveCat"
>

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 move these inputs to labels

Comment thread src/index.html Outdated

<label class="form-field distance">
Comments:
<textarea></textarea>

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Every field should have name attribute

@AnnBusya AnnBusya requested a review from vlad-demchuk April 27, 2023 08:26
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.

3 participants