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

toulouse-2022: remove background gradient on tables (#71297) #14

Closed
nroche wants to merge 1 commits from wip/71297-toulouse-2022-remove-gradient-on-tables into main
pull from: wip/71297-toulouse-2022-remove-gradient-on-tables
merge into: entrouvert:main
Conversation 7 Commits 1 Files Changed 1 +2
nroche commented 2022-11-14 18:09:03 +01:00
Owner
Copy Link

Parce que pour Toulouse l'ombre noir sur les lignes zébrés ne semble pas appropriée.

Parce que pour Toulouse l'ombre noir sur les lignes zébrés ne semble pas appropriée.
nroche commented 2022-11-14 18:11:26 +01:00
Author
Owner
Copy Link

Voir le ticket client pour le rendu initial : #71142
(ci-joint le rendu avec le patch)

Voir le ticket client pour le rendu initial : #71142 (ci-joint le rendu avec le patch)
Screenshot_2022-11-14 Titre du site (option de l'inté graphique) - test factures.png
38 KiB
Screenshot_2022-11-14 Titre du site (option de l'inté graphique) - test factures.png
tjund commented 2022-11-15 08:30:45 +01:00
Owner
Copy Link

L'ombre ne devrait pas s'afficher en desktop.
Uniquement en mobile et du côté scollable.
C'est un standard de fait pour indiquer un élément scollable aux usagers mobile.

Avant de supprimer l'ombre je regarderais pourquoi l'ombre s'affiche sur desktop et je proposerais une couleur non opaque pour les zebras.

L'ombre ne devrait pas s'afficher en desktop. Uniquement en mobile et du côté scollable. C'est un standard de fait pour indiquer un élément scollable aux usagers mobile. Avant de supprimer l'ombre je regarderais pourquoi l'ombre s'affiche sur desktop et je proposerais une couleur non opaque pour les zebras.
tjund requested review from tjund 2022-11-15 08:31:02 +01:00
tjund commented 2022-11-15 09:15:50 +01:00
Owner
Copy Link

le table-wrapper a besoin d'un background pour masquer les ombres lorsqu'elles ne sont pas necessaires.

Une variable scss est dispo pour cela. Ajoute la dans les variables du thème :

$table-wrapper-background: white;

le table-wrapper a besoin d'un background pour masquer les ombres lorsqu'elles ne sont pas necessaires. Une variable scss est dispo pour cela. Ajoute la dans les variables du thème : `$table-wrapper-background: white;`
nroche force-pushed wip/71297-toulouse-2022-remove-gradient-on-tables from b6f0c71e9e to 5b323c2dd8 2022-11-15 11:34:50 +01:00 Compare
tjund commented 2022-11-15 11:58:47 +01:00
Owner
Copy Link

Il reste le soucis des backgrounds opaques des zebras.
@nroche Quel est la cellule utilisé dans l'exemple #71142 ?
Il y a une url consultable ?

Il reste le soucis des backgrounds opaques des zebras. @nroche Quel est la cellule utilisé dans l'exemple #71142 ? Il y a une url consultable ?
nroche commented 2022-11-15 18:21:02 +01:00
Author
Owner
Copy Link

Il reste le soucis des backgrounds opaques des zebras.
Oui j'ai oublié de t'en parler, en fait je ne comprend pas le props (pour moi on a le même rendu sur clappoti-les-canards).

Il y a une url consultable ?
Non, je demande.
(j'ai instancé le connecteur factice de factures, la plateforme de payement factice et une régie afin de pouvoir instancier la cellule historique des factures).

> Il reste le soucis des backgrounds opaques des zebras. Oui j'ai oublié de t'en parler, en fait je ne comprend pas le props (pour moi on a le même rendu sur clappoti-les-canards). > Il y a une url consultable ? Non, je demande. (j'ai instancé le connecteur factice de factures, la plateforme de payement factice et une régie afin de pouvoir instancier la cellule historique des factures).
nroche commented 2022-11-16 11:24:55 +01:00
Author
Owner
Copy Link

Quel est la cellule utilisé dans l'exemple

Historique des factures
https://montoulouse.eservices.toulouse-metropole.fr/manage/pages/44/#open:lingo_itemshistory-1-general

Il y a une url consultable ?

https://montoulouse.eservices.toulouse-metropole.fr/facturation/
Se connecter pour avoir des factures :
publik.toulouse+parent@gmail.com / Toulouse31#

> Quel est la cellule utilisé dans l'exemple Historique des factures https://montoulouse.eservices.toulouse-metropole.fr/manage/pages/44/#open:lingo_itemshistory-1-general > Il y a une url consultable ? https://montoulouse.eservices.toulouse-metropole.fr/facturation/ Se connecter pour avoir des factures : publik.toulouse+parent@gmail.com / Toulouse31#
tjund commented 2022-11-17 09:47:13 +01:00
Owner
Copy Link

C'est du lingo et le CSS des tableaux lingo ne se base pas sur table.scss.
La couleur des zabra est solide.

Je valide cette PR.
Et ouvre on va passer par un nouveau ticket pour les CSS des tableaux lingo

C'est du lingo et le CSS des tableaux lingo ne se base pas sur _table.scss_. La couleur des zabra est solide. Je valide cette PR. Et ouvre on va passer par un nouveau ticket pour les CSS des tableaux lingo
tjund approved these changes 2022-11-17 09:47:37 +01:00
nroche force-pushed wip/71297-toulouse-2022-remove-gradient-on-tables from 5b323c2dd8 to 148e419783 2022-11-17 10:24:36 +01:00 Compare
tjund closed this pull request 2022-12-08 16:00:20 +01:00
Some checks reported errors
gitea-wip/publik-base-theme/pipeline/pr-main This commit looks good
Details
gitea-wip/publik-base-theme/pipeline/head Build started...
Details
gitea/publik-base-theme/pipeline/head Something is wrong with the build of this commit
Details

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No reviewers
tjund
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#14
No description provided.
Delete Branch "wip/71297-toulouse-2022-remove-gradient-on-tables"

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?