Egnarts94 / JEM-Project

JEM - an Event Manager for Joomla
0 stars 2 forks source link

mod_jem_banner: destroys template in wide module positions (e.g. banner) #69

Open jojo12 opened 5 years ago

jojo12 commented 5 years ago

the eventimage is enormous and it goes over the rest of the site

depebo commented 5 years ago

@Egnarts94 modules\mod_jem_banner\tmpl\responsive\default.php has embeded style on this page, lines 39-95. This results in event image being stretched to 100% page width. Before I start changing mo_jem_banner output can you share the logic behind it please.

jojo12 commented 5 years ago

The logic (in my eyes) The mod_jem banner is an adapted mod_jem_teaser. I use it after all without calendar-img as special adviser for some events like a banner. So the eventimage is bigger and it doesn't need the venueimage. for that the shuffle too.

depebo commented 5 years ago

@jojo12 In any case event image should not be resized to 100% of available space, right? If you have a larger event image you can use it, if not - hide it. I wanted to check with @Egnarts94 the logic behind embedded css there before changing it.

jojo12 commented 5 years ago

or perhaps better a note: "not ideal to use on wide positions"