Forum Replies Created
- AuthorPosts
kernel
ParticipantAll green ticks, same error.
kernel
ParticipantOk, i am able to get this to work, but all javascript and css has to be called from the <head> section. I now have one error, from this snipit in jquery.iphorm.js:
// Preload the images in the base theme
$(window).load(function () {
window.iPhorm.preload([
'captcha-refresh-icon.png',
'date-picker.png',
'default-loading.gif',
'error.png',
'file-upload-tick.png',
'help-icon.png',
'refresh-icon.png',
'success.png'
], iphormL10n.plugin_url + '/images/');
});
The error is “Can’t find variable: iphormL10n” Please take a look at line 781 in jquery.iphorm.js, this is the only error and would greatly appreciate not having it.
Thanks
kernel
Participant^lol
kernel
ParticipantI have tried the examples shown which eliminates most if not all errors found but this does not include the call for iphorm.
There should be something using javascript and or:
<?php
require('/wp-load.php');
?>
I really need this functionality, i would appreciate a solution. My source for 404.php is accessible for testing on your end to find a result, attached here:
Thanks
kernel
ParticipantWith any plugin an author will always look for features and compatibility. What many disregard, overlook or place last is resource utilization. I would place this plugin in this category, you have many features and you have placed resources to the side.
I like to use this plugin, i don’t mind paying for it. What i DO NOT like is that i have 9 scripts running to have a popup contact form, this does not include the stylesheets.
You need to consolidate, and include options within the admin panel to minimize load. This issue is not known to many users, as most will not be checking with google page speed or take the time to minify the javascript and stylesheets that are bloated. One thing is certain, nobody will object to the files in a minified and consolidated form. You just don’t do it, and yes it may be one more step for each update, but this one plugin equals 6 or more plugins in terms of resources and render blocking ability.
Furthermore, at least 6 of your scripts are less than 3 lines in length, i have a suggestion…
CONSOLIDATE!
What I don’t want to do is hack and edit every file in this plugin and consolidate myself, every time you update. Quform for wordpress is rich in features, but it does not need to be as bloated as you are making it. I understand why you are providing it this way, it’s easier for you to make changes and placing the burden on a paying customers domain. I think we have enough plugins out there to blame for delays in page speed, please take some time in ensuring this plugin is not to blame. Making changes to consolidate this plugin is a win for everyone, hopefully you will see that and do something to improve upon it.
Thanks
kernel
ParticipantResolved,
The “?1” trick didn’t work, i just renamed the .css and corrected the common.php file to resolve the cache.
kernel
ParticipantI found a solution, by adding to my style.css adding a color as well with a 1px size instead of 10.
#fancybox-content[style] {
border-width: 1px !important;
border-color: #545454 !important;
}kernel
ParticipantWith the following code i was able to easily omit the footer.php code and keep everything within the code in a css menu, adding overlayOpacity:
<li><?php if (function_exists('iphorm_popup')) echo iphorm_popup(1, 'CONTACT',
'{ overlayOpacity: 0.6 }'); ?></a></li>You just need to change the “1” to the form you are needing.
kernel
ParticipantThe above text, if the proper method (or) along with the following code to achieve overlay color change should be made available within your guides. Both of which were time consuming to achieve.
Changing the overlay color:
#fancybox-overlay[style] {
background-color: black !important;
}kernel
ParticipantThis works, but is it the way it should be done:
<li><?php if (function_exists('iphorm_popup')) echo iphorm_popup(1, 'CONTACT'); ?></a></li>
kernel
Participantneed code for the html markup listed above, that will fit inside ” ” for a popup form
kernel
Participant<li><a href="the following code">CONTACT</a></li>
- AuthorPosts