Is PE going to be supported on D8 as well?

Comments

DamienMcKenna’s picture

Title: Future Proof? » Drupal 8 port of Panels Everywhere
Category: Support request » Task
Issue tags: -D8 +Drupal 8
Related issues: +#2169319: [meta] Drupal 8 port of Panels

I'm personally too busy right now working on D7 contrib projects to consider it.

DamienMcKenna’s picture

FYI dsnopek worked out a proof of concept of Panels Everywhere for D8: https://github.com/dsnopek/panels_everywhere_poc

Needless to say, it'll definitely be possible and will be worked on once the port of Panels itself is done.

evanjenkins’s picture

I have forked dsnopek's repo and started some more work here https://github.com/evanjenkins/panels_everywhere... I didn't have to change much but this is working so far with all the latest required modules.

DamienMcKenna’s picture

@evanjenkins: That's excellent! Would you mind adding a README.txt that would provide some details of what works and how to use it? Thanks.

dsnopek’s picture

For the record: while my PoC (and @evanjenkins fork) use Page Manager, I really think that PE should define it's own config entity for storing variants and then use composition to pull in the Panels components and UIs that it needs. I discussed this with @merlinofchaos on IRC at some point in the past, and he lamented that D7 PE used Page Manager and agreed that this would be a better implementation.

dsnopek’s picture

Also, the stuff we're doing in PanelsEverywhereDisplayVariant to implement PageVariantInterface and support page title and main content should just be merged into PanelsDisplayVariant in Panels. Even though it's PE that is giving the variant to core to render the page, (1) that interface is from core, (2) it's easy to implement and (3) other modules could potentially want to pass PanelsDisplayVariants to core that don't come from PE, and I think Panels should support that. This should be made into an issue in the Panels queue.

evanjenkins’s picture

I agree! We will probably continue to push this forward and refactor. Project needed a different site template based on a context and your POC was a great jumping off point.

DamienMcKenna’s picture

Status: Active » Fixed

I've uploaded all of dsnopek's and evanjenkins' code to the 8.x-1.x branch, lets continue this discussion in separate tickets.

Status: Fixed » Closed (fixed)

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