Opened on 04/04/2014 at 08:40:01 AM

Closed on 05/09/2014 at 06:05:34 AM

Last modified on 05/09/2014 at 06:14:20 AM

#256 closed defect (fixed)

Home/End keys are broken when editing a filter

Reported by: arthur Assignee: trev
Priority: P3 Milestone: Adblock-Plus-2.6.1-for-Firefox
Module: Adblock-Plus-for-Firefox Keywords:
Cc: smultron45@gmail.com, trev Blocked By:
Blocking: Platform:
Ready: yes Confidential: no
Tester: Verified working: no
Review URL(s):

http://codereview.adblockplus.org/4834220086132736/

Description (last modified by arthur)

Environment

Latest Nightly (works on Firefox 28)
Adblock Plus 2.5.1.3788
Windows 8.1 Pro x64

How to reproduce

  1. Open filter preferences
  2. Edit a custom filter
  3. Press the home and end key

Observed behaviour

Nothing happens.

Expected behaviour

The cursor should jump in front of and at the end of the filter.

Forum topic

https://adblockplus.org/forum/viewtopic.php?f=11&t=21597

Attachments (0)

Change History (8)

comment:1 Changed on 04/04/2014 at 08:41:04 AM by arthur

  • Description modified (diff)

comment:2 Changed on 04/04/2014 at 09:22:45 AM by mapx

  • Cc smultron45@gmail.com added

comment:3 Changed on 05/08/2014 at 02:05:48 PM by trev

  • Cc trev added

I can reproduce this - works fine in Firefox 29, broken in the current nightly.

comment:4 Changed on 05/08/2014 at 02:11:46 PM by trev

This seems to be a regression from https://bugzilla.mozilla.org/show_bug.cgi?id=950373 (http://hg.mozilla.org/mozilla-central/rev/b57f9be43a50 in particular), it affects Up and Down keys as well.

comment:5 Changed on 05/08/2014 at 02:34:23 PM by trev

  • Keywords externaldependency added

Filed https://bugzilla.mozilla.org/show_bug.cgi?id=1007680 on this. Unfortunately, the way the tree widget works I don't see what we can do on our end here.

comment:6 Changed on 05/08/2014 at 07:00:15 PM by trev

  • Keywords externaldependency removed
  • Review URL(s) modified (diff)
  • Status changed from new to reviewing

Ouch, the issue is really https://bugzilla.mozilla.org/show_bug.cgi?id=777832 here and we are already working around it - but only for the keypress event, not keydown.

comment:7 Changed on 05/09/2014 at 06:05:34 AM by trev

  • Resolution set to fixed
  • Status changed from reviewing to closed

comment:8 Changed on 05/09/2014 at 06:14:20 AM by trev

  • Milestone set to Adblock-Plus-for-Firefox-next
  • Owner set to trev

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 trev.
 
Note: See TracTickets for help on using tickets.