Opened on 07/23/2014 at 03:16:47 PM
Closed on 12/14/2016 at 12:45:46 PM
Last modified on 03/14/2017 at 02:21:18 PM
#1121 closed defect (fixed)
Long filter texts get cut off in options page
| Reported by: | passbrains | Assignee: | wspee |
|---|---|---|---|
| Priority: | P4 | Milestone: | Adblock-Plus-1.13-for-Chrome-Opera |
| Module: | User-Interface | Keywords: | goodfirstbug |
| Cc: | mapx, sven | Blocked By: | |
| Blocking: | Platform: | Chrome | |
| Ready: | yes | Confidential: | no |
| Tester: | Ross | Verified working: | yes |
| Review URL(s): | |||
Description (last modified by greiner)
Adapted from https://www.passbrains.com/dashboard/view-ticket.php?ticket_no=ACH-12
Environment
Windows 7 64bit Chrome English
ABP version 1.8.3.1204
How to reproduce
- Add the following custom filter
guasgdfigifiuseuisfiuguiefasiulefsuifsefiausefiusefiuasieufasuiefasefialisuefisufalsiefsefhsidfhsoidfgsdf
Observed behaviour
The filter text gets cut off
Expected behaviour
The full filter text should be shown in the custom filter list
What to change
Add the style overflow: auto to the options page element select#userFiltersBox.
Attachments (1)
Change History (11)
Changed on 07/23/2014 at 03:16:49 PM by passbrains
comment:2 Changed on 08/27/2014 at 07:03:34 PM by sebastian
- Cc sven added
- Component changed from Unknown to User-Interface
comment:3 Changed on 09/05/2014 at 03:18:38 PM by greiner
- Priority changed from Unknown to P4
We might be able to use a horizontal scrollbar on the element for that but styling <option> tags is tricky so we may even need to create a custom element for that. Therefore I'm setting the priority to P4.
comment:4 Changed on 04/16/2015 at 10:36:20 AM by greiner
- Description modified (diff)
- Ready set
- Summary changed from Text gets cut off if the path of the blocked element is too long to Long filter texts get cut off in options page
comment:5 Changed on 04/16/2015 at 10:39:47 AM by greiner
- Keywords goodfirstbug added
comment:6 Changed on 12/07/2016 at 09:55:06 AM by wspee
- Owner set to wspee
- Tester set to Unknown
comment:7 Changed on 12/07/2016 at 09:59:31 AM by wspee
- Review URL(s) modified (diff)
- Status changed from new to reviewing
comment:8 Changed on 12/14/2016 at 11:21:23 AM by abpbot
A commit referencing this issue has landed:
Issue 1121 - Don't cut off long filters in userFiltersBox on options page.
comment:9 Changed on 12/14/2016 at 12:45:46 PM by wspee
- Milestone set to Adblock-Plus-for-Chrome-Opera-next
- Resolution set to fixed
- Status changed from reviewing to closed
comment:10 Changed on 03/14/2017 at 02:21:18 PM by Ross
- Tester changed from Unknown to Ross
- Verified working set
Done. But doesn't appear in Safari/FF WebEx installs (opened as #4987).
ABP 1.12.4.1739
Chrome 49 / 56 / Windows 10
Chrome 56 / OS X 10.12
Chrome 56 / Ubuntu 16.04
Opera 37 / 41 / Windows 7
Safari 10 / OS X 10.12
I agree that the UI isn't great. However IMO, wrapping long filters into multiple linesm, would make it even harder to use with the current design. I hope sven comes up with a smart way to edit filters with the new UI.