backdrop-contrib / mini_layouts

Provide blocks which allow positioning content within them in layouts.
GNU General Public License v2.0
7 stars 4 forks source link

When adding a mini-layout to a regular layout should not see option to switch layout #5

Closed herbdool closed 2 years ago

herbdool commented 2 years ago

I created a mini-layout then went to my default main template to add the mini-layout block. I then see:

Screenshot from 2021-10-26 14-45-44

I don't think users should be allowed to switch the layout here since it could mean blocks would disappear.

docwilmot commented 2 years ago

Hmm, also considered this, but thought someone might want it this way. But this probably doesnt make much sense in fact. Will leave for a while and see if anyone else chimes in either way.

philsward commented 2 years ago

I'll agree that this should not be shown. The layout template should be controlled by the mini-layout. I'm fine with leaving it in for now to gather feedback, but I don't think it's necessary and creates a bit of UX confusion.

docwilmot commented 2 years ago

Removed