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

[Feature] Share button #21

Open
wants to merge 7 commits into
base: develop
Choose a base branch
from

Conversation

TuentyFaiv
Copy link

@TuentyFaiv TuentyFaiv commented Mar 25, 2020

DESCRIPTION

Agregue la característica de poder compartir los posts en Facebook, Twitter, copiando el enlace y por medio del share nativo en navegadores mobiles

También acabo de completar la configuración para que el blog sea PWA

INFORMATION

name: Tonalli López
username: https://platzi.com/@TuentyFaiv
coach: Zally
cohort: 1

PR TYPE

  • BlogPost
  • BugFix
  • Feature
  • Enhancement

@@ -0,0 +1 @@
<svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 48 48" width="50px" height="50px"><path fill="#03A9F4" d="M42,12.429c-1.323,0.586-2.746,0.977-4.247,1.162c1.526-0.906,2.7-2.351,3.251-4.058c-1.428,0.837-3.01,1.452-4.693,1.776C34.967,9.884,33.05,9,30.926,9c-4.08,0-7.387,3.278-7.387,7.32c0,0.572,0.067,1.129,0.193,1.67c-6.138-0.308-11.582-3.226-15.224-7.654c-0.64,1.082-1,2.349-1,3.686c0,2.541,1.301,4.778,3.285,6.096c-1.211-0.037-2.351-0.374-3.349-0.914c0,0.022,0,0.055,0,0.086c0,3.551,2.547,6.508,5.923,7.181c-0.617,0.169-1.269,0.263-1.941,0.263c-0.477,0-0.942-0.054-1.392-0.135c0.94,2.902,3.667,5.023,6.898,5.086c-2.528,1.96-5.712,3.134-9.174,3.134c-0.598,0-1.183-0.034-1.761-0.104C9.268,36.786,13.152,38,17.321,38c13.585,0,21.017-11.156,21.017-20.834c0-0.317-0.01-0.633-0.025-0.945C39.763,15.197,41.013,13.905,42,12.429"/></svg>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Por que este elemento no es PNG como los otros iconos?

src/templates/blog-post.js Outdated Show resolved Hide resolved
navigator.clipboard.writeText(url_post).then(function () {
alert(`Enlace: ${url_post} Copiado`);
}, function () {
alert("Error al copiar enlace");
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

¿alert?

{
modal &&
<>
<div className="blogpost__share-modal">
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Esto puede ser un componente independiente

@gndx gndx self-assigned this Mar 25, 2020
@gndx gndx added the enhancement New feature or request label Mar 25, 2020
@TuentyFaiv TuentyFaiv requested a review from gndx March 28, 2020 02:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants