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

Add an example for for#2376

Merged
queengooborg merged 21 commits intomdn:mainfrom
MrBrain295:patch-3
Aug 2, 2023
Merged

Add an example for for#2376
queengooborg merged 21 commits intomdn:mainfrom
MrBrain295:patch-3

Conversation

@MrBrain295
Copy link
Copy Markdown
Contributor

@NiedziolkaMichal
Copy link
Copy Markdown
Member

Currently, this example doesn't show how attribute for is useful. You have also set its content to the value of attribute id, while name should be used instead. I think that the value of property height is currently invalid for this amount of content.

@NiedziolkaMichal NiedziolkaMichal mentioned this pull request Jan 6, 2023
94 tasks
@MrBrain295
Copy link
Copy Markdown
Contributor Author

@NiedziolkaMichal I have made the requested changes.

@NiedziolkaMichal
Copy link
Copy Markdown
Member

@MrBrain295 Sorry but I don't think that you have. The value of attribute for is still pointing to name instead of id and the example doesn't show how this attribute is useful. You should also get rid of form element since we don't use it for similar examples.

@MrBrain295
Copy link
Copy Markdown
Contributor Author

I’ve applied the feedback except for demonstrating it’s usefulness which I don’t know how I should do.

@NiedziolkaMichal
Copy link
Copy Markdown
Member

It would be great if your CSS would have an immediate visual impact. Additionally, the attribute selector might be confusing to the user when the name of the attribute is for.
How about something like that?
image
When the reader will click on the labels, he/she might notice that label without the attribute for is not setting the focus state to the control field. Additionally, usage its usage in the output element is shown. This example should also have JS code though because output is supposed to be an interactive element.

@github-actions github-actions Bot added the idle Issues and pull requests with no activity for three months. label Feb 18, 2023
@estelle
Copy link
Copy Markdown
Member

estelle commented Jul 26, 2023

i added your suggestions. Now we just need to add the JS file.

@github-actions github-actions Bot removed the idle Issues and pull requests with no activity for three months. label Jul 27, 2023
@Josh-Cena Josh-Cena added the Content:HTML issues related to HTML examples. label Jul 31, 2023
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Aug 2, 2023

This pull request has merge conflicts that must be resolved before it can be merged.

@queengooborg queengooborg requested a review from a team as a code owner August 2, 2023 15:02
@queengooborg queengooborg requested review from estelle and removed request for a team August 2, 2023 15:02
@queengooborg
Copy link
Copy Markdown
Collaborator

Thanks everyone -- because this has been sitting around for a while, I've decided to go ahead and make the requested changes. This is now ready to go!

@queengooborg queengooborg merged commit 755b4ba into mdn:main Aug 2, 2023
@MrBrain295 MrBrain295 deleted the patch-3 branch August 2, 2023 15:10
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Content:HTML issues related to HTML examples.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants