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

New relative sharing tool (%age) #96

Merged
merged 3 commits into from
Jan 12, 2016
Merged

New relative sharing tool (%age) #96

merged 3 commits into from
Jan 12, 2016

Conversation

MathieuNls
Copy link
Member

#61

@MathieuNls MathieuNls added this to the Tw 1.3 milestone Jan 7, 2016
@MathieuNls MathieuNls deployed to tw-prepod-pr-96 January 7, 2016 22:04 Active
@AlphonseJr
Copy link
Collaborator

Hate de tester ça ! J'ai lancé une fake mesure, je test dans 12h+, je vais me coucher j'ai une grosse réunion demain !
Thanks pirate

@AlphonseJr
Copy link
Collaborator

Il y a une faute, c'est "your" plutôt que "you're" et les services de sharing ne sont pas les bons, il faudrait Twitter, Facebook et Mail à la place de Google.

capture d ecran 2016-01-10 a 21 32 10

Il y a aussi un peu trop d'espace avant le back to measure, on est obligé de scroll pour le voir !

Petite question : pour faire les choses bien, je peux faire ce commentaire ici ou plutôt sur l'issue à l'origine ?

Thanks !

@MathieuNls
Copy link
Member Author

  • Il y a une faute, c'est "your" plutôt que "you're" -> OK
  • et les services de sharing ne sont pas les bons, il faudrait Twitter, Facebook et Mail à la place de Google. -> Fb et twitter devraient apparaitre ici, est-ce que t'as un bloqueur de pub ?. De plus email, je suis en contre, du moins pas en replacement de G+. Si on a des share sur G+, c est du referencement gratuit.
  • Il y a aussi un peu trop d'espace avant le back to measure, on est obligé de scroll pour le voir ! -> Il doit y avoir le feed insta dans cet espace, est-ce que t'as un bloqueur de pub ?
  • Petite question : pour faire les choses bien, je peux faire ce commentaire ici ou plutôt sur l'issue à l'origine ? -> ici

@AlphonseJr
Copy link
Collaborator

Oui tu as raison pour G+, ça ne fait pas de mal donc go pour les 4 !
J'ai Adblock mais ça ne m'empêchait pas d'avoir les résults du share de pingdom par exemple et on me dit dans l'oreillette qu'on est un paquet à avoir Adblock :D
Et si ça peut t'aider, j'ai toujours eu Adblock d'installé mais avant je voyais bien le feed instagram

I removed the sharrre library which was vector of problems (adblocks & all). I also added a sharing by email. @AlphonseJr: Note that it's not possible anymore to open a share popup with pre-set text for Facebook & Gmail. It still works for tweeter and by email
@MathieuNls MathieuNls deployed to tw-prepod-pr-96 January 10, 2016 23:37 Active
@AlphonseJr
Copy link
Collaborator

Ok alors je viens de tester, c'est cool avec les nouveaux boutons ! :)
On devrait changer le lien dans le mailto vers le site plutôt que la page facebook 👍

share email

Et le feed instagram n'apparait toujours pas chez moi :(

missing instagram

On y est presque !

@MathieuNls MathieuNls deployed to tw-prepod-pr-96 January 12, 2016 19:01 Active
@AlphonseJr
Copy link
Collaborator

All good ! Merging pull request :)

AlphonseJr added a commit that referenced this pull request Jan 12, 2016
New relative sharing tool (%age)
@AlphonseJr AlphonseJr merged commit 88c13c4 into dev Jan 12, 2016
@MathieuNls MathieuNls deleted the 61_sharing branch February 15, 2016 15:44
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