In the settings/gallery/install page the status tables have an id associated with them (line 135 of gallery_install.inc for example). Some of the ids generated may be common (eg 'search') which can conflict with other elements on the page (eg my search box) and so end up being formatted incorrectly. I'm not sure why ids are needed (?), but maybe they could be made more unique (eg gallery-search ...)

Comments

profix898’s picture

profix898 CreditAttribution: profix898 commented

The id was originally added to allow links from the "short status" on "General" tab (or somewhere else) to point directly to the problematic item in the table (similar to the links pointing to the access control table in Drupal core). And the url parameter for this is still available, but its actually not used anymore.
However I not convinced this is really needed, so we should simply eliminate the ids (and probably the url parameter as well).

profix898’s picture

profix898 CreditAttribution: profix898 commented
Status: Active » Fixed

Upps! I was wrong. The code to jump directly to a table row is still there. I just committed a patch which simply prefixes the ids like this: 'gallery-install-plugin-'. $key. I think that should be OK!?

Anonymous’s picture

Anonymous (not verified) CreditAttribution: Anonymous commented
Status: Fixed » Closed (fixed)

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