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 7 years, 9 months ago by Halyra.

Bonjour,

pouvez-vous me dire où ce filtre doit être ajouté ?

Merci à vous !

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://www.ads-software.com/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; 
  }

Bonjour,

J’ai rajouté ce script dans le functions.php du thème child et malheureusement ?a fait planter le site avec une erreur 500. Dans les logs, j’ai cette erreur => “PHP message: PHP Parse error: syntax error, unexpected ‘;’, expecting ‘(‘ in /home/preprodfunnybeeparis/public_html/wp-content/themes/bridge-child/functions.php on line 104” while reading response header from upstream, client: 109.9.187.4, server: preprod.funnybeeparis.com, request: “GET / HTTP/1.1”, upstream: “fastcgi://unix:/var/run/preprodfunnybeeparis_fpm.sock:”, host: “preprod.funnybeeparis.com”

La ligne 104 correspond à cette ligne de code => add_filter( "cdi_filterhtml_retrait_selectoptions","example_cdi_filterhtml_retrait_selectoptions", 10, 2) ;

Malheureusement, après ?a dépasse mes compétences pour corriger l’erreur… ??

Merci,
Mathias

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.

Hello,
In the new 1.18.2 version, you will find an example in examples/CDI-responsivedropdown-example.php . You can test it by doing an include at the bottom of the colissimo-delivery-integration.php file.

Bonjour,
Dans la nouvelle version 1.18.2?, vous trouverez un exemple dans examples/CDI-selectfromgooglemap-example.php . Vous pouvez le tester en faisant un include dans le bas du fichier colissimo-delivery-integration.php .