Opened 4 years ago

Last modified 2 years ago

#3941 closed change

Set exact window size of Firefox window. — at Version 2

Reported by: sergz Assignee:
Priority: Unknown Milestone:
Module: Extensions-for-Adblock-Plus Keywords: abpcrawler
Cc: trev, TobiasHilleke, tschuster Blocked By:
Blocking: Platform: Unknown / Cross platform
Ready: yes Confidential: no
Tester: Unknown Verified working: no
Review URL(s):

Description (last modified by sergz)

Background

acceptable ads has a criterion regarding the size of ad, in particular what portion of visible space it occupies. To stay consistent in terms of the size of screenshots among runs of abpcrawler it would be good to set a window size according to the screen size mentioned in acceptable ads.

I guess, we may neglect the fact that at least fonts are differently rendered on different machines or after updates, the important thing is to distinguish between e.g. 3840x2160 and 1368x768 configurations.

What to change

Call window.resizeTo(1368, 768); at the beginning of run function in crawler.js. Ideally, there should be a possibility to configure the actual parameter values through command line.

Change History (2)

comment:1 Changed 4 years ago by trev

I'd prefer to have this configurable rather than hardcoded. We can have a command line parameter defaulting to 1368x768. And we should start thinking about proper configuration files...

comment:2 Changed 4 years ago by sergz

  • Description modified (diff)

I also thought about it, however decided to don't mention it in this issue so far because if we decide to test it on different resolutions then according to https://adblockplus.org/acceptable-ads we should rather consider a possibility to test on or at least better emulate different platforms (mobile, tablet). Anyway, I have added it into the description.

Note: See TracTickets for help on using tickets.