supprimer class hikashop_module

  • Posts: 2639
  • Thank you received: 66
  • Hikashop Business
12 years 8 months ago #42085

Bonjour

dans ./modules/mod_hikashop/mod_hikashop.php
il y a

<div class="hikashop_module">
<?php echo $html; ?>
</div>

je voudrais virer ce div qui bloque toute possibilité de CSS (float ou autre) mais le problème c'est qu'il est en dur dans le code

il y a un moyen ?
- d'override
- de le supprimer dans une future version ?
- de pouvoir nommer cette class pour la differencier selon mes module

merci

Please Log in or Create an account to join the conversation.

  • Posts: 82863
  • Thank you received: 13372
  • MODERATOR
12 years 8 months ago #42089

pas d'override non.
Et non, nous n'allons pas la supprimer car elle est très utile.
Nous pouvons par contre ajouter une seconde classe au div qui serait spécifique à chaque module comme ceci:

<div class="hikashop_module hikashop_module_<?php echo $module->id; ?>">
<?php echo $html; ?>
</div>

Ce DIV ne bloque en rien le CSS. Il suffit d'adapter le CSS.

Please Log in or Create an account to join the conversation.

  • Posts: 2639
  • Thank you received: 66
  • Hikashop Business
12 years 8 months ago #42092

ce div bloque tout le CSS s'il existe ou s'il est in-identifiable si voulez un float left dans certain cas et d'autres non c'est impossible, actuellement je l'ai supprimer car il ne fait que rajouter un div inutile

<div class="hikashop_module hikashop_module_<?php echo $module->id; ?>"> est une mauvaise solution car on doit courir partout chercher l'id et un id n'est pas signifiant il vaut mieux un suffixe de class du module

Last edit: 12 years 8 months ago by erickb.

Please Log in or Create an account to join the conversation.

  • Posts: 82863
  • Thank you received: 13372
  • MODERATOR
12 years 8 months ago #42293

Pourquoi pas.
Comme ceci donc:
<div class="hikashop_module <?php echo @$module_options; ?>">

Please Log in or Create an account to join the conversation.

  • Posts: 2639
  • Thank you received: 66
  • Hikashop Business
12 years 8 months ago #42301

oui le coup c'est de pouvoir le différencier

je préfère la solution que j'utilise sur ./templates/MONTPL/html/mod_custom/default.php

if($moduleclass_sfx){
	echo '<div class="'.$moduleclass_sfx .'">' . $module->content .  '</div>';
}
else{	
	echo $module->content;
}

comme ça tout le monde est content, rien n'empêche de mettre hikashop_module par défaut dans les paramètres du module qu'on peut supprimer ou modifier

Last edit: 12 years 8 months ago by erickb.

Please Log in or Create an account to join the conversation.

  • Posts: 82863
  • Thank you received: 13372
  • MODERATOR
12 years 8 months ago #42314

Je vois. Le problème c'est que cela casse la rétro compatibilité. Pour les personnes utilisant hikashop_module dans leur CSS, il faudra modifier tous les modules hikashop qu'ils ont si on fait cette modification.
Donc je préfère rester sur la solution du message précédent.

Please Log in or Create an account to join the conversation.

  • Posts: 2639
  • Thank you received: 66
  • Hikashop Business
12 years 8 months ago #42316

si vous mettez hikashop_module en defaut dans la class non , il y a que ceux qui la supprime qui la perdent

Please Log in or Create an account to join the conversation.

  • Posts: 82863
  • Thank you received: 13372
  • MODERATOR
12 years 8 months ago #42317

La valeur par défaut du xml des modules est uniquement pour les nouveaux modules. Pour les modules existant c'est la valeur dans la base de données qui est utilisée.
Donc non, cela ne fonctionnera pas.

Please Log in or Create an account to join the conversation.

  • Posts: 2639
  • Thank you received: 66
  • Hikashop Business
12 years 8 months ago #42318

ok

Please Log in or Create an account to join the conversation.

  • Posts: 2639
  • Thank you received: 66
  • Hikashop Business
12 years 8 months ago #42492

ou une solution du genre

<?
$sfx = '';
f($module_options['moduleclass_sfx']){
	$sfx = '_'. $module_options['moduleclass_sfx'] ;
}
?>
<div class="hikashop_module <?= $sfx ; ?>">

Please Log in or Create an account to join the conversation.

  • Posts: 82863
  • Thank you received: 13372
  • MODERATOR
12 years 8 months ago #42658

Oui mais pourquoi le _ si de toute facon il y a un espace avant ? Et meme pourquoi rajouter un _ tout court ?

Please Log in or Create an account to join the conversation.

  • Posts: 2639
  • Thank you received: 66
  • Hikashop Business
12 years 8 months ago #42662

oui bien sur on peu mettre 2 class et donc un espace

Please Log in or Create an account to join the conversation.

Time to create page: 0.088 seconds
Powered by Kunena Forum