eurelien-cd28-2022: retours v1 (#71798) #61

Merged
csechet merged 11 commits from wip/71798-eurelien-retours-v- into main 2022-12-14 12:13:29 +01:00
Owner
No description provided.
Owner

"favicon": "eurelien-cd28-2022/img/favicon.ico"

  • pas de fichier .ico, mais .png
  • ne rien faire ? = laisser le client gérer l'import et le choix du favicon depuis combo, menu PWA.
> "favicon": "eurelien-cd28-2022/img/favicon.ico" * pas de fichier .ico, mais .png * ne rien faire ? = laisser le client gérer l'import et le choix du favicon depuis combo, menu PWA.
tjund reviewed 2022-12-05 16:53:18 +01:00
@ -6,18 +6,14 @@
{% with title|split:" " as title_elements %}
{% if title_elements|length > 1 %}
<h1>
Owner

Pas efficace de donner un h1 à un élément qui n'apporte aucune information utile.

Pas efficace de donner un h1 à un élément qui n'apporte aucune information utile.
Author
Owner

C'est une demande du client.

C'est une demande du client.
Owner

Misère

Misère
tjund marked this conversation as resolved
tjund reviewed 2022-12-05 16:55:04 +01:00
@ -11,2 +9,4 @@
<span class="cd28-title-header">Les</span>
d&eacute;marches
</h1>
{% else %}
Owner

le else ne sert plus à rien je pense. Sauf afficher le titre de la démarche ici uniquement lorqu'il est composé d'1 seul mot !?

le else ne sert plus à rien je pense. Sauf afficher le titre de la démarche ici uniquement lorqu'il est composé d'1 seul mot !?
Author
Owner

Pas réveillé hier... C'est fixé.

Pas réveillé hier... C'est fixé.
tjund marked this conversation as resolved
tjund reviewed 2022-12-05 16:55:49 +01:00
@ -20,4 +17,3 @@
</div>
{% endblock %}
{% block wcs-form-title %}{% endblock %}
Owner

retaurer le titre par défaut ok, mais il s'affiche tout/trop petit ?

retaurer le titre par défaut ok, mais il s'affiche tout/trop petit ?
Author
Owner

Idem, demande du client.

Idem, demande du client.
tjund marked this conversation as resolved
tjund requested changes 2022-12-05 17:01:44 +01:00
@ -39,2 +43,4 @@
border-radius: 0;
}
div.ui-dialog {
Owner

Je pense pas que ce soit une bonne idée de se lancer en l'état à de la customisation sur class propriétaires jquery-ui.

Je pense pas que ce soit une bonne idée de se lancer en l'état à de la customisation sur class propriétaires jquery-ui.
Author
Owner
J'ai supprimé le commit, c'était suite à https://gitea.entrouvert.org/entrouvert/publik-base-theme/pulls/40#issuecomment-717
tjund marked this conversation as resolved
csechet force-pushed wip/71798-eurelien-retours-v- from f5310f74de to f9c0649030 2022-12-06 11:18:00 +01:00 Compare
Author
Owner

"favicon": "eurelien-cd28-2022/img/favicon.ico"

  • pas de fichier .ico, mais .png
  • ne rien faire ? = laisser le client gérer l'import et le choix du favicon depuis combo, menu PWA.

Je ne savais pas que c'était possible, j'ai supprimé le commit.

> > "favicon": "eurelien-cd28-2022/img/favicon.ico" > > * pas de fichier .ico, mais .png > * ne rien faire ? = laisser le client gérer l'import et le choix du favicon depuis combo, menu PWA. Je ne savais pas que c'était possible, j'ai supprimé le commit.
tjund requested changes 2022-12-08 16:30:46 +01:00
@ -130,3 +122,2 @@
@media($min-desktop-viewport) {
content: "\ea4e"; // arrow-down-s-line
margin-left: $space-small;
content: ""; // arrow-down-s-line
Owner

Pour ne pas générer un pseudo élément, il suffit de mettre

content: none;
Pour ne pas générer un pseudo élément, il suffit de mettre ```CSS content: none; ```
Author
Owner

Fixé, et j'ai ajouté deux commits suite à des demandes dans https://dev.entrouvert.org/issues/72184.

Fixé, et j'ai ajouté deux commits suite à des demandes dans https://dev.entrouvert.org/issues/72184.
Owner

code OK.
Quelques retouches niveau commits:

  • stasher f65633f50c avec 35d7df84ad
  • eurelien-cd28-2022: fix big titles icon vertical alignment(#71798) : ajouter espace avant (#71798)
  • eurelien-cd28-2022: do not inherit parent page's page header (71798) : manque #
code OK. Quelques retouches niveau commits: * stasher f65633f50c avec 35d7df84ad * eurelien-cd28-2022: fix big titles icon vertical alignment(#71798) : ajouter espace avant (#71798) * eurelien-cd28-2022: do not inherit parent page's page header (71798) : manque #
csechet force-pushed wip/71798-eurelien-retours-v- from 5510129f6c to c954d797a5 2022-12-14 10:51:04 +01:00 Compare
tjund approved these changes 2022-12-14 11:24:05 +01:00
csechet merged commit 618baf5f1e into main 2022-12-14 12:13:29 +01:00
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/publik-base-theme#61
No description provided.