Opened on 09/10/2014 at 09:59:22 AM

Closed on 04/16/2018 at 10:59:18 AM

#1347 closed defect (invalid)

Inconsistent padding on "Whitelisted domains" options tab

Reported by: passbrains Assignee: dreyguy
Priority: P5 Milestone:
Module: User-Interface Keywords: goodfirstbug
Cc: greiner Blocked By:
Blocking: Platform: Unknown
Ready: yes Confidential: no
Tester: Unknown Verified working: no
Review URL(s):

Description (last modified by greiner)

Adapted from https://www.passbrains.com/dashboard/view-ticket.php?ticket_no=AOP-29

Environment

Tested on Chrome, Opera, Mac OS X 10.8 and Windows 7 64bit, likely effects all.
ABP version 1.8.3.1207

How to reproduce

  1. Open the Adblock Plus options page
  2. Navigate to the "Whitelisted domains" tab

Observed behaviour

There is no padding between the field to create a new exception and the "Add domain" button.

What to change

Apply the style box-sizing: border-box; to the elements input#newFilter and input#newWhitelistDomain.

Attachments (2)

1598_1409405836_abp_op1.png (115.3 KB) - added by passbrains on 09/10/2014 at 09:59:25 AM.
1598_1409405836_abp_op2.png (87.9 KB) - added by passbrains on 09/10/2014 at 09:59:25 AM.

Download all attachments as: .zip

Change History (9)

Changed on 09/10/2014 at 09:59:25 AM by passbrains

Changed on 09/10/2014 at 09:59:25 AM by passbrains

comment:1 Changed on 09/10/2014 at 10:06:22 AM by kzar

  • Component changed from Unknown to User-Interface
  • Description modified (diff)
  • Platform changed from Opera to Chrome
  • Summary changed from The distance from field to button is not always the same to Inconsistent padding on "Whitelisted domains" options tab

comment:2 Changed on 09/16/2014 at 09:52:28 AM by greiner

  • Priority changed from Unknown to P4

comment:3 Changed on 04/02/2015 at 04:17:18 PM by greiner

  • Cc greiner added
  • Description modified (diff)
  • Keywords goodfirstbug added
  • Platform changed from Chrome to Unknown
  • Ready set

comment:4 Changed on 11/04/2016 at 12:38:51 AM by dreyguy

First time contributor here... I can take a look at this.

comment:5 Changed on 01/24/2017 at 05:20:33 PM by greiner

  • Owner set to dreyguy
  • Tester set to Unknown

comment:6 Changed on 01/15/2018 at 06:36:48 PM by greiner

  • Priority changed from P4 to P5

This ticket will soon no longer be relevant since we're going to replace the entire options page. Therefore I'm decreasing its priority for now.

comment:7 Changed on 04/16/2018 at 10:59:18 AM by kzar

  • Resolution set to invalid
  • Status changed from new to closed

Closing this since it's about the old options page.

Add Comment

Modify Ticket

Change Properties
Action
as closed .
The resolution will be deleted. Next status will be 'reopened'.
to The owner will be changed from dreyguy.
 
Note: See TracTickets for help on using tickets.