dailybruin / caeruleum

Daily Bruin's WordPress theme and special projects (2.0)
http://dailybruin.com
8 stars 15 forks source link

Fix mobile ads #110

Open hongee opened 7 years ago

hongee commented 7 years ago

Mobile ads are not loading - and shrinking the existing large banner ads are not possible.

Jeremy's reference implementation below

SITE: Daily Bruin Mobile
AD UNIT: Mobile Banner
AD SIZE(S): 320x50

<!--/* OpenX Asynchronous JavaScript tag */-->

<div id="538378860_INSERT_SLOT_ID_HERE" style="width:320px;height:50px;margin:0;padding:0">
  <noscript><iframe id="b301e745ca" name="b301e745ca" src="//oncampusweb-d.openx.net/w/1.0/afr?auid=538378860&cb=INSERT_RANDOM_NUMBER_HERE" frameborder="0" scrolling="no" width="320" height="50"><a href="//oncampusweb-d.openx.net/w/1.0/rc?cs=b301e745ca&cb=INSERT_RANDOM_NUMBER_HERE" ><img src="//oncampusweb-d.openx.net/w/1.0/ai?auid=538378860&cs=b301e745ca&cb=INSERT_RANDOM_NUMBER_HERE" border="0" alt=""></a></iframe></noscript>
</div>
<script type="text/javascript">
  var OX_ads = OX_ads || [];
  OX_ads.push({
     slot_id: "538378860_INSERT_SLOT_ID_HERE",
     auid: "538378860"
  });
</script>

<script type="text/javascript" src="//oncampusweb-d.openx.net/w/1.0/jstag"></script>
zeehang commented 7 years ago

Mobile ads are getting 3x the traffic - does that mean they're loading on desktop too and not getting displayed? Which doesn't make a ton of sense as the reverse should also be true?

http://mobiledetect.net/

We can look into using this to serve the correct ad size and not just load both and display one.