Opened 8 years ago
Closed 8 years ago
#42210 closed defect (bug) (fixed)
Code Editors: "Help" cursor shows up even when there isn't an error
Reported by: |
|
Owned by: |
|
---|---|---|---|
Milestone: | 4.9 | Priority: | normal |
Severity: | normal | Version: | |
Component: | General | Keywords: | has-patch needs-testing |
Focuses: | ui | Cc: |
Description
Looks like the hover state is being triggered even when an error or warning isn't visible. See video for details: https://cloudup.com/cGLOZtacmg7
Attachments (4)
Change History (13)
This ticket was mentioned in Slack in #accessibility by afercia. View the logs.
8 years ago
This ticket was mentioned in Slack in #core by melchoyce. View the logs.
8 years ago
#3
@
8 years ago
It looks like the cursor was being applied en masse to the left-hand column reserved for error messages. The first patch is wrong (my mistake) but the second one works to only apply that cursor when an error icon is present.
#5
@
8 years ago
Looks good, but the help
cursor has been replaced by pointer
. Mind swapping that back in?
#7
@
8 years ago
Hey @studionashvegas, I think I miscommunicated — I meant that in your original patch, I was seeing pointer
when I hovered over the warnings, rather than seeing help
. The cursor should be help
. I think whatever you were targeting wasn't working correctly in code-editor-help.42210.2.diff.
#8
@
8 years ago
That's so weird. I rolled back my commits (which I used to create that patch) and the cursor is indeed help
...
I'm not sure why it didn't work, but I wrote -another- patch from scratch (it was only changing one line of CSS) and I'm uploading that again. I tried applying this patch too - it works for me.
~Mitch
Patch for #42210