Re: PATCH: Enable backwards search in SQL Box

Hello Dave,

Wednesday, February 26, 2014, 12:56:51 AM, you wrote:

DP> Shift-F3 should always do the opposite of what the radio
DP> buttons are set to. So, by default with the Forward radio button
DP> selected, Shift-F3 will search backwards, whilst if the Backwards
DP> radio button is selected, Shift-F3 should search forwards.

You are right. For example, SciTE does the same.

According to the above implemented it in a simplier way.
Please see the patch.


By the way, I was puzzled why search dialog's settings are saved
sometimes and sometimes are not. Depending on dialog's close method:
1. Pressing ESC or "Close" form button calls OnCancel() which does not
save;
2. Pressing title [x] button or Alt-F4 (or similar OS hotkey) calls
OnClose() which does save.

Shouldn't the behaviour be the same regardless of dialog's close
method? Or at least the difference should be noted in documentation.


Wednesday, February 26, 2014, 12:56:51 AM, you wrote:

DP> On Tue, Feb 25, 2014 at 9:38 AM, Akshay Joshi
DP> <akshay.joshi@enterprisedb.com> wrote:
DP> Hi Dave═


DP> Ashesh asked me to review the patch. I have reviewed the
DP> patch and tested it. Found one behavior which gets change after
DP> applying the patch.


DP> Behavior Before:- When search direction is forward and user
DP> press F3, it searches the string in forward direction. Similarly
DP> if search direction is backward and user press F3, it searches the
DP> string in the backward direction. Nothing will be happen when user
DP> press═Shift-F3.═


DP> Behavior After :- ═When user press F3, it searches the string
DP> in forward direction while on pressing Shift-F3, it searches the
DP> string in backward direction. It is not dependent on the search
DP> direction selected from the dialog. Also the selection of radio
DP> button is dependent on the key press, if F3 is pressed then
DP> forward button is selected and if Shift-F3 is pressed then
DP> backward is selected.═


DP> Is this behavior acceptable?═





DP> No, I don't think so (if I'm understanding correctly). I
DP> believe what should happen is that:


DP> - F3 should honour the search direction on the radio buttons,
DP> in all cases. If the Forward button is selected, F3 searches
DP> forward, and if the Backwards button is selected, F3 searches
DP> backwards.


DP> - Shift-F3 should always do the opposite of what the radio
DP> buttons are set to. So, by default with the Forward radio button
DP> selected, Shift-F3 will search backwards, whilst if the Backwards
DP> radio button is selected, Shift-F3 should search forwards.


DP> The reason for this is that the radio buttons give the most
DP> obvious visual cue, so F3 (Find Next) should follow what they are
DP> set to.
DP> ═


DP> On Thu, Feb 20, 2014 at 5:09 PM, Ashesh Vashi
DP> <ashesh.vashi@enterprisedb.com> wrote:
DP> Sure.
DP> I will do that.



DP> On Thu, Feb 20, 2014 at 4:52 PM, Dave Page <dpage@pgadmin.org> wrote:
DP> Ashesh, can you look at, and if appropriate, commit this please?

DP> Thanks.

DP> On Mon, Feb 17, 2014 at 5:46 PM, J.F. Oster <jinfroster@mail.ru> wrote:
>> Hello!
>>
>> Pressing F3 in query editor makes the previously started search
>> continue forwards.
>> This patch adds expected behavior for Shift-F3.
>>
>> There is a minor inconvenience when just changing the search direction
>> - you have to press (Shift-)F3 *twice* for the (previous)next occurence
>> to get highlighted. That's because first time goes for cursor movement
>> to the corresponding boundary of current occurence. But that also
>> happens when search direction is changed with dialog radio-buttons, so
>> I think it's ok.
>>


--
Best regards,
 J.F.