Adaptive Theme uses #main-content and AT Panels Everywhere uses #content

I can roll a patch but which do you prefer? :)

CommentFileSizeAuthor
#2 at_panels_everywhere-7.x-1.x-1733116-2.patch565 bytesseandunaway
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Jeff Burnz’s picture

Jeff Burnz CreditAttribution: Jeff Burnz commented

Patches are gooood :)

seandunaway’s picture

seandunaway CreditAttribution: seandunaway commented
Assigned: Unassigned » seandunaway
Status: Active » Needs review
FileSize
at_panels_everywhere-7.x-1.x-1733116-2.patch565 bytes
kingfisher64’s picture

kingfisher64 CreditAttribution: kingfisher64 commented

Has this been committed?

Jeff Burnz’s picture

Jeff Burnz CreditAttribution: Jeff Burnz commented

Not yet committed, not sure if this is the right fix because it removes an ID that users might be using for CSS or JS in their themes and could break existing sites. I'll have a think about it.

Remember - if you are using AT 7.x-3.x you can alter the skip link target from the appearance settings:

Extensions > Markup Overrides > Accessibility > Skip to navigation target ID

This should serve as a workaround until this is fixed - change it to whatever you need (maybe #content).

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna commented

Why the hesitation to fix a bug in ATPE to make it match AT's output?