Contribution to Arty Popup

  • Resolved Cristian Tacchi

    (@cristiantacchi)


    12 years, 2 months ago

    Hi EnigmaWeb,
    I downloaded your plugin. it works great, but i found that the size of the popup is fixed at 800px (width) and 650px (height).

    I’m not a developer, but i found that this values are recorded and readed from the wordpress database, so i added some lines of code in setting_page.php, after “Popup Image” title, to be able to change them.

    <tr valign="top">
	<td>Popup Image Size</td>
	<td>
	<p><input type="text" name="arty_settings[popup_width]" id="popup_width" size="3" value="<?php echo $options['popup_width']; ?>"> Width of the image</p>
	<p><input type="text" name="arty_settings[popup_height]" id="popup_height" size="3" value="<?php echo $options['popup_height']; ?>"> Height of the image</p>
	</td>
</tr>

    I hope you can add this code (or a better one) to the official version.

    Have a nice day!

    https://www.ads-software.com/extend/plugins/arty-popup/

Viewing 6 replies - 1 through 6 (of 6 total)
  • Plugin Author Maeve Lander

    (@enigmaweb)

    Thanks Cristian, I’m glad you managed to make it the size you want with this.

    Yeah I considered having more options in the settings including custom size like you suggest, but the question is always where to draw the line with features… I just wanted to make this popup as simple and lightweight as possible. But I’ll definitely keep a list of any request from users and if this keeps coming up then I’ll include it in later version.

    Thanks again for your input. ??

    dainismichel

    (@dainismichel)

    hi, is there an easy “is user logged in” switch?

    can that be added to the php file easily?

    Plugin Author Maeve Lander

    (@enigmaweb)

    @dainismichel

    The first option is a box to ‘show popup to admin only’ which you can use for testing purposes…. is this what you mean?

    dainismichel

    (@dainismichel)

    close, i mean the other way around: only show the popup to visitors who are not logged in.

    this one:

    https://codex.www.ads-software.com/Function_Reference/is_user_logged_in

    Plugin Author Maeve Lander

    (@enigmaweb)

    No sorry, not a feature at present.

    Anonymous User 11262101

    (@anonymized-11262101)

    Cristian Tacchi, a great contribution!

Viewing 6 replies - 1 through 6 (of 6 total)
  • The topic ‘Contribution to Arty Popup’ is closed to new replies.