Enable selection of which panels area to use. [#144018]

Currently the 'left' area is used for the panels block. It would be better to enable users to select which area to use. This is particularly an issue if new layouts are added that don't include left. (All current layouts do include it.)

Comment	File	Size	Author
#3	panelsblock_areas.patch	3.17 KB	recidive

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

Wim Leers

Ghent 🇧🇪🇪🇺

CreditAttribution: Wim Leers commented 17 May 2007 at 14:18

By "users" you mean the "end-users", right? I.e. show multiple tabs, one for each area in the panel.

Comment #2

nedjo

he/him/his

English

CreditAttribution: nedjo commented 17 May 2007 at 14:23

Actually, no, I mean site admins.

I hadn't thought of the idea of having each of the panels be a tab. That would be a separate feature request issue.

Comment #3

recidive CreditAttribution: recidive commented 18 May 2007 at 21:32

Assigned:	nedjo	» recidive
Status:	Active	» Needs review

File	Size
panelsblock_areas.patch	3.17 KB

Here is a patch that does this. It adds a select to the panels form where users can choose what panel area they want to expose as a block. The question I have in mind is how to do with backward compatibility. For now I've changed PANELS_BLOCK_AREA to PANELS_BLOCK_DEFAULT_AREA, so if a exposed block doesn't have an area set, it will use the default.

Comment #4

nedjo

he/him/his

English

CreditAttribution: nedjo commented 18 May 2007 at 21:46

Status:

Needs review

» Reviewed & tested by the community

Nice, this looks great!

A small suggestion before you apply this. We should use the default area as the default if it exists. Here's an untested idea:


    $options = panelsblock_get_areas($form['layout']['#value']);

    $form['panelsblock']['expose_area'] = array(
      '#type' => 'select',
      '#title' => t('Display area'),
      '#default_value' => $blocks[$form['did']['#value']] && $blocks[$form['did']['#value']]['area'] ? $blocks[$form['did']['#value']]['area'] : (array_key_exists(PANELSBLOCK_DEFAULT_AREA, $options) ? PANELSBLOCK_DEFAULT_AREA : ''),
      '#options' => $options,
      '#description' => t('Select the area that will be displayed.'),
    );

Then we don't need to justify the inclusion of this constant--it has a purpose beyond the update. Otherwise, we'd need an update that set the area for each existing registered panelsblock.