I have some blocks set with the core block layout as well as a context that sets some other block with "Include blocks from Block layout" checked.

when viewing a page with this context, the correct blocks are displayed but blocks that are weighted from the core block form are not in the correct order.

thanks.

CommentFileSizeAuthor
#6 interdiff_5-6.txt1.34 KBdeaom
#6 context-3156610-6.patch1.04 KBdeaom
#5 context-Core_block_weight_not_respected_when_using_a_context_block_reaction-3156610-d8-5.patch1.04 KBpaulocs

Comments

loze created an issue. See original summary.

himanshu_sindhwani’s picture

himanshu_sindhwani commented
Assigned: Unassigned » himanshu_sindhwani

I will have a look on this issue.

loze’s picture

loze commented

I have a little more info.
What appears to be happening for me is the Main content bock (set with core) always has a weight of 0, regardless of the weight given to it from the core UI, but only when the context is active.

For example in the core ui content region I have:
- Block A
- Main Content Block
- Block B
- Block C

In my context I have a block reaction, and block placed in other regions (not content region) The content region is empty in my context.
But when viewing the page the content region displays as

- Block A
- Block B
- Block C
- Main Content Block

When I disable the context it displays correctly.
I can get around this if in the core UI I force the weight of blocks B and C to be more than 0.

Hope that helps. Thanks.

himanshu_sindhwani’s picture

himanshu_sindhwani commented
Assigned: himanshu_sindhwani » Unassigned

Got busy in some other work. Will have a look at it sometimes after. Unassigning myself until then.

paulocs’s picture

paulocs
Primary language Portuguese, Brazil
Location Belo Horizonte
commented
Status: Active » Needs review
StatusFileSize
new context-Core_block_weight_not_respected_when_using_a_context_block_reaction-3156610-d8-5.patch1.04 KB

Hello @all,

Here is a patch. Please review it.

deaom’s picture

deaom commented
StatusFileSize
new context-3156610-6.patch1.04 KB
new interdiff_5-6.txt1.34 KB

Hi, I could reproduce the issue and once the patch was applied it corrected the problem with the main system order. I just corrected coding standards, so new patch added.

  • paulocs committed cf88744 on 8.x-4.x 
    Issue #3156610 by DeaOm, paulocs, loze: Core block weight not respected...
paulocs’s picture

paulocs
Primary language Portuguese, Brazil
Location Belo Horizonte
commented
Status: Needs review » Fixed

Fixed in dev branch.

Thanks @all!

neslee canil pinto’s picture

neslee canil pinto
he/him
Primary language English
commented

@paulocs, thanks. Much needed one👍🏻

Status: Fixed » Closed (fixed)

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