feat(user): polish list + edit dialog + harmonized confirmations (D1+D2+D4)#39
Merged
Merged
Conversation
- D1: align action gears (UserListView column actions -> align: end, row gear sized x-small to match the table tools gear in the header). Status column header centered above the lock icons. - D2: bold+red name in all sensitive confirmations (lock/unlock/isolate/restore/resetPassword + delete) across UserListView, UserEditDialog (in-popup action menu) and GroupMembersView, using the LigojConfirmDialog default slot. Bulk-delete count also rendered bold+red via two plugin-local fragments. CompanyEditView and GroupEditView already follow the pattern, no-op there. - D4: emails as a list (v-chip in UserListView + v-combobox multiple in UserEditDialog). Fixes a latent bug where editing a user with multiple emails would drop all but the first at save time. Note: the equivalent upgrade on DelegateEditDialog (introduced by PR #35) is intentionally deferred to a follow-up mini-commit once #35 has merged, to keep the two PRs independent.
|
|
Terracosmos
added a commit
that referenced
this pull request
May 27, 2026
Both delete dialogs of the Delegate screen now render the receiver name in <strong class="text-error"> via the LigojConfirmDialog default slot, matching the pattern of PR #39 for User / Company / Group / GroupMembers: - DelegateListView: the trash-icon dialog from each row (inline <v-dialog> converted to LigojConfirmDialog + slot) - DelegateEditDialog: the "Delete" button inside the edit popup (existing LigojConfirmDialog with :message= switched to the slot) The host's monolithic delegate.deleteConfirm key stays intact; two plugin-local fragments (delegate.deleteConfirmBefore/After) are added in FR + EN to wrap the name. Mini-commit deliberately split from PR #39 because the Delegate dialog files did not exist on feature/vuejs until PR #35 was merged. Closes #37
4 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Contexte
Trois améliorations sur l'écran Utilisateurs :
Détail des changements
D1 — Alignement des engrenages
UserListView.vue: colonneactionsenalign: 'end'et bouton de ligne ensize="x-small"pour matcher exactement le bouton de menu de la table (qui sert d'engrenage de configuration). Le décalage venait du fait que le menu de table utilisesize="x-small"alors que les boutons de ligne étaient ensize="small".locked(Statut) enalign: 'center'pour centrer le header au-dessus des cadenas.D2 — Confirmations en gras+rouge via le slot rich content de LigojConfirmDialog
UserListViewdeleteDialog<v-dialog>avec strong text-errorUserListViewactionDialog (lock/isolate/reset)<v-dialog>, monolithicuser.<action>Confirm+{id}UserListViewbulkDelete<v-dialog>+ monolithic messageUserEditDialogconfirmDeleteUserEditDialogactionDialog:message=monolithiqueGroupMembersViewremoveDialog:message=monolithiqueCompanyEditView,GroupEditViewD4 — Emails comme liste
UserListViewslot#item.mails: v-chip pour chaque mail, limité à 2 + indicateur+Nsi l'utilisateur en a plus.UserEditDialog:form.mail: ''→form.mails: []. Champ texte remplacé parv-combobox+multiple+chips+closable-chips. Hint i18nPress Enter or Tab to confirm each email. Fallback string→array au load pour tolérer un payload legacy.mail: form.value.mail(singulier) — seul le 1er email était sauvegardé, les autres perdus.i18n (plugin-id, FR + EN)
user.{lock,unlock,isolate,restore,resetPassword}ConfirmBefore/Afterid.group.removeConfirmBefore/After(et suppression de la clé monolithiqueid.group.removeConfirm)common.bulkDeleteConfirmintact) :common.bulkDeleteConfirmBefore/Afteruser.emailHintpour le comboboxOut of scope
DelegateEditDialog.vue(créé par PR feat(delegate): popup dialog + list polish (D3+D5+D6+D7+D8) #35) est intentionnellement reporté à un mini-commit post-merge de feat(delegate): popup dialog + list polish (D3+D5+D6+D7+D8) #35, pour découpler les deux PRs.LigojConfirmDialogcôté core n'est pas touché — on utilise son slot par défaut existant.Test plan
npm run build→ 28 modules, 115 kB JSvitest run→ 14/14mvn install -DskipTests→ BUILD SUCCESS