[ENH] Added FormControlLabel for clickable checkbox text

[samadpls]

• Closes [ENH] Make text associated with checkbox clickable #58

Changes proposed in this pull request:

• Added FormControlLabel for clickable checkbox text

Checklist

This section is for the PR reviewer

• PR has an interpretable title with a prefix ([ENH], [FIX], [REF], [TST], [CI], [MNT], [INF], [MODEL], [DOC]) (see our Contributing Guidelines for more info)
• PR has a label for the release changelog or skip-release (to be applied by maintainers only)
• PR links to GitHub issue with mention Closes #XXXX
• Tests pass
• Checks pass

For new features:

• Tests have been added

For bug fixes:

• There is at least one test that would fail under the original bug conditions.

[netlify]

✅ Deploy Preview for neurobagel-query ready!

Name	Link
🔨 Latest commit	99db6a4
🔍 Latest deploy log	https://app.netlify.com/sites/neurobagel-query/deploys/66312531ecf5190008c44508
😎 Deploy Preview	https://deploy-preview-63--neurobagel-query.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site configuration.

[surchs]

Thanks @samadpls for the PR - but I think we need to talk a bit more about what the goal is here. We don't want to add a new label just to make the checkbox work. Maybe @rmanaem has some ideas here as well. But the main goal from a user perspective is "I can click outside the checkbox and the checkbox will still become checked". That can be achieved in a number of ways. Maybe you have some ideas?

[surchs]

This is a nice fix, but we don't want to add a label next to the checkbox, especially not with the datasetName that you can already see on the dataset card. So I think we need to talk a bit more about what to do here. From a UX perspective, the nicest thing might be to make the entire card element clickable to set and unset the checkbox.

[surchs]

@samadpls would you like to either discuss or propose a solution here without adding a new label to the checkbox? Essentially without this:

[samadpls]

@samadpls would you like to either discuss or propose a solution here without adding a new label to the checkbox? Essentially without this:

I agree with your suggestion to avoid adding a new label to the checkbox. Perhaps we can add an onClick event to the Card component or explore other options you recommend.

[rmanaem]

I agree with your suggestion to avoid adding a new label to the checkbox. Perhaps we can add an onClick event to the Card component or explore other options you recommend.

Let's try the onClick event and explore other solutions if that doesn't work out as expected.

[samadpls]

Apologies for any inconvenience caused. Due to time constraints, I won't be able to follow up on this specific PR.😢
However, It was a great experience working with all of you during the GSoC selection process!😊

[rmanaem]

No problem @samadpls, It's understandable. Thanks for contributing to the project!
Please stay in touch and feel welcome to contribute again to our repos!