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

Amélioration de l'impression depuis un navigateur #812

Merged
merged 6 commits into from
Nov 24, 2023

Conversation

Agath21
Copy link
Collaborator

@Agath21 Agath21 commented Oct 30, 2023

Cette contribution propose les modifications suivantes :

Ne parlant pas allemand, ni slovaque la traduction est basée sur un traducteur, n'hésitez donc pas à me proposer une amélioration en cas d'erreur :)

Tests à réaliser :

  • Activation du paramètre mapprint = affichage du bouton dans l'interface
  • Ouverture de la fenêtre d'impression lors du clic
  • Affichage de la carte et de la légende
  • Affichage du #right-panel en cas d'interrogation d'une entité
  • Affichage du #bottom-panel en cas d'interrogation d'une entité

@Agath21 Agath21 added this to To test in Module d'impression Oct 30, 2023
@Gaetanbrl
Copy link
Collaborator

Gaetanbrl commented Nov 6, 2023

Testés :

  • Affichage indépendant right panel
  • Affichage indépendant bottom panel
  • Affichage simultané du right et bottom panel
  • Absence du paramètre mapprint
  • Valeur false du paramètre mapprint
  • Valeur true du paramètre mapprint
  • Affichage de la fenêtre d'impression via les fonctionnalités natives navigateur
  • Prise en compte des paramètres natifs de la fenêtre du navigateur
  • Lecture du code

Ca me semble bon.

L'impression est plus exploitable via le navigateur avec cette PR.
Merci @Agath21 👍 !

image

Copy link
Collaborator

@Gaetanbrl Gaetanbrl left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@lecault lecault left a comment

Choose a reason for hiding this comment

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

Hello, merci pour le dev.

Quelques remarques mineures ne nécessitant pas forcément de modifications :

  • En lien avec la remarque sur CORS dans l'export PNG (cf doc) j'ai cherché un cas où ça ne fonctionne pas, j'ai pas trouvé.
  • La partie carte ne pourrait t'elle pas prend plus de place en bas ? Je trouve qu'il y a un trop gros espace.
  • Le pied de page "Propulsé par mviewer (https://mviewer.netlify.app/fr/) 3.10-snapshot" me semble trop imposant.
  • En impression mobile, le résultat est moins bon

@Gaetanbrl
Copy link
Collaborator

Quelques remarques mineures ne nécessitant pas forcément de modifications :

Merci pour ces retour. Est-ce que c'est des éléments qui peuvent être améliorés @Agath21 ?

En impression mobile, le résultat est moins bon

Pas testé, je n'utilise jamais utiliser mviewer avec mon mobile.

@lecault quand même c'est mieux qu'avant ? :)

@lecault
Copy link
Contributor

lecault commented Nov 10, 2023

Ah oui je valide même si mes remarques ne sont pas prises en compte. C'est du facultatif :).
Le seul qui nécessite enquête pour moi est l'espace blanc entre la carte et la source.
Pour le mobile je pense que ce n'est pas utilisé.

@Gaetanbrl
Copy link
Collaborator

Le seul qui nécessite enquête pour moi est l'espace blanc entre la carte et la source.

Est-ce que tu peux me mettre sur une capture la zone que tu voudrais qu'on réduise stp ? (avec un encadré rouge)

En l'état je ne vois pas trop (sinon je verrai avec Agathe)

@lecault
Copy link
Contributor

lecault commented Nov 20, 2023

Ici :

image

@Gaetanbrl
Copy link
Collaborator

A voir avec @Agath21 mais en regardant le code j'ai l'impression que ce blanc est obligatoire pour conserver les ratio de la carte.
sinon la carte sera déformée je pense.

@Agath21
Copy link
Collaborator Author

Agath21 commented Nov 21, 2023

Ici :

image

L'affichage varie selon les navigateurs, je ne reproduis pas ce problème sous Mozilla. Pour supprimer la bande blanche dans chrome, j'ai ajouté une propriété supplémentaire de zoom sur la carte et donne ce rendu :
image

Si cela te convient @lecault, on peut merger après test ;)

@Agath21 Agath21 closed this Nov 21, 2023
@Agath21 Agath21 reopened this Nov 21, 2023
@lecault
Copy link
Contributor

lecault commented Nov 21, 2023

Merci Agathe.

Je viens de tester on a effet de bord, le cadrage de la carte n'est pas bon.

Exemple sur le default sans changer la vue :

image

@Gaetanbrl
Copy link
Collaborator

Gaetanbrl commented Nov 21, 2023

Je viens de tester on a effet de bord, le cadrage de la carte n'est pas bon.

Je pense que le cadrage de la carte ne correspond pas au format (e.g A4). Donc quand on ouvre la boîte de dialogue d'impression, le navigateur (qui est l'outil utilisé) va faire s'arranger du cadrage qui correspond au A4.

En reprenant la carte par défaut, si on change la div #map en forçant le A4 (297mm x 210mm) on a bien le même cadrage que la boîte de dialogue du navigateur :

image

image

Si on souhaite avoir un cadrage "carré" avec les formats de la boîte de dialogue, alors il faut utiliser le plugin "print" ou bien prévoir de modifier la taille de la map (via une UI supplémentaire) pour avoir un format identique au format d'impression. Sinon on aura toujours un décalage entre la carte (selon les écrans qui plus est) et la boite de dialogue qui réduit la div de la map au format choisi (e.g A4).

@Agath21
Copy link
Collaborator Author

Agath21 commented Nov 21, 2023

Du coup j'ai supprimé la propriété, je laisse en l'état car je ne souhaite pas surchargé le css avec des propriétés supplémentaires ou du javascript pour refresh la carte (comme le plugin). L'utilisateur pourra régler l'impression avec les paramètres du navigateur (format, marges, mise à l'échelle et divers options).

@Gaetanbrl
Copy link
Collaborator

@lecault je viens de vérifier et le rendu avant cette PR intégré une bande blanche et un cadrage aussi erroné (et d'autres élément non désirables).

image

Je propose de merger cette PR (qui améliore le rendu dans la boite d'impression dans tous les cas...). En sachant qu'on a le plugin d'impression qui donne de bien meilleur résultat.

@Gaetanbrl Gaetanbrl self-assigned this Nov 22, 2023
@Gaetanbrl
Copy link
Collaborator

@Agath21 je m'affecte la PR pour corriger le conflit avec le index.html que j'ai du apporté via ma dernière PR avec OL8 (j'ai du passer du formatage dans le html...).

Copy link
Collaborator

@Gaetanbrl Gaetanbrl left a comment

Choose a reason for hiding this comment

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

pour moi-même - corriger le conflit index.html

Copy link
Collaborator

@Gaetanbrl Gaetanbrl left a comment

Choose a reason for hiding this comment

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

Conflit géré.

@Gaetanbrl Gaetanbrl merged commit 4e4d4a0 into mviewer:develop Nov 24, 2023
1 check passed
@lecault lecault added this to the 3.10 milestone Nov 30, 2023
@lecault lecault mentioned this pull request Dec 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging this pull request may close these issues.

None yet

3 participants