I'd love to see more HTML5 elements out-of-the-box in this module, mostly I'm missing main, article, section, footer and aside. I know I can install fences to get these elements and I'm not sure if there is a scenario where you would want semantic panels but not want fences? Therefore I'm raising this question here :-)

      'main' => 'MAIN',
      'nav' => 'NAV',
      'article' => 'ARTICLE',
      'section' => 'SECTION',
      'aside' => 'ASIDE',
      'footer' => 'FOOTER',
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mathiasericsson’s picture

mathiasericsson’s picture

dajjen’s picture

Patch file seems to be empty.

mathiasericsson’s picture

dajjen’s picture

Status: Active » Needs review
+++ b/semantic_panels.module
@@ -64,6 +64,12 @@ function _semantic_panels_get_elements() {
+        'main' => 'MAIN',
+        'nav' => 'NAV',
+        'article' => 'ARTICLE',
+        'section' => 'SECTION',
+        'aside' => 'ASIDE',
+        'footer' => 'FOOTER',

Now the code seems to be in the patch file

skorzh’s picture

Status: Needs review » Reviewed & tested by the community

Very good idea, the patch works for me