Make WordPress Core

Opened 11 years ago

Closed 11 years ago

#31202 closed defect (bug) (fixed)

Customizer preview iframe should have a title

Reported by: celloexpressions's profile celloexpressions Owned by: ocean90's profile ocean90
Milestone: 4.2 Priority: normal
Severity: normal Version: 3.4
Component: Customize Keywords: has-patch
Focuses: accessibility Cc:

Attachments (2)

31202.diff (470 bytes) - added by celloexpressions 11 years ago.
Add a title to the customize-preview iframe
31202.2.patch (4.1 KB) - added by afercia 11 years ago.

Download all attachments as: .zip

Change History (9)

@celloexpressions
11 years ago

Add a title to the customize-preview iframe

#1 @westonruter
11 years ago

  • Keywords commit added
  • Owner set to ocean90
  • Status changed from new to reviewing

I was afraid that this change might have resulted in the tooltip appearing whenever you put your mouse over any part of the preview iframe, but this doesn't seem to be the case.

#2 @ocean90
11 years ago

I don't think that the container of the iframe should get the title, rather the iframe itself. See https://core.trac.wordpress.org/ticket/28823#comment:9 for plugin details iframe.

#3 @DrewAPicture
11 years ago

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

See comment:2

This ticket was mentioned in Slack in #core-customize by afercia. View the logs.


11 years ago

@afercia
11 years ago

#5 @afercia
11 years ago

New patch tries to add a title attribute for:

  • main iframe (used just when the customizer overlays an admin page)
  • preview iframe
  • login iframe (shown in lieu of the preview iframe after session expiration)

Please check carefully, I'm not sure about a couple of things, one of them is:

iframe = $('<iframe src="' + self.previewUrl() + '" title="' + wp.customize.l10n.previewIframeTitle + '" />').hide();

why this way to select an already existing iframe? or maybe there are edge case conditions when the iframe needs to be created and suddenly hidden? Definitely don't have a deep knowledge of all the customizer code, but I'm a bit confused about that piece of code.

#6 @joedolson
11 years ago

  • Keywords has-patch added; needs-patch removed

#7 @ocean90
11 years ago

  • Resolution set to fixed
  • Status changed from reviewing to closed

In 31704:

Customizer: Add title attributes to iframes.

props afercia.
fixes #31202.

Note: See TracTickets for help on using tickets.