• Resolved shaunbowen

    (@shaunbowen)


    Hi There, I’ve just found your plugin and think it is one of the best popup plugins available! I have a few suggestions though:

    • It would be good if the main div was display:flex rather than display:block when visible. This would make it easier to centre the content, rather than everything always being aligned to the top.
    • Are you planning to add the option to resize the popup to fit the content? Right now I need to define the height for desktop and mobile, and this often leaves a lot of blank space on some devices (or the content has to scroll if too big)
    • a max-height setting on mobile screens would be good. I have used custom CSS to set this as 95%, but it would be better to have this as an option in the plugin.

    Otherwise great work!

Viewing 3 replies - 1 through 3 (of 3 total)
  • Plugin Support Jasmine

    (@hyenokian)

    Dear @shaunbowen,

    Thank you for the topic, kind words and feedback.
    All suggestions are welcomed and appreciated by our team as we always strive to improve the quality of our plugins.

    Please find the answers to your questions below:

    1. At the moment, implementing this change into the general update of the plugin can cause some errors in the level of the code.
      You can write a CSS code with an !important property to achieve your desired result.
    2. Please clarify what spaces you are referring to. Please provide us with the link, where the given popup is enabled so that we can check out the case. Also, please provide us with screenshots for a better understanding.
    3. I will pass your request to our Product Management Team so they can consider adding this feature in the general update of the plugin.

    Looking forward to your response.

    Thank you.

    Thread Starter shaunbowen

    (@shaunbowen)

    Hi Jasmine, thanks for the reply.

    I can see you have already implemented the Max Height option with the latest update – really great response!

    Regarding my request number 1 of using flex instead of block, unfortunately since this value is applied by the javascript that activates the popup, using display:flex !important would just make the popup show all the time, as it would also override the default display:none value.

    Regarding suggestion 2, what I meant was could you implement a ‘dynamic popup height’ option that will adjust to the content inside it, rather than just fixed heights?

    Plugin Support Jasmine

    (@hyenokian)

    Dear @shaunbowen,

    Thank you for your response.

    As to the max height.
    Yes, we already added this feature to the latest update of the plugin. Hope you enjoy the possibilities this functionality gives you.

    As to the case related to the display:flex.
    As mentioned earlier, unfortunately, we can’t implement the changes to the general update of the plugin as it may cause errors in the code.

    As to the dynamic popup.
    Thanks a lot for your suggestion. I will open a ticket as per your request and will let you know in case we include such a feature in one of the future updates of the plugin.

    Thank you.

Viewing 3 replies - 1 through 3 (of 3 total)
  • The topic ‘Some Suggestions / Feature Requests’ is closed to new replies.