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

Merged
fpeters merged 22 commits from wip/75382-form-field-on-error into main 1 month ago
tjund commented 3 months ago
Owner
There is no content yet.
tjund added 1 commit 3 months ago
gitea/publik-base-theme/pipeline/head This commit looks good Details
6e89fe920e
css: improve styles of form field on error (#75382)
tjund force-pushed wip/75382-form-field-on-error from 6e89fe920e to e2c4f84cd0 3 months ago
tjund added 2 commits 2 months ago
gitea/publik-base-theme/pipeline/head This commit looks good Details
8fb171d904
remove unused $light error styles
tjund added 1 commit 2 months ago
gitea/publik-base-theme/pipeline/head This commit looks good Details
350226de43
classic style
tjund force-pushed wip/75382-form-field-on-error from 350226de43 to 0c42632237 2 months ago
tjund force-pushed wip/75382-form-field-on-error from 0c42632237 to d8ef6804f6 2 months ago
tjund force-pushed wip/75382-form-field-on-error from d8ef6804f6 to 0dbc13cacb 2 months ago
tjund changed title from WIP: css: improve styles of form field on error (#75382) to css: improve styles of form field on error (#75382) 2 months ago
tjund commented 2 months ago
Poster
Owner
  • 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 2 months ago
Owner

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 2 months ago
fpeters reviewed 2 months ago
@ -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;
Owner

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 2 months ago
fpeters reviewed 2 months ago
@ -177,3 +177,3 @@
div.infonotice::before, div.warningnotice::before, div.errornotice::before {
content: url(/static/dialog-warning.png);
content: url(dialog-warning.png);
Owner

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é.
Owner

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.

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 2 months ago
fpeters approved these changes 1 month ago
fpeters left a comment
Owner

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 1 month ago
fpeters deleted branch wip/75382-form-field-on-error 1 month ago

Reviewers

csechet was requested for review 2 months ago
fpeters approved these changes 1 month ago
gitea/publik-base-theme/pipeline/head This commit looks good
The pull request has been merged as 8f1d4e52ce.
You can also view command line instructions.

Step 1:

From your project repository, check out a new branch and test the changes.
git checkout -b wip/75382-form-field-on-error main
git pull origin wip/75382-form-field-on-error

Step 2:

Merge the changes and update on Gitea.
git checkout main
git merge --no-ff wip/75382-form-field-on-error
git push origin main
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#202
Loading…
There is no content yet.