sWmode Opaque / Caching / Code-Cleanup

AdrianB - January 15, 2007 - 14:13
Project:sIFR
Version:HEAD
Component:Code
Category:task
Priority:normal
Assigned:sun
Status:closed
Description

I needed to set sWmode to "opaque" but I coudn't find the setting for this. There is a "transparent" option for sWmode already, so it should be a minor change. (I did it myself as an ugly hack and it worked.)

Why opaque mode? It's sometime needed to place menus (like nice menus) above sIFR text (and other flash movies), especially in IE/win.

This is my first issue here, so be gentle if I done something wrong.

#1

sun - February 20, 2007 - 20:40
Title:Adding "opaque" option for sWmode» sWmode option Opaque
Version:4.7.x-1.x-dev» HEAD
Component:User interface» Code
Assigned to:Anonymous» sun
Status:active» needs review

Attached patch adds support for opaque. Note: It's for HEAD/5.x.

AttachmentSize
sifr.module.opaque.patch 1.77 KB

#2

sun - February 20, 2007 - 22:21
Title:sWmode option Opaque» sWmode Opaque / Caching / Code-Cleanup
Category:feature request» task
Status:needs review» reviewed & tested by the community

Whilst working on this issue I've recognized that the output of sIFR isn't cached at all, but could be, since it's usually setup just once for a site. So the attached patch additionally introduces a (variable) cache for sIFR.

Furthermore, the generation of a sIFR rule was a bit hacky. So this patch also optimizes that code for better readability.

@jjeff: Are the changes ok with you?
I know, mixing several issues in one patch isn't welcome. So please don't blame me for that. If the proposed changes are ok with you, I'll commit that patch to HEAD and 5.x.

AttachmentSize
sifr.module.properties-n-cache.patch 4.74 KB

#3

sun - February 24, 2007 - 19:57
Status:reviewed & tested by the community» fixed

Committed to HEAD and 5.x.

#4

Anonymous - March 10, 2007 - 20:00
Status:fixed» closed
 
 

Drupal is a registered trademark of Dries Buytaert.