Hello, I installed the recent Skinr version for D7 but it seems that there is no integration with Fusion. Anyone knows anything about that? Thanks

CommentFileSizeAuthor
#7 patch_commit_4f86b1fc1ba4.patch96.85 KBHari
#3 fusion_skins_02_4_11.zip36.32 KBHari
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

stephthegeek’s picture

stephthegeek CreditAttribution: stephthegeek commented
Title: Skinr compatibily in D7 » Skinr compatibility in D7
Category: bug » task
Priority: Major » Critical

Skinr does not currently have a functional release for Drupal 7. It's been undergoing a major rewrite. We're tracking it closely and Skinr support (better than ever!) will be back in Fusion as soon as it's possible.

If anyone wants to help this along, reviewing/writing patches for Skinr would speed things up! http://drupal.org/project/issues/skinr?text=&status=Open&priorities=All&...

daniorama’s picture

daniorama CreditAttribution: daniorama commented
Version: » 7.x-1.x-dev

Hello Stephen, sadly I do not have enough experience with Skinr to help :( But I can help with the testing. I'm updating Fusion and Skinr as soon as new releases come, so I will tell you if I encounter any problems. I have the 7.1 dev version of Fusion right now.

Hari’s picture

Hari CreditAttribution: Hari commented
Component: User interface » Code
Priority: Critical » Normal
Status: Active » Needs work
FileSize
fusion_skins_02_4_11.zip36.32 KB

I have been working on a port of the fusion skins for the new Skinr. Attached are two skins to be placed in a folder named skins in the fusion_core directory. The skins are:

  1. Fusion (the ported skin)
  2. Wires (A skin that I have been using to visually see that the skins are working and something is happening)

There are several parts that need to be fixed that are above my skill level. I have commented them in the fusion.inc(the skin) file. As far as I can see they are all related either to JS or changes that need to made to template.php.

The following lines need to be added to the fusion_core.info file:

skinr[api] = 2
skinr[directory] = skins

I can't take the development any further than this but hope that this helps.

Hari’s picture

Hari CreditAttribution: Hari commented

I'll submit it as a patch as soon as I figure out how to make the patch.

ogi’s picture

ogi CreditAttribution: ogi commented

subscribe

bkmarsh’s picture

bkmarsh CreditAttribution: bkmarsh commented

subscribe

Hari’s picture

Hari CreditAttribution: Hari commented
FileSize
patch_commit_4f86b1fc1ba4.patch96.85 KB

This patch is a port of the skins from fusion for D6 to fusion for D7. The patch converts the skins that were previously in the .info file in D6 into a Skinr compatible skin for Fusion D7. The only things missing from this patch are proper JS integration.

Here is what is working:

  • Widths for 16 column, 960px/fluid grid
  • Widths for 12 column, 960px/fluid grid
  • Content alignment
  • General styles
  • List styles
  • Additional menu styles
  • Stackable list/menu styles

Here is a list of what needs to be worked on:
located in skins/fusion/fusion.inc:

  • Block positioning starting Line: 176
  • General style - Equal heights Line: 259
  • Superfish menu styles Line: 341
  • Image styles Line: 359
  • User login block styles Line: 382
stephthegeek’s picture

stephthegeek CreditAttribution: stephthegeek commented

Thanks for the help Hari!! This is a great headstart and we'll be working this soon since Skinr is getting solid.

Hari’s picture

Hari CreditAttribution: Hari commented

Your welcome :D

drupalicious’s picture

drupalicious CreditAttribution: drupalicious commented

subscribe

stephthegeek’s picture

stephthegeek CreditAttribution: stephthegeek commented
Status: Needs work » Fixed

Status: Fixed » Closed (fixed)

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