Needs review
Project:
SimpleAds Overlay
Version:
7.x-1.x-dev
Component:
Miscellaneous
Priority:
Major
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
9 Oct 2014 at 13:41 UTC
Updated:
12 May 2015 at 21:31 UTC
Jump to comment: Most recent
Comments
Comment #1
JonathanATC commentedComment #2
daften commentedI have the same thing, but only for image type ads
Comment #3
liquidcms commentedComment #4
daften commentedChanging the priority without explanation is not done IMO, especially if the module just doesn't work properly in some use cases.
Comment #5
pmusaraj commentedYes, you are correct, daften, the image ads do not work, and this is a major issue. I will look for a solution over the next little while. (Patches are welcome!)
Resetting priority to major, as a compromise.
Comment #6
pmusaraj commentedThis issue is now fixed in the -dev branch.
Comment #7
daften commentedThat was quick, thanks. I'll check as soon as I find the time :)
Comment #8
liquidcms commented@daften: fyi... Critical issues are typically reserved for issue which cause your site to fail. The fact that img ads don't work is hardly critical (not even major).
Comment #9
daften commentedThanks both.
Tested it quickly, and it seems to work, but the overlay ad is not shown in an overlay for me.
Comment #10
liquidcms commentedwhat is working then? any screenshots of output or config?
Comment #11
daften commentedIt just shows the figure on the page, not in an overlay.