Make WordPress Core

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: melchoyce's profile melchoyce Owned by: melchoyce's profile melchoyce
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)

code-editor-help.42210.diff (384 bytes) - added by studionashvegas 8 years ago.
Patch for #42210
code-editor-help.42210.2.diff (384 bytes) - added by studionashvegas 8 years ago.
CORRECT patch to fix help icon on Code Editor
code-editor-help.3.42210.diff (446 bytes) - added by studionashvegas 8 years ago.
Replaces 'help' with 'pointer', by request
42210-code-editor-fix.diff (390 bytes) - added by studionashvegas 8 years ago.
Yet Another 42210 Patch That Should Work This Time

Download all attachments as: .zip

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 @studionashvegas
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.

@studionashvegas
8 years ago

CORRECT patch to fix help icon on Code Editor

#4 @studionashvegas
8 years ago

  • Keywords has-patch needs-testing added; needs-patch removed

#5 @melchoyce
8 years ago

Looks good, but the help cursor has been replaced by pointer. Mind swapping that back in?

#6 @studionashvegas
8 years ago

Absolutely. Patch incoming!

@studionashvegas
8 years ago

Replaces 'help' with 'pointer', by request

#7 @melchoyce
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 @studionashvegas
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

@studionashvegas
8 years ago

Yet Another 42210 Patch That Should Work This Time

#9 @melchoyce
8 years ago

  • Owner set to melchoyce
  • Resolution set to fixed
  • Status changed from new to closed

In 41897:

Code Editors: Fix display of "help" cursor.

Props studionashvegas .
Fixes #42210.

Note: See TracTickets for help on using tickets.