Jump to content

Responsive banner/campaign sizes


AngryWarrior

Recommended Posts

  • 1 year later...
  • 1 year later...

Hi @andrewatfornax

We (business) have been using OnlyMega (adserver) for a few months and today we decided to try RA. Unfortunately the only problem is the lack of responsiveness (100% width). In our case, we use HTML5 banners made on Google Web Designer. All of our ads are responsive but revive doesn't natively support it. On OnlyMega, we're able to just select a checkbox with "reponsive" and the ad works nicely by adjusting height to 100% width. In any case, i've read all the discussions and I see this is a long term issue. I/my company wouldn't mind becoming patreon supporters (actually if we used revive it would save a few thousand dollars per year) but at the moment it just doesn't make sense for us because our website is extremely depentent on the screen size... I've read the "missing features" page and I don't know what to expect. Should I donate and wait indefinitely for it? What is the best course of action? 

 

Anyway, revive 99% awesome...

Thank you, regards.

Link to comment
Share on other sites

  • 5 months later...
  • 1 month later...
  • 7 months later...

Dear all,

thanks for sharing the changes of the source code, which I have implemented accordingly. But there is nothing happening. I do not see anything different than before in the backend. Did I do something wrong? What do I have to do in order to have responsive banner? Is there something more to be changed if I run a language other than English?

Thanks for your support.

Chris

Link to comment
Share on other sites

First add a class to the image tag like this (e.g. Bootstrap)

$imageTag = "$clickTag<img src='" . htmlspecialchars($imageUrl, ENT_QUOTES) . "' width='$width' height='$height' alt='$alt' title='$alt' class='img-fluid'$imgStatus />$clickTagEnd";

I changed this in the file asyncspc.php at line 3462 (folder 'delivery') but this depend on the method you are using.

Then copy this file
https://raw.githubusercontent.com/revive-adserver/revive-adserver/35aa0c7c0507be3f4a94b7a7501f6100a9884b85/www/delivery_dev/async.js
and replace the file in the folder 'delivery'.
You could minify it if you like.

 

 

Link to comment
Share on other sites

Dear stephan,

thanks for the further changes. Just in order to get it right.

Do I have to add in asyncspc.php

$imageTag = "$clickTag<img src='" . htmlspecialchars($imageUrl, ENT_QUOTES) . "' width='$width' height='$height' alt='$alt' title='$alt' class='img-fluid'$imgStatus />$clickTagEnd";

additionaly to

$imageTag = "$clickTag<img src='" . htmlspecialchars($imageUrl, ENT_QUOTES) . "' width='$width' height='$height' alt='$alt' title='$alt' border='0' class='" . $conf['defaultBanner']['class'] . "' $imgStatus />$clickTagEnd";

which is one of the 5 changes in Github?

It looks for me quite similar.

async.js was not modified in the original Github changes.

Therefore, do I have to change only with your modifications (asyncspc.php and async.js), or do I have to add your modifications additionally to the 5 changes described in Github?

I believe this is not only important for me, but for a lot of other Revive user, because all of us want to use the responsive banners which are state of the art now.

Thanks again for your support.

Chris

Link to comment
Share on other sites

On 8/28/2022 at 1:34 PM, chris said:

I believe this is not only important for me, but for a lot of other Revive user, because all of us want to use the responsive banners which are state of the art now.

Please make some noise ?
I would like to get rid off my patch.
Maybe you could leave a comment at the patch.

With my patch about responsive images there are a bit more changes but until my patch is not added it is easier to add only the change I described here.

Edited by stephan
Link to comment
Share on other sites

Dear stephan,

yes we have to do so.

I have implented your changes now as well. This means the change in asyncspc.php and replacing async.js.

But in the Revive backend there are no changes visible if you select between the banner web, url, html or text. There is still the asterix in width and hight in order to put a value there. Without that you cannot save the banner. No responsive tag can be integrated.

Please can you describe for me and all the others what (exactly) to do in order to put an responsive banner into the backend.

That would be really great.

Thanks again.

Chris

Link to comment
Share on other sites

  • 2 weeks later...
  • 1 year later...

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.
Note: Your post will require moderator approval before it will be visible.

Guest
Reply to this topic...

×   Pasted as rich text.   Paste as plain text instead

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.



×
×
  • Create New...