D6: overlay-displace-top was implemented but not overlay-displace-bottom, so stuff displayed at the bottom (like the contrib l10n_client module) will roll above or behind the toolbar possibly

CommentFileSizeAuthor
#6 overlay-displace-bottom.patch3.75 KBcasey

Comments

David_Rothstein’s picture

Title: Overlay D6: implement .overlay-displace-bottom » Implement .overlay-displace-bottom
Component: other » overlay.module
Issue tags: -overlay

Is this still an issue?

Looks like there is an overlay-displace-bottom in the code, but it only appears once - I don't have time now to check if that's a complete implementation or not.

casey’s picture

If it was it isn't there any more. I'll provide a patch...

aspilicious’s picture

Still an issue?

aspilicious’s picture

Bump... Can I close this or not?

ksenzee’s picture

No, this is still an active issue.

casey’s picture

Status: Active » Needs review
StatusFileSize
new3.75 KB

Attached patch should make it work. Also replaced "displaced regions" with "displaced elements" for clarity; The word "region" means something else in Drupalsh.

aspilicious’s picture

ksenzee or others can you review this?

cosmicdreams’s picture

#6: overlay-displace-bottom.patch queued for re-testing.

casey’s picture

Status: Fixed » Closed (fixed)

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