Skip to main content

supprimer class hikashop_module

More
14 years 6 months ago #42085 by erickb
supprimer class hikashop_module was created by erickb
Bonjour

dans ./modules/mod_hikashop/mod_hikashop.php
il y a
Code:
<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.

More
14 years 6 months ago #42089 by nicolas
Replied by nicolas on topic Re: supprimer class hikashop_module
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.

More
14 years 6 months ago - 14 years 6 months ago #42092 by erickb
Replied by erickb on topic Re: supprimer class hikashop_module
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: 14 years 6 months ago by erickb.

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

More
14 years 6 months ago #42293 by nicolas
Replied by nicolas on topic Re: supprimer class hikashop_module
Pourquoi pas.
Comme ceci donc:
<div class="hikashop_module <?php echo @$module_options; ?>">

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

More
14 years 6 months ago - 14 years 6 months ago #42301 by erickb
Replied by erickb on topic Re: supprimer class hikashop_module
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
Code:
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: 14 years 6 months ago by erickb.

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

More
14 years 6 months ago #42314 by nicolas
Replied by nicolas on topic Re: supprimer class hikashop_module
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.

More
14 years 6 months ago #42316 by erickb
Replied by erickb on topic Re: supprimer class hikashop_module
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.

More
14 years 6 months ago #42317 by nicolas
Replied by nicolas on topic Re: supprimer class hikashop_module
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.

More
14 years 6 months ago #42318 by erickb
Replied by erickb on topic Re: supprimer class hikashop_module
ok

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

More
14 years 6 months ago #42492 by erickb
Replied by erickb on topic Re: supprimer class hikashop_module
ou une solution du genre
Code:
<? $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.

More
14 years 6 months ago #42658 by nicolas
Replied by nicolas on topic Re: supprimer class hikashop_module
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.

More
14 years 6 months ago #42662 by erickb
Replied by erickb on topic Re: supprimer class hikashop_module
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.273 seconds
Powered by Kunena Forum