nicegamer7
Forum Replies Created
-
Actually, to be honest, we might just start using one of the the remaining photo services that are already integrated into Photonic.
I understand this plugin is a hobby, that’s why I mentioned it’s probably not worth solving the above using any elaborate solution. Especially given the fact that Google keeps changing the API. Thank you for your effort!
I see, that makes sense. I think there are maybe some solutions such as caching in the DB and asynchronous calls, but this would require time and effort to implement and might not be worth it in the end. Would you be open to a contribution if I ever come up with something like that? Or maybe it could be a add-on plugin of sorts.
Also regarding the changes to the API, you reminded me, I wanted to ask you what exactly this means for the plugin. Just for clarification, is this the change that’s supposed to take effect on March 31, 2025? I got an email about a month ago about this and I wanted to ask you about it.
Awesome! Just updated and it seems to be working well. Thanks.
Forum: Plugins
In reply to: [Media Library Assistant] Thumbnail URL coming up blankThanks for getting back to me. Your comment about
{+thumbnail_url+}being extracted from{+link+}helped me out.I found that I had a filter on
wp_get_attachment_linkthat was affecting MLA.Thanks a million!
Ah, that’s okay.
The issue is with this option, https://imgur.com/qHnmQ6K. The issue is that the title does not actually slide up, it just appears. The reason it sort of looks like it’s sliding is because the padding is animated, but the height isn’t. This happens because CSS cannot animate the height changing from 0 to “auto”.
Here’s my attempt to fix it. Instead of setting height to “auto” using the slideup-show class, I set the height to the exact height using JS. The issue with this approach is that if the screen is resized while a title is open, it’s height may no longer be correct depending on if the text wrapped to the next line. I was originally trying a different approach to tackle that issue, but it was exhibiting some strange behaviour when quickly moving in and out of thumbnails, so I dropped it. My new idea to solve that is to simply add an event listener for the resize event, and resize any open title that needs resizing. Unfortunately, I didn’t get around to trying that, so here’s the changes I made without that event listener I was talking about.
I tested these changes by just changing the already built files. It’s my first time working with SVN, so hopefully the patch is fine.
https://drive.google.com/file/d/1uTrxc9lm0xEGgMhdP28TE16JZAtvP43m/view?usp=sharing
Forum: Plugins
In reply to: [Flamingo] Auto-Trash All Submissions (Not only Spam)While I understand the feature may be useful, you must understand the author’s point of view.
The author maintains several plugins, each of which are extremely popular and used on many WordPress sites. Adding more features makes the plugin more difficult to maintain, and possibly more complex. Additionally, the author provides these plugins and support for them free of charge, and most likely in his spare time.
Regardless, since I need this feature, I planned to make an addon plugin that will do this. Once I finish, I will publish it and let you know.
Thanks @wfpeter.
Thanks for getting back to me.
While you are correct that when the WAF is optimized, the rules will be updated before anything else runs, that is assuming that the rules are to be updated on that day. For example, take the free version of Wordfence. Rule updates happen at most once a week. If you miss the threshold (which I believe is set to 2 days), you’ll have to wait for the next week for them to be updated automatically.
Even though this is not ideal, this is the best that can be done for sites that are not using their server’s cron. But for sites that are using their server’s cron, it would be beneficial for them if an option was added to Wordfence to use WP Cron, since it is guaranteed to be run periodically, and therefore there is no need to run these jobs for every visitor.
Also, please consider that there are sites that are set up to run WP Cron periodically without a web request. In this situation, the site would again benefit from an option to use WP Cron jobs, since that would allow the rules to be kept up to date. See Trellis’ implementation using wp-cli here.
I acknowledge that this would not be beneficial in every situation, but for sites that do use their server’s cron, the option to use WP Cron jobs would mean less work is done on every page load, and in the case of Trellis (for example), the rules would always be kept up to date. So in the end, I would really appreciate it if an option to use WP Cron jobs was included with Wordfence (maybe buried in the settings). Thanks for taking the time to read all this 😛
- This reply was modified 5 years, 2 months ago by nicegamer7.
- This reply was modified 5 years, 2 months ago by Steven Stern (sterndata).
I will just amend my post above. After looking at the code again, it seems Wordfence does use WP Cron for some things, but not for rule updates. So my question was with regards to rule updates.
Forum: Plugins
In reply to: [Flamingo] Auto-Trash All Submissions (Not only Spam)Okay, thanks for getting back to me 👍.
Forum: Plugins
In reply to: [Flamingo] Auto-Trash All Submissions (Not only Spam)I’d like to implement this feature and submit a PR, but please advise on how you would like it to be done.
Forum: Plugins
In reply to: [Photonic Gallery & Lightbox for Flickr, SmugMug & Others] Preload WarningOkay, I do see both. So I suppose all is good.
Forum: Plugins
In reply to: [Photonic Gallery & Lightbox for Flickr, SmugMug & Others] Preload WarningYes, there is indeed a way to disable Rocket Loader (it turns out it’s called Rocket Loader, not RocketLoader, sorry) for individual scripts.
I’m not sure whether this should be something the plugin does automatically or if it should be available via a filter, but I think it must somehow be integrated in the plugin. See here.
edit: I also question whether this is even needed, as the plugin seems to function correctly, even with the warning. Although, you did mention the double loading that this would be causing, so in that sense I guess it is necessary.
- This reply was modified 5 years, 3 months ago by nicegamer7.
Forum: Plugins
In reply to: [Photonic Gallery & Lightbox for Flickr, SmugMug & Others] Preload WarningSo it is caused by RocketLoader.
After turning RocketLoader off, the warnings go away. Although, even with RocketLoader on, the script still had the correct
typeattribute.Regardless, I’ll just keep it off. Thanks, Sayontan.