- Issue created by @djsagar
- ๐ฎ๐ณIndia TanujJain-TJ
Tanuj. โ made their first commit to this issueโs fork.
- Merge request !64763419531 - removing form label cursor pointer โ (Open) created by Unnamed author
- Status changed to Needs review
10 months ago 7:31pm 6 February 2024 - ๐ฎ๐ณIndia TanujJain-TJ
Removed cursor pointer for form label, it was not related to only blocks but this was throughout Drupal in editing node form or having exposed form in views. created a MR for this please review.
- Status changed to Needs work
10 months ago 11:58pm 6 February 2024 - ๐บ๐ธUnited States smustgrave
Seems to have a test failure.
But not 100% sure this is a bug when I hover over a label yes I see the pointer but when I click it focus goes to the text box, or checks a box.
- ๐บ๐ธUnited States pcate
do not display a cursor pointer when a checkbox toggle is not present, aligning with Drupal coding standards
@djsagar, can you provide a link to the Drupal coding standards where this is stated?
- ๐ฎ๐ณIndia djsagar
@PCate, Hovering over labels displays the pointer cursor, while clicking focuses on text boxes or checks boxes. However, the pointer cursor should be limited to specific label elements, rather than being displayed universally across the site.
- ๐บ๐ธUnited States pcate
Hovering over labels displays the pointer cursor, while clicking focuses on text boxes or checks boxes.
It is common for form labels to be styled to display a pointer cursor when hovering over them to give a visual indicator that the labels are interactive. Form labels with a
for
attribute should be interactive.However, the pointer cursor should be limited to specific label elements, rather than being displayed universally across the site
What is the criteria for whether a label should use the pointer cursor? Is this documented somewhere?
Right now Claro is showing the pointer cursor for elements with the
.form-item__label
class and afor
attribute. Are you seeing this class and attribute being added to non-label form elements?I'm seeing the same CSS for form labels is in the core Olivero theme as well.
- ๐ฎ๐ณIndia djsagar
@PCate,
Thank you for the knowledge transfer.
I have also verified this issue with the core Olivero theme. Therefore, we can proceed to close this issue. - Status changed to Closed: works as designed
10 months ago 1:52pm 8 February 2024