Improves accessibility. I will re-use the block subject.

Patch to follow shortly

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

fullerja’s picture

Issue summary: View changes
fullerja’s picture

Issue summary: View changes
fullerja’s picture

Status: Active » Needs review
FileSize
5.57 KB
drozas’s picture

Excellent, thanks fullerja!

However, the patch you provided is based on an older version of the code (with other default URLs and before #2286343: Wrong variable name in hook_theme() was solved). It would be great if you could re-create it using the latest dev version. If not, I will merge the code myself when I find some time.

Thanks so much for your feedback and contribution!

David

fullerja’s picture

FileSize
3.76 KB

Sorry about that. Re-rolled against latest dev

drozas’s picture

Great, thanks fullerja!

I will test it as soon as I find some time, and commit it to the dev release.

Thanks a lot for the patch! :-)

Cheers,

David

  • drozas committed b0070b9 on 7.x-1.x
    Issue #2311281 by fullerja: Added title attribute to the iFrame to...
drozas’s picture

Issue tags: +LONDON_2014_AUGUST
drozas’s picture

Applied patch to 7.x-1.x branch. Thanks fullerja! Release including it coming soon...

drozas’s picture

Status: Needs review » Fixed
drozas’s picture

Patch included in new release: 7.x-1.10 - https://www.drupal.org/node/2322859

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.