-
Notifications
You must be signed in to change notification settings - Fork 1.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. Weβll occasionally send you account related emails.
Already on GitHub? Sign in to your account
β π Background image size is missing when using a dynamic tag #21359
Comments
Yup crazy ommission, this makes the new feature half baked |
Hi @perks98! Thank you for submitting your issue! π Please allow us some time to investigate it, and get back to you with our findings. π§ Best regards |
Yep, totally missed the mark on that one. They locked previous thread (#6778) which is a pain. |
Total fail so far - half baked feature - no progress - this is everything that is wrong with Elementor |
Is this being fixed or just left as is? |
Not fixed in the 3.12 beta and I don't think there are any pull requests to fix so... left as is for now. |
So half develop a feature, release, make a big fanfare about it, have bugs pointed out, and ignore them! |
My request to Elementor support:
Am I confident they've picked up the full scope of this bug? Nope. |
I don't use Elementor support anymore as most issues I raise their developers are aware of but never have an ETA on fixing ~ like #14743 & #18874 OR the excuse is that it has been an issue for so long that users have found ways around it and fixing it now will break that ~ like #20041. These days I try only to report stuff around beta season as they are more likely to action or if it is a big major issue. Unfortunately 3.11 is already out so I don't think they will action this any time soon unless we have 50 users demand for it. ~~ but I could be wrong |
5 years for a solution which is incomplete - it kind of sums Elementor up - if itβs not new and shiny they lose interest and leave bugs to fester |
Crazy that image sizes are still not possible with dynamic background images. That makes them much less useful. Super frustrating. I can't think of a sensible workaround. I just want to use a post's featured image as a banner image.. without having it load the full-resolution version. |
Hey stated that it wasnβt important to them and they didnβt see any need for it - which sucks
β¦ On 5 Apr 2023, at 6:44 pm, Casper Voogt ***@***.***> wrote:
Crazy that image sizes are still not possible with dynamic background images. That makes them much less useful. Super frustrating.
β
Reply to this email directly, view it on GitHub <#21359 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AE6PZSJAFXLO22JQBVSEYQLW7WVP5ANCNFSM6AAAAAAU5JTUIA>.
You are receiving this because you commented.
|
They have been focusing on performance improvements... and this is a big one. If they really cared about performance, they'd fix this. |
Will this half-completed feature ever be finished - it is useless as it stands - the ability to select Background Image Size Options when using a Dynamic Image source is really important for performance |
Any news @nicholaszein just got some questions about it |
Evidently not - another one into the black hole I guess |
@DavidGMiles and @VerdiH please refer to Amitai's section on Elementor's Q3 Roadmap Event. It should clear out any doubt about how we are dealing with bug reports. @DavidGMiles please, if you have any complaints towards the Elementor Company, use the following channel: https://elementor.com/contact This is not the place for this kind of comment. Thank you very much. |
This feature is definitely needed. Was surpised to find that the image size selection was added for background images in 3.11 because I could not see this option. Mostly use background images loaded from post featured image.. |
When I recently asked Elementor Support about this I was told 'This is not classified as a bug by our support teams, this is intended functionality' |
+1 |
Hi @nicholaszein, |
#24146 <-- not sure if that potential code relates to this issue but if it is then it looks like this issue may not get fixed and just a message of "Image size settings donβt apply to Dynamic Images." instead which isn't really a solution. |
Anything for this? |
I'm also awaiting a fix for this. |
This has a huge impact on performance. |
Damn. Still no fix. This is VERY problematic |
This is totally unacceptable. |
Anything? Any work arounds? |
I would also like to have an update on this. Still using full sized images in every dynamic element I do. I tried as workaround to use images but I can't make images center/fit as well as background images. I define min-height for images and object fit to center center but i can't get images center/cover properly like in background images. |
So it's been 624 days and still nothing... |
A very simple update that could boost performance and give design freedom. It's sad. |
This is really a problem. I am using (and loving) loop grids and setting background images using dynamic tags, but can't use an image size with that. I have to use a workaround consisting of absolutely positioned Image elements and a bunch of custom CSS. In this video I show the issue, along with my workaround using Image element and some absolutely positioned elements. This is not an ideal workaround at all; it serves to illustrate why the background image option desperately needs an image size option. |
Yes @caspervoogt tks for sharing but I refuse to go that path to do something so simple, unless i need have a project that requires some high degree of performance. Right now, with image compression and CDN i can live with that but super frustrated of course. |
@miguel-cravidao I know, it is ridiculous isn't it. Granted, it didn't take me long to do (because I am so used to having to use this workaround by now) ... but it's silly and sad to have to jump through such hoops. |
Hey everyone! An update to this issue: Initially this functionality was not intended to be implemented, and images loaded via dynamic tags couldn't have the resolution set. We intended to update the controls to inform this limitation by adding a note below the Background Image selector indicating this was not possible:
However, due to the demand, we added this to our roadmap as a new feature. Unfortunately, after performing various tests we concluded implementing this feature could have a heavy impact on performance both inside the Editor, as well as on the frontend during the first rendering of the page, when the CSS is being generated. So we placed this feature on hold until we improved our infrastructure. But we have great news! π Support for this feature will be considered for Elementor v4, after all dynamic capabilities are implemented, in a future phase!For this reason, we are closing this issue, and merging it with the Feature Request #22750. Please join the discussion there, and cast your vote as well! Thank you all for your feedback! π |
Prerequisites
Description
When using a dynamic tag to pull in an image like the featured image there are no image size options, while without using any tags you get the option to adjust the size.
Also when you add an image to the background first to make the size options appear then decide to use a dynamic tag for an image the size option says there but is not used in final frontend image render.
See steps to reproduce section for more info.
Steps to reproduce
Using a dynamic tag right away doesn't present the image size option:
2023-02-15.20-09-52.mp4
Choosing an image first then dynamic tag doesn't respect the size chosen in preview or frontend (should be thumbnail size in example):
2023-02-15.20-10-49.mp4
Isolating the problem
System Info
System Info
The text was updated successfully, but these errors were encountered: