shine and polish the pelt theme so that it closely resembles the pelt skin

• Key: FOR-811
• Type: Sub-task
• Status: Reopened
• Priority: Major
• Assignee: Gavin
• Reporter: David Crossley
• Component/s: Plugin: internal.dispatcher, Skins (general issues)

Description

So that they can just switch from "pelt skin" to "pelt theme" and not get any surprises, e.g. "pelt skin' has no "xml doc" link.

Comments

[Thorsten Scherler]

Why not using the current pelt theme for this?

Just comment out the contacts that are not in the pelt skin.

[David Crossley]

Changed Summary to better describe the issue.

It doesn't need to exactly match, just no surprises for existing users. Where possible we will refine it to have better CSS and hooks.

[Thorsten Scherler]

We have to finish FOR-811 before investing time to solve this (FOR-816) issue.

More information http://marc.theaimsgroup.com/?l=forrest-dev&w=2&r=1&q=b&s=FOR-816

[Gavin]

3 Attatchments go a way towards addressing this issue. There are still some minor adjustments to make, however what has been done needs checking first. The CSS file pelt.basic.css has been done away with and any applicable CSS from it transferred into pelt.screen.css. Some changes were required also to pelt.fv and siteinfo-credits.fv.

Please check & Test, I have tested these files using a dispatcher enabled seed-sample site using forrest run.

[Thorsten Scherler]

Applied patch for FOR-811 provided by Gavin.

Thx Gavin. :)

Further enhanced and polished the pelt theme.

Please review.

[Gavin]

There is a width problem concerning the tables in IE6, they are too wide making the pages scrollable horizontally, it's fine in Firefox. There is a hack in the CSS already to cure this, but appears not to be working, I will investigate and then provide a patch.

This issue should then be closed so as to not hold up FOR-816 as I think everything else looks ok.

[Thorsten Scherler]

David changed status to reopen on the 27.02.06.

However the attached patches are applied.

David, what is missing to resolve this issue?

[David Crossley]

Well Gavin's previous comments said that patches were partially addressing the issues, so someone needs to check to see what else. Then there is the tables issue that he mentions. Also last time i looked there were some defaults that were different, e.g. IIRC two search boxes.

[Gavin]

More enhancements committed, mainly the nav & tab menu font colors etc, added in an optional <div> area below the navigation.
The Table problem I described seems to have gone.

More to do, which I'll describe on list.

Gav...

[Gavin]

Remove 'Patch Available' option, these have been applied.

[Gavin]

Last remaining Issue is that of subtabs not displaying correctly.
I think we need to expand the samples in each subtab to separate them.