Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

#3950 [Risk] add: move medias inside medias bloc #3957

Open
wants to merge 8 commits into
base: develop
Choose a base branch
from

Conversation

evarisk-francois
Copy link
Contributor

No description provided.

Copy link
Member

@evarisk-micka evarisk-micka left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

pense à faire la même correction dans toute les occurrences des modales d'évaluation
(comme riskassessment_view_edit_modal)
C'est toujours important de vérifier les occurrences du code, comme des fois on fait du spécifique il peut y avoir des duplicatas ailleurs : ici j'ai regardé où était appelé DIGIRISKDOLIBARR_ADVANCED_RISKASSESSMENT_METHOD pour voir si il y avait d'autres fichiers

<div class="risk-evaluation-media-wrapper">
<div class="risk-evaluation-header">
<span class="section-title"><?php echo ' ' . $langs->trans('RiskAssessment'); ?></span>
<?php if ($conf->global->DIGIRISKDOLIBARR_ADVANCED_RISKASSESSMENT_METHOD) : ?>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Quitte à modifier un peu le code, pense à remplacer le $conf->global->DIGIRISKDOLIBARR_ADVANCED_RISKASSESSMENT_METHOD
Par getDolGlobalInt('DIGIRISKDOLIBARR_ADVANCED_RISKASSESSMENT_METHOD') > 0

@evarisk-micka
Copy link
Member

image

@evarisk-micka
Copy link
Member

Profites en dans la vue edit pour ajuster la hauteur de la modal qui dépasse sur la barre de menu en haut
Et aussi demander à Amandine des avis sur le positionnement des blocs (mettre le commentaire sous l'évaluation et le bloc média ?)

image

@evarisk-francois
Copy link
Contributor Author

@eoxia-amandine faire aussi le css pour l'ajout d'évaluation de risque (digiriskdolibarr_riskassessment_view.tpl.php) et l'édition d'évaluation de risque (digiriskdolibarr_riskassessment_view_edit_modal.tpl.php)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1 Amélioration Minor feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Amélioration] - IHM - mettre le bloc photo avec l'évaluation
3 participants