Problème responsive / sélection point relais

Hello,
The text in the drop-down list can be modified with the filter: apply_filters (“cdi_filterhtml_retrait_selectoptions”, $insertselect, $listPointRetraitAcheminate); Which makes it possible to shorten it according to the capacities of the screens and / or the theme.
But my question is about who should do it: CDI who does not know the theme used, or a specific CDI adaptation for the theme and targeted screens ?. Let’s say that it remains in debate, but at the moment I would rather choose the 2nd solution.
But I’m ready to contribute if you need it.

Bonjour,
Le texte de la liste déroulante est modifiable avec le filtre : apply_filters(“cdi_filterhtml_retrait_selectoptions”, $insertselect, $listePointRetraitAcheminement) ; ce qui permet de le racourcir selon les capacités des écrans et/ou du thème.
Mais mon interrogation porte sur qui doit le faire : CDI qui ne maitrise pas le thème utilisé, ou une adaptation CDI spécifique pour le thème et les écrans ciblés ?. Disons que ca reste en débat, mais à l’instant je pencherais plutot pour la 2ème solution.
Mais je suis pret à vous apporter ma contribution si vous en avez besoin.

• This reply was modified 9 years, 1 month ago by Halyra.

Hello,
The filter is to be placed in the function.php file of your theme, or in one of your private plugins.
The next version of CDI will have a model in its examples file.

Bonjour,
Le filtre est a placer dans le fichier function.php de votre thème ou dans un de vos plugins privés.
La prochaine version de CDI aura un modèle dans son fichier d’exemples.

//
// *** Example to have a more responsive pickup location drop-down list
//
// Wp_is_mobile() is a simplified wordpress function that considers tablets to be mobile.
// For a finer setting, we advise you to use the many functions of Mobble plugin (https://wordpress.org/plugins/mobble/)
//

  add_filter( "cdi_filterhtml_retrait_selectoptions","example_cdi_filterhtml_retrait_selectoptions", 10, 2) ; 
  function example_cdi_filterhtml_retrait_selectoptions ($insertselect, $listePointRetraitAcheminement) {
    if (function_exists(is_mobile)) { // Mobble plugin seems active
      $ismobile = is_mobile() ;
    }else{ // Mobble plugin not active, so take the wp function
      $ismobile = wp_is_mobile() ;
    }
    if ($ismobile){ // Mobile
      $newhtml = "<select id="pickupselect" name="pickupselect">" . "<option value="">Choisir votre point de retrait</option>" ;
      foreach ($listePointRetraitAcheminement as $PointRetrait) {
        $newhtml .= "<option value=" . $PointRetrait->identifiant . ">" . $PointRetrait->nom . " à " . $PointRetrait->distanceEnMetre . "m" . "</option>" ;
      }
      $newhtml .= "</select>" ;
    }else{
      $newhtml = $insertselect ; // No change for Desktop
    }
    return $newhtml; 
  }

Je suis désolé, mais le code n’a pas passé intact le forum (mélange de ‘ et de “).
Il faut :
1) changer tous les ” par ‘ .
2) remplacer la ligne au milieu par :
$newhtml = ‘<select id=”pickupselect” name=”pickupselect”>’ . ‘<option value=””>Choisir votre point de retrait</option>’ ;

Sinon, attendre la livraison.