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',
CommentFileSizeAuthor
#4 semantic_panels-html5_tags-2161471-4.patch487 bytesmathiasericsson
#1 semantic_panels-html5_tags-2161471-1.patch0 bytesmathiasericsson

Comments

mathiasericsson’s picture

mathiasericsson commented
StatusFileSize
new semantic_panels-html5_tags-2161471-1.patch0 bytes
mathiasericsson’s picture

mathiasericsson commented
dajjen’s picture

dajjen
Location Östersund (Sweden)
commented

Patch file seems to be empty.

mathiasericsson’s picture

mathiasericsson commented
StatusFileSize
new semantic_panels-html5_tags-2161471-4.patch487 bytes
dajjen’s picture

dajjen
Location Östersund (Sweden)
commented
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

skdrupal88’s picture

skdrupal88
Primary language Russian
Location Belarus
commented
Status: Needs review » Reviewed & tested by the community

Very good idea, the patch works for me