Andrew Munro / AffiliateWP
Forum Replies Created
-
Great!
Give this a shot – I’ve tested to confirm the issue first using your theme’s CSS and also that the CSS below correctly hides that first empty ::before selector, but only for any grid of downloads outputted by EDD:
.edd_downloads_list::before { display: none; }Ok thanks for confirming!
Let me see if I can put together some CSS that will fix the issue with the space. Are you comfortable with adding some custom CSS to your theme? You can add it to the WordPress customizer, there’s a section in there for custom CSS.
Thanks, very helpful.
1. How many downloads have you set to show? 2, or 3?
2. Are both your sites that you tested on using the same theme?
3. Can you do a quick test for me and temporarily activate another theme such as Twenty Nineteen, or Twenty Seventeen and then see if that blank box is still there?The reason for 3 above is because your theme’s CSS is outputting this rule:
div::before, article::before, aside::before, details::before, figcaption::before, figure::before, footer::before, header::before, hgroup::before, menu::before, nav::before, section::before, div::after, ul::after, article::after, aside::after, details::after, figcaption::after, figure::after, footer::after, header::after, hgroup::after, menu::after, nav::after, section::after { content: ""; display: table; }The selector that is being executed is the div::before, a pseudo selector.
Essentially, the content line is inserting a blank element before every single div on the website. Because the Downloads grid uses CSS Grid, the empty space is being treated as part of the grid and thus becomes a grid element with the same dimensions as the others. Since there’s no content it just appears as a blank space.
The downloads shortcode in the current version of EDD does not yet use CSS grid, but it will in the next major upgrade. This is why you don’t see the blank space when using the downloads shortcode (since it uses floats to layout the downloads).
Let me know if the issue persists with the theme swap and we’ll go from there. You can change your theme back of course after we’ve confirmed the issue.
- This reply was modified 7 years, 4 months ago by Andrew Munro / AffiliateWP.
Hi there,
That’s very strange, I’ll try and fix this for you.
I’m not able to replicate this issue, would you be able to show me a live URL of the download grid with the first download missing?
Thanks!
Thanks for the suggestion!
You’re absolutely correct, the ordering does get messed up. I can see why FES resets the array now.
I’m instead going to now release an update for Featured Downloads, that will support FES when it is active. After you see the update, you can revert the change you made to FES and you’ll still see the Featured column as the last column on the downloads screen.
You’re welcome!
Absolutely,
1. In FES, go to
classes/admin/downloads/class-download-table.php
2. In thecolumns()method on line 314, delete the following line:$columns = array();- This reply was modified 7 years, 9 months ago by Andrew Munro / AffiliateWP.
- This reply was modified 7 years, 9 months ago by Andrew Munro / AffiliateWP.
I replicated the issue and worked out why it’s not showing (both the column and the quick edit setting). FES is clearing out the columns on the main downloads screen before setting its own custom columns. This means that any extension that adds custom columns to that screen will have theirs removed.
FES shouldn’t be doing this IMO so I’ve submitted a patch to the FES repo to prevent this behaviour. After it has been reviewed and a new version released, EDD Featured Downloads will work correctly with FES installed.
Excellent, thanks for that! I’ll install FES to replicate and post back when I know more 🙂 hang tight!
Hi,
Sorry, I can’t seem to replicate the issue you are experiencing, I see both the featured column and the quick edit option. For the column, can you check it’s not simply turned off via the “Screen Options” tab?
It’s certainly possible that another plugin is interfering. If that’s the case, I can dig in and find out why. Could you please temporarily deactivate all other plugins except EDD and EDD Featured Downloads and then see if the column and quick edit option show?
If so, activate your plugins again in small batches until the problem comes back and you can identity the plugin that is causing the issue. Post back here with the plugin name and I’ll do my best to replicate the issue and find the cause.
Forum: Plugins
In reply to: [EDD Purchase Gravatars] Not enough gravatars showing despite settingCould do! But then you’d block their gravatars from showing on all their purchases on your site. Perhaps that’s exactly what you want to do, rather than doing it on a per-download basis.
Forum: Plugins
In reply to: [EDD Purchase Gravatars] Not enough gravatars showing despite settingThanks, great suggestion, I can certainly see how that could be helpful. It’s possible for a customer to purchase while logged out so there won’t be an ID to rely on. It could either be a “logged-in” option or it might be better to approach this from a payment ID angle. The Payment stores the email address of the person who purchased, which then fetches their gravatar.
Forum: Plugins
In reply to: [EDD Purchase Gravatars] Not enough gravatars showing despite settingHey Jack,
There’s a few other issues this plugin needs for the next update (including your other suggestion) so I’ll schedule in some time and get those completed.
In the mean time, If you make a small tweak to the main plugin file I can show you how to fix this issue now:
1. Open up the main plugin file,
edd-purchase-gravatars.php
2. On line 148, replace:$logs = $edd_logging->get_logs( $download_id, 'sale' );with:
$logs = $edd_logging->get_connected_logs( array( 'post_parent' => $download_id, 'log_type' => 'sale', 'posts_per_page' => -1 ) );Here’s the exact line to change for reference
https://github.com/easydigitaldownloads/edd-purchase-gravatars/blob/master/edd-purchase-gravatars.php#L148Forum: Plugins
In reply to: [EDD Purchase Gravatars] Not enough gravatars showing despite settingThanks for doing that Jack – I’ve now replicated the issue. I’ll dig in and get a patch going. Cheers!