materialize icon indicating copy to clipboard operation
materialize copied to clipboard
verified publisher icon Dogfalo

Checkbox Documentation Incorrect?

Open Major-Mike opened this issue 5 years ago • 0 comments

Expected Behavior

Documentation states: image

Current Behavior

But the sample markup doesn't contain these requirements of for or id: image

Possible Solution

I am not sure if that was intentional because it was in the instructions or it was overlooked when the breaking changes for this feature crossed into 1.x but figured I point it out in case.

Steps to Reproduce (for bugs)

Go to https://materializecss.com/checkboxes.html page for the documentation

Context

I noticed it by accident actually, I had read the instructions, so to expedite figured let me copy the snippet and then change it, so I don't miss anything stupid. But as I was editing it realized hmm they mention the for and id being required but it is not in the snippet and some searching seems to imply that was part of the change from previous version. So I figured I mention it in case you need to update that.

Your Environment

These shouldn't be an issue to this, since it is not functional but still.

  • Version used: 1.0.0
  • Browser Name and version: Opera 68.0.3618.173
  • Operating System and version (desktop or mobile): Windows 10 Pro x64 1909 (18363.900)
  • Link to your project (if appropriate): Sorry it is internal covered by NDA :(

Major-Mike avatar Jun 24 '20 17:06 Major-Mike