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

Mail button without title generates invalid mailto link #478

Closed
1 task done
jhinzmann opened this issue Jul 24, 2020 · 0 comments · Fixed by #479
Closed
1 task done

Mail button without title generates invalid mailto link #478

jhinzmann opened this issue Jul 24, 2020 · 0 comments · Fixed by #479

Comments

@jhinzmann
Copy link

I am submitting a

  • Bug Report

What is the expected behavior?

The email button should set the URL as body text even when no further params are given.

What is the current behavior?

The generated share URL looks like mailto:?&body=https%3A%2F%2Fngx-sharebuttons-6xp6ew.stackblitz.io%2F%23%2F when no title is given.
This URL is invalid since the '?' is directly followed by an '&'.
Therefore the mail client opens but does not set anything in the mail body.

What are the steps to reproduce?

What is the use-case or motivation for changing an existing behavior?

The mail button should be usable without providing a title.

Which versions are you using for the following packages?

Share buttons: 8.0.0

jhinzmann added a commit to jhinzmann/ngx-sharebuttons that referenced this issue Jul 24, 2020
Empty params produced invalid URLs by adding an unnecessary '&' in the URL

fixes MurhafSousli#478
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 a pull request may close this issue.

1 participant