entrouvert
/
publik-base-theme
15
1
Fork 1
Code Issues Pull Requests 10 Releases Wiki Activity

css: improve styles of form field on error (#75382) #202

Merged
fpeters merged 22 commits from wip/75382-form-field-on-error into main 2023-04-20 18:36:46 +02:00
Conversation 3 Commits 22 Files Changed 34 +183 -160
tjund commented 2023-03-16 14:18:25 +01:00
Owner
Copy Link
No description provided.
tjund force-pushed wip/75382-form-field-on-error from 6e89fe920e to e2c4f84cd0 2023-03-16 14:26:21 +01:00 Compare
tjund force-pushed wip/75382-form-field-on-error from 350226de43 to 0c42632237 2023-04-06 12:04:50 +02:00 Compare
tjund force-pushed wip/75382-form-field-on-error from 0c42632237 to d8ef6804f6 2023-04-06 17:46:49 +02:00 Compare
tjund force-pushed wip/75382-form-field-on-error from d8ef6804f6 to 0dbc13cacb 2023-04-11 15:11:49 +02:00 Compare
tjund changed title from WIP: css: improve styles of form field on error (#75382) to css: improve styles of form field on error (#75382) 2023-04-11 15:12:31 +02:00
tjund commented 2023-04-11 15:21:29 +02:00
Author
Owner
Copy Link
  • Wrappé le nouveau style dans une condition @if ($field-on-error-style == classic).
  • Tous les thèmes héritent de ce nouveau style "classic" sauf les thèmes qui avaient déjà définis des styles pour les champs en erreur.
    ** Ces thèmes définissent $field-on-error-style: custom;
    ** 19 thèmes patchés

Cela permet de faire évoluer le style "classic" sans impacter les thèmes avec style "custom"

* Wrappé le nouveau style dans une condition @if ($field-on-error-style == classic). * Tous les thèmes héritent de ce nouveau style "classic" sauf les thèmes qui avaient déjà définis des styles pour les champs en erreur. ** Ces thèmes définissent $field-on-error-style: custom; ** 19 thèmes patchés Cela permet de faire évoluer le style "classic" sans impacter les thèmes avec style "custom"
fpeters force-pushed wip/75382-form-field-on-error from 0dbc13cacb to 8d3aea750d 2023-04-18 14:36:48 +02:00 Compare
fpeters commented 2023-04-18 14:37:41 +02:00
Owner
Copy Link

Branche rebasée avec les commits modifiés pour ne pas introduire de tabulations et espaces en fin de ligne. (Thomas: sans doute une configuration locale perdue).

Branche rebasée avec les commits modifiés pour ne pas introduire de tabulations et espaces en fin de ligne. (Thomas: sans doute une configuration locale perdue).
fpeters force-pushed wip/75382-form-field-on-error from 8d3aea750d to 2b08b22096 2023-04-18 15:19:49 +02:00 Compare
fpeters reviewed 2023-04-18 15:21:23 +02:00
static/includes/_forms.scss Outdated
@ -282,3 +275,3 @@
padding: $widget-padding;
max-width: 100%;
transition: background 0.5s ease, color 0.5s ease, border 0.5s ease;
transition: background 0.5s ease, color 0.5s ease;
fpeters commented 2023-04-18 15:21:23 +02:00
Owner
Copy Link

Je retirerais cette transition (préexistante); comme le style applique un changement de largeur de la bordure gauche ça fait un mouvement lent et saccadé pas terrible.

Je retirerais cette transition (préexistante); comme le style applique un changement de largeur de la bordure gauche ça fait un mouvement lent et saccadé pas terrible.
fpeters force-pushed wip/75382-form-field-on-error from 2b08b22096 to 8f1d4e52ce 2023-04-18 16:34:09 +02:00 Compare
fpeters reviewed 2023-04-18 16:35:20 +02:00
static/villejuif/_custom.scss
@ -177,3 +177,3 @@
div.infonotice::before, div.warningnotice::before, div.errornotice::before {
content: url(/static/dialog-warning.png);
content: url(dialog-warning.png);
fpeters commented 2023-04-18 16:35:20 +02:00
Owner
Copy Link

Comme dialog-warning.png n'est plus utilisé de manière globale je le déplace dans l'intégration villejuif où il est encore référencé.

Comme dialog-warning.png n'est plus utilisé de manière globale je le déplace dans l'intégration villejuif où il est encore référencé.
fpeters commented 2023-04-18 16:48:43 +02:00
Owner
Copy Link

Voilà c'est surtout Thomas j'ai juste fait du rebase mais il y a ces deux commits supplémentaires, #202 (comment), qui font que je ne vais pas valider de moi-même le tout.

Voilà c'est surtout Thomas j'ai juste fait du rebase mais il y a ces deux commits supplémentaires, https://git.entrouvert.org/entrouvert/publik-base-theme/pulls/202#issuecomment-8313, qui font que je ne vais pas valider de moi-même le tout.
fpeters requested review from csechet 2023-04-18 16:49:05 +02:00
fpeters approved these changes 2023-04-20 18:36:36 +02:00
fpeters left a comment
Owner
Copy Link

Voilà c'est surtout Thomas j'ai juste fait du rebase mais il y a ces deux commits supplémentaires, #202, qui font que je ne vais pas valider de moi-même le tout.

Adjugé ils sont minimes.

> Voilà c'est surtout Thomas j'ai juste fait du rebase mais il y a ces deux commits supplémentaires, #202, qui font que je ne vais pas valider de moi-même le tout. Adjugé ils sont minimes.
fpeters merged commit 8f1d4e52ce into main 2023-04-20 18:36:46 +02:00
fpeters deleted branch wip/75382-form-field-on-error 2023-04-20 18:36:46 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
csechet
fpeters
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
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#202
No description provided.
Delete Branch "wip/75382-form-field-on-error"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?