Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

google ads set to appear at top, which are not full-screen width, are placed in top left, not top center. #108

Open
georgejecook opened this issue Dec 8, 2014 · 9 comments

Comments

@georgejecook
Copy link

How to correct this?

just had a look through the code and it seems extraordinarily difficult to resolve without hacking the source, i.e. using delegates, etc.

I'd prefer not to have to go hacking your source as I install your lib with cocoapods.

@larsacus
Copy link
Owner

larsacus commented Dec 8, 2014

Can you provide some more information such as OS, platform, logging, etc?

@georgejecook
Copy link
Author

iOS 8.

are there any hooks for resizing the ad views?

@larsacus
Copy link
Owner

larsacus commented Dec 9, 2014

You don't have any control as to the specific size of the ads. The ad network returns an ad at a particular size. I can only specify what type of ads to receive based on platform, etc.

@georgejecook
Copy link
Author

I mean the view it sits in.. any way for me to better control the position (i.e. center it horizontally?).. or at least know when it's been updated so I can place it into a correctly sized black view, and center it in that.

@larsacus
Copy link
Owner

It should automatically center itself horizontally internally -- if not, that should be a bug. You have access to the container view on LARSAdController to manipulate any way you please:

/** The container view that the ads are contained in. Exposed so you can do anything you would want with it.
 */
@property (nonatomic, readonly, strong) LARSAdContainer *containerView;

@georgejecook
Copy link
Author

But is there notification to say when it's added an ad? coz if not, I'd have no way of knowing it's done something silly with the sizing.

@larsacus
Copy link
Owner

What do you mean "added an ad"? You shouldn't have to listen for individual ads coming and going from the view as that's the entire point of this control and to abstract that information away from you.

@georgejecook
Copy link
Author

Ok, so is the bug fixed then? "It should automatically center itself horizontally internally -- if not, that should be a bug. You have access to the container view on LARSAdController to manipulate any way you please:"

It doesn't center horizontally if the ad is smaller than the screen width.

@larsacus
Copy link
Owner

Please attach some logs, screenshots, or any other information that could be used to determine if this actually is a "bug" or not before I can claim that it is fixed. Yes, it should currently center itself horizontally, as that is how I am currently seeing it on my device right now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants