robertocapannelli/newsletter-card

WordPress Plugin to easily have a newsletter card pop up from bottom page on scroll

newsletter-card's Issues

Force input font size to 16px to avoid iphone to zoom in

Fix outerHeight in main.js

Describe the bug
A clear and concise description of what the bug is.

To Reproduce
Steps to reproduce the behavior:

1. Go to '...'
2. Click on '....'
3. Scroll down to '....'
4. See error

Expected behavior
A clear and concise description of what you expected to happen.

Screenshots
If applicable, add screenshots to help explain your problem.

Desktop (please complete the following information):

• OS: [e.g. iOS]
• Browser [e.g. chrome, safari]
• Version [e.g. 22]

Smartphone (please complete the following information):

• Device: [e.g. iPhone6]
• OS: [e.g. iOS8.1]
• Browser [e.g. stock browser, safari]
• Version [e.g. 22]

Additional context
Add any other context about the problem here.

Mobile layout

Add

#newsletter-card-wrapper {
right: 0;
width: calc(100% - 30px);
margin: 15rem;
max-width: 400px;
bottom: 0;
}

toolchain javascript/css needed

Is your feature request related to a problem? Please describe.
Need a package manager to handle scripts and styles compiling all together

Describe the solution you'd like
Use something like npm

Set cookie when the form is submitted

Describe the bug
When a user submitted the form the first time without push the closing X, cookie won't set automatically.
To Reproduce
Steps to reproduce the behavior:

1. Scroll the page will appear the pop-up newsletter
2. Fill the form
3. Submit
   4.Then refresh the page a the card will appear again

Expected behavior
It should set a longer cookie (about 30 days) even after the form is submitted, before set it it should check if the form is filled correctly without any mandatory field missing.

FIX: use a different duplicated proof name for nonce

Describe the bug
Using the suggested name from WordPress for ajax nonce could be create conflicts between plugins and themes

To Reproduce
Just use my_ajax_obj for nonce name to reproduce the error and use it for another purpose as well

Expected behavior
wp-admin-ajax.php returns 404 error

Show just in home checkbox

Allow submit with return button

Is your feature request related to a problem? Please describe.
You can't submit the form pushing return button on the keyboard

Describe the solution you'd like
Allow submit the form with return button
Describe alternatives you've considered
Let things like now

.env with composer?

Custom style

Describe the solution you'd like
It should give the possibility to customise the front-end, maybe with a custom css field, or let decide the user where the card will be positioned (left or right)

Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.

Additional context
Add any other context or screenshots about the feature request here.

Choose style for background and font size in settings page

Exclude pages

Is your feature request related to a problem? Please describe.
It shouldn't show the card every where at anytime
Describe the solution you'd like
It should let the user decide if some pages could be excluded to showing the card
Describe alternatives you've considered
Do nothing

Fix container class style

Describe the bug
The container class has been changed to newsletter-card-container to avoid css conflict with other plugins or framework

Expected behavior
The class should be fixed in css file

Possibility to choose the card position (left or right)