Well as any good programmer you learn new ways of doing things and more efficient ways of completing a task every day (or week :P). Since my original work in making pfSense themeable I have found a couple of ways to "improve" upon this.
My first and only idea that I am going to share at this point with you is basically the concept Firefox uses with extension development. See Figure 1 for an example of the heirachical (sp?) structure. Basically I am thinking of making the themeing system use an xml file to contain the "elements" of the theme and just have the webgui read the xml file to find out what CSS files need to be included, javascript, etc ... overall or per page. ++++++++++++++++++ + Figure 1.1 + +----------------+ + /themes + + /metallic + + /css + + /images + + /javascript + + metallic.xml + ++++++++++++++++++ Comments? :) (I know GeekGod will have plenty, as this is the first time he is seeing this idea :P) -Erik ----------Original Message---------- > I appreciate that :) > > Let's let Erik chime in on how we should approach fixing this. > > > On 1/12/06, Rajkumar S <[EMAIL PROTECTED]> wrote: > > Scott Ullrich wrote: > > > On 1/11/06, alan walters <[EMAIL PROTECTED]> wrote: > > >> It seems on error or when settings are saved there are some bits of the > > >> themining that is in the code rather than in the style sheet can someone > > >> else confirm this. > > > > > > That is correct. > > > > > >> Would there be plans to clean this up before q release > > > > > > Doubtful. > > > > The fix is bit difficult as Rounded is called to create the round > > effect in the menus. What you can do is to grep for "Rounded" in the php > > scripts and replace the colours manually. I have done one theme like > > that but I cannot distribute since to change the theme also requires to > > change the php code. I am willing to create a patch to fix this, if the > > core team can point towards a "blessed" way to do this. :) ie, I do not > > want to fix this in a way that is incompatible with project direction, > > leading to a rejected patch. > > > > raj > > > > --------------------------------------------------------------------- > > To unsubscribe, e-mail: [EMAIL PROTECTED] > > For additional commands, e-mail: [EMAIL PROTECTED] > > > > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: [EMAIL PROTECTED] > For additional commands, e-mail: [EMAIL PROTECTED] > > --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
