Editable fields inherit form layout [#1649498]

I have noticed that when you have editable fields displayed on the full view of your node, and you are using Display Suite for it, the editable fields inherit the layout from Renderable Elements since they are inside a form.

Right now there is not way (at least I didn't find it) to specify a layout for the node form and another layout for the editable field. Also the link "Manage form display" is displayed all the time on the editable field and when you click it you get nothing in the popup window.

Maybe if setting a different layout for editable fields is too complicated you at least could stop this layout propagation.

Thanks a lot.

Comment	File	Size	Author
#7	1649498-7.patch	889 bytes	swentel

Comments

Comment #1

johaziel

French

Bretagne

commented 9 October 2012 at 10:40

Project:	Renderable elements	» Display Suite
Version:	7.x-1.0-alpha1	» 7.x-2.0

I get same bug but without using Renderable elements

I found that is ds_forms_form_alter who does this bug

so i changed the function like this, and that works for me but may be there is a better way to do that

function ds_forms_form_alter(&$form, &$form_state, $form_id) {
  
  if ($ds_form = ds_build_load($form, $form_id)) {
    if (!preg_match('/^editablefields_form_/', $form_id)) {
      if ($layout = ds_get_layout($ds_form->entity_type, $ds_form->bundle, 'form', FALSE)) {
        // Add the theming function and add the layout as a class.
        $form['#theme'] = array('ds_forms_custom_form');
        $class = strtr($layout['layout'], '_', '-');
        if ((isset($form['#attributes']['class']) && is_array($form['#attributes']['class'])) || !(isset($form['#attributes']['class']))) {
          $form['#attributes']['class'][] = $class;
        }
        elseif (isset($form['#attributes']['class']) && is_string($form['#attributes']['class'])) {
          $form['#attributes']['class'] .= ' ' . $class . ' ';
        }
      }
    }
  }
}