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

Les boutons d'actions dans la sidebar ne sont pas ajaxifiés sur mobile #1973

Closed
firm1 opened this issue Dec 23, 2014 · 8 comments

Comments

@firm1
Copy link
Contributor

commented Dec 23, 2014

Quand (depuis mon mobile) je follow/unfollow un topic ou que je le marque en résolu, j'ai droit a une page qui se recharge.

Sur Chrome comme sur Firefox.

Le bouton citer lui marche très bien

@gustavi gustavi added this to the "Futur proche" (v1.x) milestone Jan 3, 2015

@ghost

This comment has been minimized.

Copy link

commented Feb 11, 2015

Je prends.

@ghost

This comment has been minimized.

Copy link

commented Feb 11, 2015

Le problème vient de la PR : #2226. Qui a été partiellement réalisé.

@Eskimon Eskimon assigned ghost Feb 11, 2015

@ghost

This comment has been minimized.

Copy link

commented Feb 11, 2015

J'ai corrigé le problème pour :

  • Marquer comme résolu
  • Suivre ce sujet
  • Être notifié par courriel

Pas besoin d'être sur mobile, suffit de diminuer la largeur de la page.

@Eskimon Eskimon closed this in d042834 Feb 11, 2015

Eskimon added a commit that referenced this issue Feb 11, 2015
Merge pull request #2283 from A-312/fix-1973-ajaxforget
Fix  #1973: Le menu mobile n'était pas ajaxifié

@Eskimon Eskimon modified the milestones: Version 1.6, "Futur proche" (v1.x) Feb 11, 2015

Situphen added a commit to Situphen/zds-site that referenced this issue Feb 13, 2015
@firm1

This comment has been minimized.

Copy link
Contributor Author

commented Feb 14, 2015

Je ré-ouvre cette issue car la correction ne prend pas en compte un cas.

Scénario:

  • Depuis mon mobile, je vais sur un sujet que je ne suivais pas
  • Je vais dans la sidebar
  • Je clique sur "Etre notifié par mail"
  • Problème : le bouton du dessus me demande encore si je veux suivre le sujet alors que la demande de notification par mail signifie qu'on suit déjà le topic.

Le comportement de l'ajax sur mobile n'a donc pas le meme comportement que sur pc.

@firm1 firm1 reopened this Feb 14, 2015

@ghost

This comment has been minimized.

Copy link

commented Feb 15, 2015

Je ne comprends quel "bouton du dessus" ?

@Eskimon

This comment has been minimized.

Copy link
Member

commented Feb 16, 2015

Je ne comprends quel "bouton du dessus" ?

Lorsqu'on suit par email, le bouton "suivre ce sujet" doit automatiquement etre clique aussi. Et si on on suit pas email, lorsque l'on clique sur "arreter de suivre le sujet" le suivi par email doit etre arrete aussi.

J'ai mal QA, je n'avais pas pense a l'implication de l'un avec l'autre.

@ghost

This comment has been minimized.

Copy link

commented Feb 16, 2015

J'ai mal QA, je n'avais pas pense a l'implication de l'un avec l'autre.

C'est surtout ma faute. ^^"

@firm1 firm1 closed this in 3193288 Feb 24, 2015

firm1 added a commit that referenced this issue Feb 24, 2015
Merge pull request #2347 from A-312/fix-1973-ajaxforget
Fix #1973: Corrige le texte du menu ajaxifié
@firm1

This comment has been minimized.

Copy link
Contributor Author

commented Feb 24, 2015

C'est résolu

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.