Posted by ay13 on January 18, 2013 at 8:06pm
4 followers
| Project: | Drupal Commons |
| Version: | 7.x-3.x-dev |
| Component: | Events |
| Category: | bug report |
| Priority: | major |
| Assigned: | jpontani |
| Status: | closed (fixed) |
| Issue tags: | theme blocker |
Issue Summary
Events node has 2 main content regions, one within the other. this makes the main column 66% of 66% of the page. the regions need to be cleaned up.
| Attachment | Size |
|---|---|
| Screenshot_1_18_13_1_58_PM.jpeg | 203.41 KB |
Comments
#1
#2
The issue is that there is a fallback context on node/%node page manager. Events does it's own panelized page, and is thus calling both the Events page handler, as well as the Fallback handler, and doing a 66/33 layout twice.
As far as I can tell, there is no need for the fallback handler as nothing is being displayed in the right (33%) panel, and only the existing node content is being displayed in the left (66%) panel.
Attached a patch that removes the fallback handler.
#3
Instead of #2, switched to full page override for Event node panelizer page. Tested with the fallback context enabled and it works for me.
http://drupalcode.org/project/commons_events.git/commit/eb8b463
#4
This needs to be rendered as node content instead of rendered node. The attached screenshot is the settings from post content type that works.
#5
To be consistent with the other content types, the Panelizer config for events should be their own feature but I'm fine with that being a separate "minor" issue as a followup.
#6
Committed a fix. I will make a new issue to move the panelizer config into a separate sub-module.
http://drupalcode.org/project/commons_events.git/commitdiff/c0db4f7
#7
Automatically closed -- issue fixed for 2 weeks with no activity.