Diomed1780
Forum Replies Created
-
And of course by changing to say 95vh you might solve the problem for the iPad issue, but you’re bringing the top of the top panel into view on other browsers, somewhat spoiling the full-page presentation.
Interestingly this same effect occurs on the WordPress Twenty Seventeen demo site, Needle & Hook. I checked it out when I got the same issue as Brian. So maybe it’s something to look at, even though Brian’s issue was resolved.
Forum: Plugins
In reply to: [wpMandrill] Adapting wp_new_user_notificationOK, I have a working solution, though it’s not ideal and if anyone can develop on it to produce a better way, then I would be happy to hear.
My problem was that once I tried to adapt using a Mandrill payload filter, there are two parts of the wp_new_user_notification – a message ($message) that goes to the admins to alert them to the new user, and one (also $message) that goes to the new user, with their username, password, plus the text that I have added. Using the payload filter and adding a custom $message[‘subject’] or $message[‘html’] affects BOTH messages, so that they are identical. If I don’t adapt the $message, then it reverts to the standard WordPress new user notification messages.
My quick-and-dirty solution (not that I arrived at it so quickly) – and I realise it is not the approved way – was to hack the function in the pluggable.php file. Thus, when it reverts to the ‘standard’ WordPress messages, it produces my customised versions.
Forum: Plugins
In reply to: [wpMandrill] Adapting wp_new_user_notificationHello Will,
Have tried altering the code to
function filterWelcomeSubject($message)
{
if(in_array('wp_wp_new_user_notification', $message['tags']['automatic']) && ( strpos ($message['to']['email'],'[email protected]') !== false ))
{...
Have also tried a number of other permutations, but no joy so far. Also tried setting up a separate ‘if’ condition to test the recipient’s address inside the first condition. The e-mails are received by the new user, and by the admin, but the format reverts to the standard WP notifications, which is what I have adapted in the past, and I would still wish to adapt, to be more welcoming to the new user.Hey ho, will think about it further in the morning.
Forum: Plugins
In reply to: [wpMandrill] Adapting wp_new_user_notificationHi, I’ve been thinking and have been playing around with something like this:
function filterWelcomeSubject($message)
{
if(in_array('wp_wp_new_user_notification', $message['tags']['automatic']) && ( strpos ($message['to'],'[email protected]') !== false ))
{...The idea being, as you suggested, that I try to write a filter for the e-mail that goes to the admin, and one for the mail that goes to any other address. It isn’t yet working – ie it reverts to the standard WordPress filters – and I think my problem must lie in the fact that I am checking for a string in $message[‘to’]. I think I need to call up the ‘to’ address differently – could you offer me some guidance, please?