Opened on 03/03/2016 at 01:22:23 PM
Closed on 03/03/2016 at 05:45:28 PM
Last modified on 03/07/2016 at 02:42:59 PM
#3729 closed defect (fixed)
Default size of Add filter popup window hides buttons / requires scrollbars
| Reported by: | Ross | Assignee: | sebastian |
|---|---|---|---|
| Priority: | P2 | Milestone: | Adblock-Plus-1.11-for-Chrome-Opera-Safari |
| Module: | Unknown | Keywords: | |
| Cc: | Blocked By: | ||
| Blocking: | Platform: | Unknown / Cross platform | |
| Ready: | yes | Confidential: | no |
| Tester: | Ross | Verified working: | yes |
| Review URL(s): | |||
Description
Environment
ABP 1.10.2.1558
Reproduced in:
- Chrome 30 / 48 / Windows 7 x86
- Opera 23 / Windows 7 x86
- Opera 35 / Ubuntu 14.04 x64
- Opera 19 / OS X 10.11
Looks fine / Not reproduced in:
- Chrome 30 / 48 / Ubuntu 14.04 x64
How to reproduce
- Navigate to BBC News (or other site).
- Select the [ABP Icon] > [Block element].
- Select an element on the page or the BBC logo.
- Observe popup window for adding filters is too small / displays scrollbars.
Observed behaviour
In Chrome: The default size of the popup seems too small on Windows (but not Ubuntu) and displays scrollbars.
In Opera: The default size of the popup is too small *and* the Opera address bar / ABP is included at the top.
See attached images.
Expected behaviour
Popup to be sized correctly / not include browser address bar (as attached Chrome-Ubuntu.png image)
Attachments (5)
Change History (10)
Changed on 03/03/2016 at 01:23:33 PM by Ross
Changed on 03/03/2016 at 01:24:19 PM by Ross
Changed on 03/03/2016 at 01:24:46 PM by Ross
Changed on 03/03/2016 at 01:25:17 PM by Ross
Changed on 03/03/2016 at 01:26:06 PM by Ross
comment:1 Changed on 03/03/2016 at 02:02:40 PM by sebastian
- Owner set to sebastian
- Ready set
comment:2 Changed on 03/03/2016 at 02:13:38 PM by sebastian
- Review URL(s) modified (diff)
- Status changed from new to reviewing
comment:3 Changed on 03/03/2016 at 05:42:57 PM by abpbot
A commit referencing this issue has landed:
https://hg.adblockplus.org/adblockpluschrome/rev/937dc9577ad9
comment:4 Changed on 03/03/2016 at 05:45:28 PM by sebastian
- Resolution set to fixed
- Status changed from reviewing to closed
comment:5 Changed on 03/07/2016 at 10:40:40 AM by Ross
- Verified working set
Fixed. Seems to works well across browser platforms/versions.
ABP 1.10.2.1559
Chrome 30 / Windows 7 x86
Chrome 47 / Windows 7 x86
Chrome 38 / OS X 10.11 x64
Chrome 48 / Ubuntu 14.04 x64
Opera 19 / Windows 7 x86
Opera 35 / Windows 7 x86
Opera 25 / OS X 10.11 x64
Opera 35 / Ubuntu 14.04 x64
Safari 6 (OS X 10.8) / Safari 8 (OS X 10.10) / Safari 9 (OS X 10.11)
It seems that problem is that the size given to chrome.windows.create() specifies the size of the whole window, including decoration and address bar if any. And since the size of the window decoration varies across different environments, the content might or might not fit. BTW, Opera showing an address bar in the first place seems to be a bug on their end. Anyway, We can simply make the size of the text field variable, automatically consuming as much space as there is left. So that we will never get scrollbars, and even have a decent user experience when the the window is resized by the user.