Opened 6 years ago

Closed 6 years ago

#10168 closed bug (fixed)

Esc keypress to close WebPositive's find bar hits 'next' first

Reported by: Kev Owned by: pulkomandy
Priority: normal Milestone: R1
Component: Applications/WebPositive Version: R1/Development
Keywords: Cc: degea@…
Blocked By: Blocking:
Has a Patch: yes Platform: All

Description

(On hrev46295, although it's been around for a while but I just thought either Web+ was being randomly flaky, or I was doing something wrong, or maybe had a hardware issue--didn't realize exact behaviour until today.)

  1. Open a web page in Web+
  2. Hit Alt+F
  3. Type something that appears in the page more than once
  4. Hit Enter, it highlights one of them
  5. Hit Escape to close the find bar: it does close properly, but first jumps to the next occurrance as if you had clicked Next before hitting Escape

I'm in the habit of closing the find bar after I find what I'm looking for, so this was a bit annoying the first few times it happened, then I just started leaving it open. Only today I forgot to do that, and then noticed what was actually happening. :)

Attachments (1)

0001-Remove-queued-messages-on-escape.patch (1.3 KB ) - added by jscipione 6 years ago.
(untested) remove any queued message after processing escape

Download all attachments as: .zip

Change History (5)

comment:1 by ttcoder, 6 years ago

Cc: degea@… added

FWIW, this bug gets triggered both if closing with <Escape> or if clicking the close button at top-left of the find bar.

by jscipione, 6 years ago

(untested) remove any queued message after processing escape

comment:2 by jscipione, 6 years ago

Has a Patch: set

comment:3 by pulkomandy, 6 years ago

Owner: changed from leavengood to pulkomandy
Status: newassigned

This also happens to me when trying to click one of the search results which also happens to be a link. I doubt hacking hte message queue is the way to go, we should instead understand why this is triggered at all.

comment:4 by pulkomandy, 6 years ago

Resolution: fixed
Status: assignedclosed

Fixed in hrev46696.

Note: See TracTickets for help on using tickets.