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

La side bar et l'onglet career affichent des associations en trop #331

Closed
noeamiot opened this issue Mar 2, 2020 · 5 comments · Fixed by #332
Closed

La side bar et l'onglet career affichent des associations en trop #331

noeamiot opened this issue Mar 2, 2020 · 5 comments · Fixed by #332
Assignees
Labels
javascript Pull requests that update Javascript code not-urgent-issue Has to be fixed, but doesn't prevent from using the app. Not to be mistaken for not-urgent-feature UI User Interface issues

Comments

@noeamiot
Copy link
Member

noeamiot commented Mar 2, 2020

La side bar et l'onglet Career du profile affichent les associations même si l'utilisateur n'a pas été accepté dans celles-ci / s'l follow uniquement l'association.
Pour la side bar, je pense qu'il faudrait afficher dans un menu séparé celles où l'utilisateur est accepté / pas accepté / uniquement follower.
Pour l'onglet career, il faut uniquement afficher les associations où l'utilisateur est accepté.

@noeamiot noeamiot added not-urgent-issue Has to be fixed, but doesn't prevent from using the app. Not to be mistaken for not-urgent-feature javascript Pull requests that update Javascript code UI User Interface issues labels Mar 2, 2020
@noeamiot noeamiot self-assigned this Mar 2, 2020
@noeamiot
Copy link
Member Author

noeamiot commented Mar 2, 2020

Je viens de me rendre compte pour la sidebar, que la différence est actuellement faite entre les assos suivies (icone pouce) et les autres (icone mains serrées). Je propose plutôt que de les séparer, d'ajouter une nouvelle icône en fonction de si l'utilisateur a été accepté ou non.

@MercierCorentin
Copy link
Contributor

Je trouve intéressant de pouvoir voir les assos qu'on a suivies lors des semestres précédents. Si on décide d'enlever les assos suivies de l'historique il faudrait également décider de où les mettre.
Pour la side bar je trouve que c'est une bonne idée, tu penses à un titre de sous-menu en particulier? Je pense à Association suivies là maintenant.
Aussi, cela fait un petit moment que je me dit que les liens utiiles devraient être en dessous des raccourcis, cela est-il une bonne idée? Peut-être pourrais-tu profiter de ce travail sur la sidebar pour changer cela en même temps.

@noeamiot
Copy link
Member Author

noeamiot commented Mar 2, 2020

Je viens de lire ton message, je vais revoir ma PR pour faire ces changements.
La solution que j'ai trouvé pour la sidebar c'est de rajouter une icone (cf photo)
image

Pour la carrière, j'ai simplement caché les associations où l'utilisateur n'était pas un membre accepté mais en effet, je pourrais mettre les assos suivies dans un autre onglet ou avec un icone particulière. Je ferais un test et enverrai une capture d'écran.

@noeamiot noeamiot linked a pull request Mar 2, 2020 that will close this issue
@noeamiot
Copy link
Member Author

noeamiot commented Mar 3, 2020

En déplaçant les liens utiles j'ai aussi dû réfléchir a où placer les services favoris. Pour la sidebar, j'ai essayé de placer liens utiles entre "raccourcis" et "services favoris" (voir photo)
image
Je pense que si je mettais liens utiles en dessous de services favoris, ils n'auraient plus été assez visibles ...

Pour l'onglet carrière, je pense que quelle association est suivie manque de clarté, sur la capture d'écran en dessous, j'ai rajouté un "rôle" association suivie.
image

Pour les deux modifications, ce serait cool d'avoir des avis sur ce qui est le plus ergonomique.

@MercierCorentin
Copy link
Contributor

Super les changements comme ça, on comprends mieux!

MercierCorentin added a commit that referenced this issue Mar 6, 2020
…-and-hide-career

Rearange assos sidebar and hide career (fixing #331)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
javascript Pull requests that update Javascript code not-urgent-issue Has to be fixed, but doesn't prevent from using the app. Not to be mistaken for not-urgent-feature UI User Interface issues
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants