Issue Details (XML | Word | Printable)

Key: FOR-811
Type: Sub-task Sub-task
Status: Reopened Reopened
Priority: Major Major
Assignee: Gavin
Reporter: David Crossley
Votes: 0
Watchers: 0
Operations

If you were logged in you would be able to see more operations.
Forrest
FOR-808

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

Created: 08/Feb/06 01:47 PM   Updated: 13/May/09 03:50 AM
Return to search
Component/s: Plugin: internal.dispatcher, Skins (general issues)
Affects Version/s: None
Fix Version/s: None

Time Tracking:
Not Specified

File Attachments:
  Size
Text File Licensed for inclusion in ASF works pelt.fv.diff 2006-02-13 10:53 PM Gavin 7 kB
Text File Licensed for inclusion in ASF works pelt.screen.css.diff 2006-02-13 10:53 PM Gavin 13 kB
Text File Licensed for inclusion in ASF works siteinfo-credits.ft.diff 2006-02-13 10:53 PM Gavin 1 kB
Issue Links:
Blocker
Reference
 


 Description  « Hide
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.

 All   Comments   Work Log   Change History   Subversion Commits      Sort Order: Ascending order - Click to sort in descending order
Thorsten Scherler added a comment - 09/Feb/06 07:49 AM
Why not using the current pelt theme for this?

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

David Crossley added a comment - 09/Feb/06 09:12 AM
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 added a comment - 13/Feb/06 07:29 PM
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 added a comment - 13/Feb/06 10:53 PM
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 added a comment - 27/Feb/06 01:35 AM
Applied patch for FOR-811 provided by Gavin.

Thx Gavin. :)

Further enhanced and polished the pelt theme.

Please review.

Gavin added a comment - 15/Mar/06 09:34 PM
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 added a comment - 01/May/06 06:37 AM
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 added a comment - 01/May/06 07:04 AM
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 added a comment - 04/Mar/07 06:40 AM
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 added a comment - 05/Mar/07 07:41 AM
Remove 'Patch Available' option, these have been applied.


Gavin added a comment - 23/Jun/07 09:20 AM
Last remaining Issue is that of subtabs not displaying correctly.
I think we need to expand the samples in each subtab to separate them.