Project

General

Profile

Bug #3148

Dans la validation des actes il faudrait mettre en surbrillance le dernier acte validé.

Added by Mikaël Ates over 11 years ago. Updated over 10 years ago.

Status:
Fermé
Priority:
Bas
Assignee:
-
Target version:
Start date:
24 June 2013
Due date:
% Done:

100%

Estimated time:
Patch proposed:
Yes
Planning:

Description

Demande initiale : Il faudrait plus faire ressortir l’acte que l’on est en train de pointer quand on est sur validation des actes. En effet, si l’on est dérangé, on ne sait plus quel acte on était en train de valider.


Files

Associated revisions

Revision 33d0696c (diff)
Added by Serghei Mihai over 10 years ago

agenda: on acts validation highlight block of changed act state

Closes #3148

Revision ce41c769 (diff)
Added by Serghei Mihai over 10 years ago

agenda: highlight last saved updated act

Closes #3148

History

#1

Updated by Mikaël Ates over 10 years ago

  • Target version set to 1.2 Autres
#2

Updated by Serghei Mihai over 10 years ago

Je propose de mettre la bordure du select modifié en rouge.

#3

Updated by Benjamin Dauvergne (en congé) over 10 years ago

Je mettrai la classe changed sur le block contenant l'acte et pas juste le select, ça permettra de choisir un autre effet. On pourrait aussi comparer avec la valeur initiale (en utilisant show_hidden_initial=True sur le champ select).

#4

Updated by Serghei Mihai over 10 years ago

Benjamin Dauvergne a écrit :

On pourrait aussi comparer avec la valeur initiale (en utilisant show_hidden_initial=True sur le champ select).

On n'utilise pas les formulaires django sur cette page, donc je ne peux pas profiter de ta suggestion.

#6

Updated by Benjamin Dauvergne (en congé) over 10 years ago

Serghei Mihai a écrit :

Benjamin Dauvergne a écrit :

On pourrait aussi comparer avec la valeur initiale (en utilisant show_hidden_initial=True sur le champ select).

On n'utilise pas les formulaires django sur cette page, donc je ne peux pas profiter de ta suggestion.

Même si ce n'est pas un formulaire Django on peut toujours ajouter un data-initial-value="{{ act.validation_state }}" ou un truc du genre.

#7

Updated by Serghei Mihai over 10 years ago

  • File 0001-agenda-on-acts-validation-highlight-block-of-changed.patch added

J'ajoute de l'hombre au block de l'acte quand le statut est différent du statut precedent(champ data-previous déjà déclaré).

#8

Updated by Benjamin Dauvergne (en congé) over 10 years ago

Serghei Mihai a écrit :

J'ajoute de l'hombre au block de l'acte quand le statut est différent du statut precedent(champ data-previous déjà déclaré).

Nickel.

#9

Updated by Serghei Mihai over 10 years ago

  • Status changed from Nouveau to Résolu (à déployer)
  • % Done changed from 50 to 100

Appliqué par commit calebasse|commit:33d0696c5b418efd6a2193575bd3f345eb8fceb9.

#10

Updated by Mikaël Ates over 10 years ago

  • Status changed from Résolu (à déployer) to En cours
  • % Done changed from 100 to 0

La modification apportée met en subrillance l'acte sur lequel on vient de faire un choix dans la liste déroulante mais ne met pas en surbrillance le dernier acte validé, c'est à dire l'acte après que l'on ait cliqué sur le bouton 'modifier' et que l'acte a été mis à jour avec succès.

#11

Updated by Serghei Mihai over 10 years ago

  • File 0001-agenda-highlight-last-saved-updated-act.patch added
  • % Done changed from 0 to 50

Nouveau patch

#12

Updated by Benjamin Dauvergne (en congé) over 10 years ago

Sur ça:

-            return HttpResponseRedirect('#acte-frame-'+acte_id)
+            return HttpResponseRedirect('?acte-frame='+acte_id)

Je pense qu'on peut se passer d'un paramètre d'URL pour obtenir l'effet escompté donc on garde ça:

             return HttpResponseRedirect('#acte-frame-'+acte_id)

Et on ajoute:

.frame.acte:target { ... changed CSS ... }
#14

Updated by Serghei Mihai over 10 years ago

  • File deleted (0001-agenda-highlight-last-saved-updated-act.patch)
#15

Updated by Serghei Mihai over 10 years ago

  • File deleted (0001-agenda-on-acts-validation-highlight-block-of-changed.patch)
#16

Updated by Serghei Mihai over 10 years ago

  • Status changed from En cours to Résolu (à déployer)
  • % Done changed from 50 to 100

Appliqué par commit calebasse|commit:ce41c769bfbecfc8acb665bd17b1a247e9d5b072.

#17

Updated by Mikaël Ates over 10 years ago

  • Status changed from Résolu (à déployer) to Fermé

Ok

Also available in: Atom PDF