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

Comments

Jeff Burnz’s picture

Jeff Burnz commented

Patches are gooood :)

seandunaway’s picture

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

kingfisher64 commented

Has this been committed?

Jeff Burnz’s picture

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 TN, USA
commented

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