Skip to content
This repository was archived by the owner on Jun 8, 2023. It is now read-only.

add before and after menu language hooks#85

Open
nicola-posa wants to merge 1 commit into
italia:masterfrom
nicola-posa:menu-language-hooks
Open

add before and after menu language hooks#85
nicola-posa wants to merge 1 commit into
italia:masterfrom
nicola-posa:menu-language-hooks

Conversation

@nicola-posa
Copy link
Copy Markdown

@nicola-posa nicola-posa commented Jun 8, 2022

Ciao a tutti!

per alcuni sviluppi avrei necessità di aggiungere contenuto all'interno dell'header principale, più precisamente all'interno di <div class="it-header-slim-wrapper-content"> (tra "Regione di Nespoli" e i selettori delle lingue, nell'immagine sotto)

image

ho pensato che potesse essere un utile feature il posizionamento di hook in punti strategici del template, in modo da poter aggiungere contenuto senza fare troppi override nel tema child

in questo modo si renderebbe meno problematico l'aggiornamento del tema.

per iniziare aggiungerei queste due:

<div class="it-header-slim-wrapper-content">
   <!-- <a class="d-none d-lg-block navbar-brand" href="#"> -->
      <img class="header-slim-img" alt="" src="<?php header_image(); ?>">
   <!-- </a> -->
   
+   <?php do_action('design_italia_before_menu_language'); ?>
   
   <?php if ( has_nav_menu( 'menu-language' ) ) { 
      echo '<div class="header-slim-right-zone">';
      echo '<label for="show-menu-lingua" class="show-menu-lingua">&#8942;</label><input type="checkbox" id="show-menu-lingua" role="button">';
      wp_nav_menu(array( 'theme_location' => 'menu-language', 'container' => 'ul', 'menu_class' => 'nav float-right' ));
      echo '</div>';
   } ?>
   
+   <?php do_action('design_italia_after_menu_language'); ?>
   
 </div>

ma se la cosa piace, posso proseguire inserendo altri hook nei punti più comodi

cosa ne pensate?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant