help to better help you:

Please: add always Joomla / JEM version and details to your posts, so we can try to reproduce your issue!

Status ribbon on mod_jem_teaser events

Re: Status ribbon on mod_jem_teaser events

2 weeks 2 days ago
#34398
Despite my years of coding experience, things have changed. I've trained the project my AI with JEM requirements and specifications, plus my coding style. After months of training, it's now a huge hel, much faster than me. I focus on design, code review, and testing. I'm sleeping little, so there's a lot of unseen work behind this. The AI has significantly improved my efficiency.

I'm glad it helps you. It's a general improvement that many others will end up using.
Thanks for reporting it. It makes JEM a little bit better
At the end of the day, computing and technology, like Joomla and JEM are here to help the end user and make our lives a little easier. Help us to be better.

Please Log in or Create an account to join the conversation.

Re: Status ribbon on mod_jem_teaser events

1 week 4 days ago
#34426
Hi,
Dunno if it's a regression or the new expected behavior but I'm now not able to get a big image on the mod_jem_teaser module. Previously, by setting "Event content" in "Associated article usage", I got a big image. Unless I missed a parameter, if I create a new event with the same settings and image, the image displayed in the module is always a thumbnail (whereas it is still a big image for the event created before the module update). You can see both events on lafourmilieredemontmidi.fr/ . Removing image on the old event doesn't work, maybe data is messed in db for it. Anyway, it would be great to be able to display a big image in the module as previously.

Please Log in or Create an account to join the conversation.

Re: Status ribbon on mod_jem_teaser events

1 week 3 days ago
#34429
Fixed. Check the last version JEM 5.0.1 RC4 (Refresh).
You have a override the teaser module, update it.
At the end of the day, computing and technology, like Joomla and JEM are here to help the end user and make our lives a little easier. Help us to be better.

Please Log in or Create an account to join the conversation.

Re: Status ribbon on mod_jem_teaser events

1 week 3 days ago
#34431
I've removed the overrides. Is there a way to get the event image in full size in the teaser module as previously? I couldn't figure out how to.

Please Log in or Create an account to join the conversation.

Re: Status ribbon on mod_jem_teaser events

1 week 3 days ago
#34432
Hi fabix,I checked the current
Code:
mod_jem_teaser
code and the related open GitHub image issues, and I think your observation is correct.Removing the old template overrides does not restore the previous full-size image behaviour.In the current
Code:
5.0.1-bugfixes
code,
Code:
mod_jem_teaser/helper.php
stores two different image paths:
Code:
$dimage['thumb']
→ used as
Code:
eventimage
Code:
$dimage['original']
→ used as
Code:
eventimageorig
The Teaser template then uses
Code:
eventimage
as the
Code:
<img src>
, which means that the thumbnail is displayed. The original image is only used as the target when clicking/enlarging the image.There was also a recent commit:
Code:
196e398c2689a9d7683ba1c6a2fd604242365075
Fix image thumbnail resolution across JEM views and modulesThat change explicitly preserves the intentional full-image behaviour for Banner and Jubilee, but Teaser continues to use the thumbnail.So unless I am overlooking an existing module option, there currently doesn't seem to be a setting in
Code:
mod_jem_teaser
to switch the displayed image back to the original/full-size image.
Code:
Associated article usage = Event content
does not change this behaviour either. It can determine where event content or an event image originates, but after that the Teaser module still applies its own image handling.Temporary solutionYou can restore the full-size image without changing JEM core files by creating a fresh Joomla template override for
Code:
mod_jem_teaser
.I would not restore an old override, because it may miss newer JEM changes such as the status ribbons and recent image handling.Create a new override from the currently installed JEM version:System → Site Templates → your template → Create Overrides → Modules → mod_jem_teaserThen edit the applicable layout, for example:
Code:
/templates/your-template/html/mod_jem_teaser/default.php
or, if you use the responsive layout:
Code:
/templates/your-template/html/mod_jem_teaser/responsive.php
The current visible event image uses:
Code:
src="<?php echo $item->eventimage; ?>"
To use the original image instead:
Code:
src="<?php echo htmlspecialchars($item->eventimageorig, ENT_QUOTES, 'UTF-8'); ?>"
There is one additional detail in the current template.The
Code:
<img>
also receives the dimensions calculated for the thumbnail:
Code:
width="<?php echo (int) $item->eventimagewidth; ?>" height="<?php echo (int) $item->eventimageheight; ?>"
So changing only the URL may still display the original image using thumbnail dimensions.For the override I would therefore change the complete image output from something like:
Code:
<img class="float_right image-preview" style="<?php echo $eventImageStyle; ?>" src="<?php echo $item->eventimage; ?>" width="<?php echo (int) $item->eventimagewidth; ?>" height="<?php echo (int) $item->eventimageheight; ?>" alt="<?php echo $item->title; ?>" itemprop="image" />
to:
Code:
<img class="float_right image-preview" style="height:auto;max-width:100%;" src="<?php echo htmlspecialchars($item->eventimageorig, ENT_QUOTES, 'UTF-8'); ?>" alt="<?php echo $item->title; ?>" itemprop="image" />
The same principle applies to
Code:
responsive.php
.This keeps the current JEM Teaser code and newer status-ribbon functionality, while changing only the image source used for display.Related current GitHub workI also noticed that the current JEM development direction already makes this distinction increasingly relevant.#2327 — New image workflow for frontend Event and Venue formsJEM 5.1 introduces separate Event intro/list and Event full/detail images, each with its own image profile/resolution.That actually fits this use case very well: a module such as Teaser is essentially a list/presentation context, while the event detail page is a full/detail context.#2330 — Move Image from the backend sidebar to a main tabThis proposal uses the same distinction in the backend:
  • Event intro/list image
  • Event full/detail image
and aims to use one common image policy in frontend and backend.#2332 — Add a dedicated Images tab to JEM SettingsThis open proposal goes another step further and centralises:
  • image processing settings;
  • event intro/list and full/detail image policies;
  • venue and category image settings;
  • storage policies.
So perhaps the long-term solution should be aligned with this work rather than treating Teaser image selection as an isolated exception.For JEM 5.0.1, a simple module option such as:Image source
  • Thumbnail
  • Original / full-size image
would provide backwards compatibility.For JEM 5.1, it might be even cleaner if Teaser follows the new image-profile concept, for example:Event image
  • Intro/list image
  • Full/detail image
possibly with the generated thumbnail/original choice handled by the applicable image policy.That would make the behaviour explicit and consistent with the direction already described in #2327, #2330 and #2332.It would also be useful to confirm whether Teaser changing to the thumbnail was an intentional behavioural change or an unintended backwards-compatibility change.Regards,
Bluefox
JEM 5.01 (beta) + Joomla 6

Please Log in or Create an account to join the conversation.

Re: Status ribbon on mod_jem_teaser events

1 week 1 day ago
#34433
Please, read this issue with the new options in the JEM module:
github.com/jemproject/JEM-Project/issues/2344
You can downoad a new refresh JEM 5.0.1 RC4 and test it.
At the end of the day, computing and technology, like Joomla and JEM are here to help the end user and make our lives a little easier. Help us to be better.

Please Log in or Create an account to join the conversation.

Time to create page: 0.705 seconds