Make WordPress Core

Opened 12 years ago

Closed 12 years ago

#29725 closed defect (bug) (fixed)

Media library arrow key navigation is triggered in textareas

Reported by: essha Owned by: wonderboymusic
Priority: low Milestone: 4.0.1
Component: Media Version: 4.0
Severity: trivial Keywords: has-patch fixed-major
Cc: Focuses: accessibility, javascript

Description

This is a follow-up to #28704.
Event is fired on arrow keys when you enter textarea, i.e. caption and description.
Please exlude TEXTAREA from event trigger as well.
https://core.trac.wordpress.org/attachment/ticket/28704/28704.diff

Attachments (1)

29725.diff​ (615 bytes ) - added by ryelle 12 years ago.

Download all attachments as: .zip

Change History (12)

@ryelle
12 years ago

#1 @ryelle
12 years ago

  • Summary Media library arrow keys: exclude nodeName → Media library arrow key navigation is triggered in textareas

Ran into this problem last night- the keyEvent function returns when the event target is an input, but not on textareas, so editing the captions becomes very frustrating (since the attachment changes out from under you).

29725.diff​ adds textarea to the conditional, and also switches to nodeName based on azaozz's comment on r29332.

#2 @ocean90
12 years ago

#29776 was marked as a duplicate.

#3 @ocean90
12 years ago

  • Focuses accessibility javascript added
  • Keywords has-patch commit added
  • Milestone Awaiting Review → 4.0.1
  • Priority normal → low
  • Severity normal → trivial
  • Type enhancement → defect (bug)

#4 @wonderboymusic
12 years ago

  • Owner set to wonderboymusic
  • Resolution → fixed
  • Status new → closed

In 29777:

Media Grid: when the modal is open, don't respond to arrow keys when <textarea> has focus.

Props ryelle.
Fixes #29725.

#5 @ocean90
12 years ago

  • Keywords fixed-major added; commit removed
  • Resolution fixed
  • Status closed → reopened

This ticket was mentioned in ​Slack in #core by jorbin. ​View the logs.


12 years ago

#7 @jorbin
12 years ago

I verified that r29777 applies correctly to the 4.0 branch and fixes the described issue. Tested in chrome 38 and 40 along with FF.

This is ready to be merged.

This ticket was mentioned in ​Slack in #core by ocean90. ​View the logs.


12 years ago

#9 @johnbillion
12 years ago

In 30378:

Update the inline docs for the keyEvent callback in the media grid.

Props adamsilverstein
See #29725

#10 @adamsilverstein
12 years ago

#30400 was marked as a duplicate.

#11 @nacin
12 years ago

  • Resolution → fixed
  • Status reopened → closed

In 30407:

Media Grid: when the modal is open, don't respond to arrow keys when <textarea> has focus.

Merges [29777] (and [30378]) to the 4.0 branch.

Props ryelle, adamsilverstein.
Fixes #29725.

Note: See TracTickets for help on using tickets.