Fixed up properties in ad mobile sizes for dfp and adsense

This commit is contained in:
vi 2015-09-01 13:44:56 +10:00
parent 67141dd884
commit 5eecf0d731
2 changed files with 40 additions and 33 deletions

View File

@ -3,6 +3,8 @@ import PageTracker from 'discourse/lib/page-tracker';
var ad_width = ''; var ad_width = '';
var ad_height = ''; var ad_height = '';
var ad_code = ''; var ad_code = '';
var ad_mobile_width = 320;
var ad_mobile_height = 50;
var ad_mobile_code = ''; var ad_mobile_code = '';
var currentUser = Discourse.User.current(); var currentUser = Discourse.User.current();
var publisher_id = Discourse.SiteSettings.adsense_publisher_code; var publisher_id = Discourse.SiteSettings.adsense_publisher_code;
@ -94,7 +96,6 @@ if (Discourse.SiteSettings.adsense_publisher_code) {
data["post-bottom"]["ad_width"] = parseInt(splitWidthInt(Discourse.SiteSettings.adsense_post_bottom_ad_sizes)); data["post-bottom"]["ad_width"] = parseInt(splitWidthInt(Discourse.SiteSettings.adsense_post_bottom_ad_sizes));
data["post-bottom"]["ad_height"] = parseInt(splitHeightInt(Discourse.SiteSettings.adsense_post_bottom_ad_sizes)); data["post-bottom"]["ad_height"] = parseInt(splitHeightInt(Discourse.SiteSettings.adsense_post_bottom_ad_sizes));
} }
// }
} }
@ -105,6 +106,8 @@ export default Ember.Component.extend({
publisher_id: publisher_id, publisher_id: publisher_id,
ad_width: ad_width, ad_width: ad_width,
ad_height: ad_height, ad_height: ad_height,
ad_mobile_width: ad_mobile_width,
ad_mobile_height: ad_mobile_height,
init: function() { init: function() {
this.set('ad_width', data[this.placement]["ad_width"] ); this.set('ad_width', data[this.placement]["ad_width"] );
@ -123,8 +126,8 @@ export default Ember.Component.extend({
}.property('adWrapperStyle'), }.property('adWrapperStyle'),
adWrapperStyleMobile: function() { adWrapperStyleMobile: function() {
return `width: 320px; height: 50px; margin:0 auto;`.htmlSafe(); return `width: ${this.get('ad_mobile_width')}px; height: ${this.get('ad_mobile_height')}px; margin:0 auto;`.htmlSafe();
}, }.property('ad_mobile_width', 'ad_mobile_height'),
adInsStyleMobile: function() { adInsStyleMobile: function() {
return `display: inline-block; ${this.get('adWrapperStyleMobile')}`.htmlSafe(); return `display: inline-block; ${this.get('adWrapperStyleMobile')}`.htmlSafe();

View File

@ -3,6 +3,8 @@ import loadScript from 'discourse/lib/load-script';
var const_width = ''; var const_width = '';
var const_height = ''; var const_height = '';
var const_mobile_width = 320;
var const_mobile_height = 50;
var currentUser = Discourse.User.current(); var currentUser = Discourse.User.current();
var _loaded = false, var _loaded = false,
@ -160,6 +162,8 @@ function loadGoogle(settings) {
export default Ember.Component.extend({ export default Ember.Component.extend({
const_width: const_width, const_width: const_width,
const_height: const_height, const_height: const_height,
const_mobile_width: const_mobile_width,
const_mobile_height: const_mobile_height,
classNames: ['google-dfp-ad'], classNames: ['google-dfp-ad'],
loadedGoogletag: false, loadedGoogletag: false,
@ -174,8 +178,8 @@ export default Ember.Component.extend({
}.property('const_width', 'const_height'), }.property('const_width', 'const_height'),
adWrapperStyleMobile: function() { adWrapperStyleMobile: function() {
return `width: 320px; height: 50px;`.htmlSafe(); return `width: ${this.get('const_mobile_width')}px; height: ${this.get('const_mobile_height')}px;`.htmlSafe();
}, }.property('const_mobile_width', 'const_mobile_height'),
_initGoogleDFP: function() { _initGoogleDFP: function() {
var self = this; var self = this;