entrouvert
/
gadjo
15
0
Fork 0
Code Issues Pull Requests 3 Releases Activity

css: add pk-hide-requisiteness form class (#88343) #56

Merged
pmarillonnet merged 1 commits from wip/88343-field-widget-rendering-hide-requisiteness into main 2024-04-16 09:55:27 +02:00
Conversation 2 Commits 1 Files Changed 1 +7
pmarillonnet commented 2024-03-19 15:06:00 +01:00
Owner
Copy Link
No description provided.
pmarillonnet added 1 commit 2024-03-19 15:06:01 +01:00
gitea/gadjo/pipeline/head This commit looks good Details
dcba1b9355 widgets: support hiding field's requisiteness (#88343)
pmarillonnet force-pushed wip/88343-field-widget-rendering-hide-requisiteness from dcba1b9355 to f80a32cd08 2024-03-19 15:14:02 +01:00 Compare
pmarillonnet added 2 commits 2024-03-19 15:19:06 +01:00
gitea/gadjo/pipeline/head This commit looks good Details
95699faa97 css: add pk-hide-requisiteness form class (#88343)
pmarillonnet force-pushed wip/88343-field-widget-rendering-hide-requisiteness from 95699faa97 to 0e1d8f1dd9 2024-03-20 11:18:54 +01:00 Compare
pmarillonnet changed title from WIP: widgets: support hiding field's requisiteness (#88343) to WIP: css: add pk-hide-requisiteness form class (#88343) 2024-03-20 11:27:13 +01:00
pmarillonnet changed title from WIP: css: add pk-hide-requisiteness form class (#88343) to css: add pk-hide-requisiteness form class (#88343) 2024-03-20 11:27:20 +01:00
fpeters approved these changes 2024-04-16 09:47:23 +02:00
fpeters left a comment
Owner
Copy Link

Ça me semble très adhoc pour le cas authentic, j'aurais bien vu ça uniquement là, mais ok.

Ça me semble très adhoc pour le cas authentic, j'aurais bien vu ça uniquement là, mais ok.
pmarillonnet commented 2024-04-16 09:53:48 +02:00
Author
Owner
Copy Link

Ça me semble très adhoc pour le cas authentic, j'aurais bien vu ça uniquement là, mais ok.

En fait je l’ai placé ici parce que ça s’articule avec https://git.entrouvert.org/entrouvert/authentic/pulls/283/files qui vient nuancer l’attributaion de la classe pk-mark-optional-fields — laquelle est définie ici — par cette nouvelle classe pk-hide-requisiteness qui est une variantion de cette première classe.

Ça me paraissait logique que les deux classes soient définies dans le même fichier ici, mais j’ai pas de réticence à définir cela uniquement dans authentic.

> Ça me semble très adhoc pour le cas authentic, j'aurais bien vu ça uniquement là, mais ok. En fait je l’ai placé ici parce que ça s’articule avec https://git.entrouvert.org/entrouvert/authentic/pulls/283/files qui vient nuancer l’attributaion de la classe `pk-mark-optional-fields` — laquelle est définie ici — par cette nouvelle classe `pk-hide-requisiteness` qui est une variantion de cette première classe. Ça me paraissait logique que les deux classes soient définies dans le même fichier ici, mais j’ai pas de réticence à définir cela uniquement dans authentic.
pmarillonnet commented 2024-04-16 09:55:19 +02:00
Author
Owner
Copy Link

[…] j’ai pas de réticence à définir cela uniquement dans authentic.

(mais le relecteur m’invite sur jabber à pousser, pour tagguer déployer dans la foulée etc.)

> […] j’ai pas de réticence à définir cela uniquement dans authentic. (mais le relecteur m’invite sur jabber à pousser, pour tagguer déployer dans la foulée etc.)
pmarillonnet merged commit 606fd39cfc into main 2024-04-16 09:55:27 +02:00
pmarillonnet deleted branch wip/88343-field-widget-rendering-hide-requisiteness 2024-04-16 09:55:27 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
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/gadjo#56
No description provided.
Delete Branch "wip/88343-field-widget-rendering-hide-requisiteness"

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?