Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Redesign #163

Closed
wants to merge 82 commits into from
Closed

Redesign #163

wants to merge 82 commits into from

Conversation

laem
Copy link
Contributor

@laem laem commented Feb 1, 2018

#160
Inclut les changements de /navigation

@Morendil Morendil temporarily deployed to syso-pr-163 February 1, 2018 16:43 Inactive
@laem laem changed the base branch from master to navigation February 1, 2018 16:43
@laem
Copy link
Contributor Author

laem commented Feb 6, 2018

  • Faire en sorte que ça tourne bien avec la saisie du salaire de base au lieu du salaire brut
  • Expliciter ce qu'est le salaire brut de base (actuellement, aucune explication du salaire, c'est un problème car c'est la saisie principale)
  • s'assurer que le changement d'objectif marche toujours après avoir entré des salaires
  • Faut-il prendre la vraie définition du salaire brut et donc y ajouter les avantages sociaux (complémentaire, prévoyance) ? Ou garder la définition du salaire comme des € purs ?

Ajouter au passage le forfait social ? Non

@Morendil Morendil temporarily deployed to syso-pr-163 February 7, 2018 14:10 Inactive
@Morendil Morendil temporarily deployed to syso-pr-163 February 7, 2018 14:12 Inactive
@laem laem changed the base branch from navigation to master February 7, 2018 14:12
@laem laem changed the title Correction salaire brut Correction salaire brut [WIP] Feb 7, 2018
@Morendil Morendil temporarily deployed to syso-pr-163 February 7, 2018 17:18 Inactive
@Morendil Morendil temporarily deployed to syso-pr-163 February 8, 2018 12:46 Inactive
@Morendil Morendil temporarily deployed to syso-pr-163 February 13, 2018 09:53 Inactive
@Morendil Morendil temporarily deployed to syso-pr-163 February 13, 2018 17:37 Inactive
@Morendil Morendil temporarily deployed to syso-pr-163 February 13, 2018 17:42 Inactive
@laem
Copy link
Contributor Author

laem commented Feb 13, 2018

@Morendil @gbayard
J'ai fait quelques améliorations des descriptions des salaires, des titres (on parle de salaire de base et plus de salaire brut de base) et d'UI depuis la démo de ce matin.

Je vous laisse retester. Attention:

  • il reste un problème lors du changement d'objectifs.
  • désactiver la possibilité de saisie "salaire de base" quand l'objectif est salaire brut. Ce n'est pas assez utile aujourd'hui (les éléments que l'on calcule ne sont pas assez fournis) et ça pète la vue niveau 2.

Je crois toujours que cette PR est une amélioration de l'existant, et qu'on pourra essayer une autre solution si on repère de très grosses incompréhensions.

On est d'accord que c'est pas l'idéal. Mais je pense que l'idéal passe par un gros travail de design de la sélection des objectifs / sélection de la saisie de salaire, pour lequel je n'ai pas vraiment de piste aujourd'hui à part l'infographie sur les salaires / les cotisations.

Au passage, les stats nous enseignent que la grosse majorité ne choisit qu'un objectif, mais qu'aucun n'est négligeable. @Morendil Je ne vois pas comment regarder ce que les gens font après statistiquement.

@Morendil Morendil temporarily deployed to syso-pr-163 February 13, 2018 18:38 Inactive
@Morendil Morendil temporarily deployed to syso-pr-163 February 26, 2018 18:04 Inactive
@laem
Copy link
Contributor Author

laem commented Feb 26, 2018

@Morendil @gbayard
Je pense avoir trouvé un design sympa pour notre problème de sélection des objectifs.
Je reprend cette PR pour y faire une première implé.

@Morendil Morendil temporarily deployed to syso-pr-163 February 28, 2018 16:47 Inactive
@Morendil Morendil temporarily deployed to syso-pr-163 March 1, 2018 10:07 Inactive
@Morendil Morendil temporarily deployed to syso-pr-163 March 1, 2018 18:09 Inactive
@laem
Copy link
Contributor Author

laem commented Mar 1, 2018

  • Vérifier que la compilation en prod est bien fonctionnelle après le passage à Webpack 4

laem and others added 25 commits April 12, 2018 16:01
La derniere version LTS, pour pouvoir utiliser la fonctionnalite object
spread
Pas de calculs s'il y a une erreur de saisie quelque part dans
'conversation'
... mais moche en termes d'UX, car ca efface tout avant la nouvelle
saisie
C'est destine au mobile, mais l'apparition automatique du clavier est
perturbante
@laem
Copy link
Contributor Author

laem commented Apr 12, 2018

On part sur le redesign2.

@laem laem closed this Apr 12, 2018
@johangirod johangirod deleted the correction-salaire-brut branch October 3, 2019 09:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants