I am checking out the new (great looking and better flowing) Panels but stumbled upon some kind of strange misbehaviour. Panels shows the Rounded Corners ok, but completly lacks the background colour. See the attached pics (old shows the pane before upgrading and new after the upgrade; it also shows the only correct pane). I would guess that some of the massive changes in Panels/Ctools caused this but strangely _ONE_ block is displayed always correctly. It is a content pane created by views. Other panel panes don't work as expected.
A quick glance with firebug showes that the panes that are incorrectly displayed miss the div. node h2 title stuff. CSS is some kind of mystery to me so that I am sorry that I cannot provide more precise information.
I tried all kind of setting variations but to no avail. Therefore I am pretty sure either acquia_marina needs some pimping or Panels. If it is some kind of misconfiguration on my site I am sorry.
I absolutly love the fact that the panes could be displayed in block style (Title on green background) and hope this can be reintroduced.
| Comment | File | Size | Author |
|---|---|---|---|
| #26 | am_panels-edit.png | 66.21 KB | jeremycaldwell |
| #24 | am_panels.png | 115.64 KB | jeremycaldwell |
| #4 | css-patch-rej.txt | 1.17 KB | dddave |
| #2 | tnt_corners_css.patch | 1.17 KB | jwolf |
| Panels-new.png | 5.32 KB | dddave |
Comments
Comment #1
dddave commentedPffff,
found these entries in the watchdog. Might be helpful. ;)
Quite a bunch of these:
And also two of those:
Comment #2
jwolf commentedComment #3
dddave commentedThx. I am going to test it tomorrow.
Comment #4
dddave commentedpatch failed :(
Don't know the proper report procedure so I attached a copy of the .rej's content. Anything else I can do?
Comment #5
jwolf commentedI tested the patch and it does work:
- Put the patch file in the plugins/styles/tnt_corners/ directory
- patch -p0 < tnt_corners_css.patch
Anyway, below is the full modified tnt_corners.css file:
Comment #6
dddave commentedHm... I retried and the patch failed again. Thanks for including the modified file. I swapped the files and it seems to be working well. All the beautiful green backgrounds are back and everything seems to behave fine. EXCEPT for this strange pane which was displayed correctly in the first place. This pane has lost the background. I am going to take a deeper testing tomorrow.
For now this patch looks good. Thanks!
Comment #7
dddave commentedTested in FF3, IE8 and Safari and everything works fine and looks beautiful. Don't know if I count as "the community" but for me it works. Thx!
Comment #8
dddave commentedSorry for the status change. Although the blocks are looking nice again something still is wrong. I keep getting bunches of errors like those above upon reordering stuff on panel pages.
Cannot make out if it is due to Panels or due to something of this theme (but I suspect it to be the theme as the the error message suggests it to my laymans eyes).
Comment #9
jwolf commentedI don't know why you're getting those errors. However I can verify that I am not seeing this problem.
I suspect that there is an issue with paths on your server. Specifically, I see in your posting above that:
/sites/all/modules/panels/plugins/styles/tnt_corners/acquia-marina-tnt-corners-box.tpl.php is not the correct location for that file. All acquia_marina panels files/styles MUST be located in the theme's plugins dir as they are in the 6.x-2.x-dev snapshot of the theme.
I also suspect that since you reported that the above patch, with instructions on how to apply the patch, does not work for you that you have a modified version of Acquia Marina 6.x-2.x-dev. You may want to try and downloading a fresh copy of the dev snapshot, clear data cache, and try again.
Comment #10
dddave commentedWell the errors above came indeed from a quite messy testinstall on my laptop yet I am also getting those kinds of errors from another install were I am quite confident to not have messed up things. I will take a look into it over the weekend because I really don't understand how this path comes up. It is working nicely just these messages are disturbing.
As far as patching is concerned, I normally don't have problems with it. So this indeed points to some problems on my sites.
Comment #11
dddave commentedI think I found the (pretty obvious) solution. I tested and tested and finally noticed that the errors seemed only to occur when I used panels "update and preview" option instead of the ordinary "update" option. So my theory quickly was, that something breaks during the preview process. Considering that I don't use Acquia_Marina on my admin pages and instead make use of Garland as admin theme points the searchlight pretty well. I switched to Acquia_Marina also on the admin pages and the problem went away.
I don't know if this problem is more one of the preview process of panels and if it can be solved within this theme but I am glad that I finally could make some sense of it.
Comment #12
jwolf commented@dddave - thanks for the great troubleshooting feedback; very helpful.
We're in the process of porting Marina over to our new theme system, Fusion.
I'm marking this as postponed because the current 2.x branch code will be replaced with Fusion and Panels 3 is still evolving.
Comment #13
jwolf commentedComment #14
mariagwyn commentedI have downloaded the most recent dev version, which does have the styles in the appropriate theme folder. But I still get this error:
warning: include(./sites/all/modules/panels/plugins/styles/tnt_corners/acquia-marina-tnt-corners-box.tpl.php) [function.include]: failed to open stream: No such file or directory in /Users/.../Sites/acquia-dpl/acquia-drupal/includes/theme.inc on line 1020.
Comment #15
dddave commentedAre you using a admin theme so that all your panel edit work happens with a non aquia_marina theme? If that is the case you have the same situation as descibed above.
Comment #16
andraeray commentedAre the pictures still there in the tnt corners patch? I don't see my user menu having the star, while using the panels 3 display.
Comment #17
ayalon commentedI had the same problem with my own styles. It cannot work becaus of this line:
'path' => panels_get_path('plugins/styles/tnt_corners'),
This will get the path relative to the panels module folder and will fail because the styles are in the templates folder.
I wrote a simple workaround:
1#: Add the following function to you theme template.php
function mytheme_get_path($file, $base_path = FALSE, $theme = 'mytheme') {
$output = $base_path ? base_path() : '';
return $output . drupal_get_path('theme', $theme) . '/' . $file;
}
2#. Change the line to:
'path' => mytheme_get_path('plugins/styles/tnt_corners');
Comment #18
jwolf commentedComment #19
jwolf commentedI've done a fresh install of Drupal. Checkout the latest version of panels 3 (6.x-3.2) and ctools (6.x-1.2). I created several different panel pages, mini panels from scratch and I am not seeing these error messages.
Any chance you are using the dev snapshot of ctools?
Comment #20
dddave commentedI am retagging and reactivating this issue because it perfectly fits my description in the OP. Basically rounded corners in Panels aren't working at the moment (Skinr 1.5, Ctools 1.3, Panels 3.3, Fusion 1.0 rc1)
and the skinr settings for Acquia Marina - Block Title Styles ( such as "Blue rounded title background with white text" or
"Green rounded title background with white text") are missing.
Therefore Panels is pretty lame at the moment with this theme. When I put some other (none views) blocks into the pane I get the Block Title Style options however they aren't applied.
Please, please reintroduce these great settings for panes.
Comment #21
dddave commentedbtw: I noticed the bunches of Panels+Skinr issues but I am not sure if the problems described above are purely skinr issues. AM3 seems to lack on this front.
Comment #22
dddave commentedFor the record: Tested with Fusion core and the result is the same. Important for me is that Acquia Marina Block Style Settings makes a comeback soon. ;)
Comment #23
dddave commentedAre the block title style settings to be reintroduced in the next release (btw. is there an ETA?)?
Comment #24
jeremycaldwell commentedI worked on this issue quite a bit this morning and have all the Skinr styles supplied in the theme working within Panels now. Attached is screenshot of it in action :)
We need to review these changes and test them locally a bit more before we release them, but it's on the way real soon.
Comment #25
dddave commentedGreat, great news.
You guys and gals really rock!
Comment #26
jeremycaldwell commentedAlso I have the edit options overlay working and not being hidden by the other regions. Attached is a screenshot. Hopefully we'll be able to test these real quick and get them in the next update of the theme.
Comment #27
dddave commentedYes, that would be a nice fix for this usability problem.
Looking forward to it.
Comment #28
jeremycaldwell commentedFixed in the Alpha 2 release.