clone 386408 -1 -2
retitle -1 xvnc4viewer: no keyboard focus in dialog windows
retitle -2 xvnc4viewer: allow command-line dialogs when started in console
severity -2 wishlist
thanks

Hello!

I'm retitleing the current bug because as I already explained the
problem is more general than the error dialog.  Moreover, during the
discussion it's showed up the wish for a complete console behavior
when vncviewer is started in console and I think this is a separated
(but connected) bug, thus I'm creating it.  Ola, I hope you don't
mind :-)

On Mon, 06 Nov 2006 17:39:58 +0100, Ola Lundqvist wrote:
> On Mon, Nov 06, 2006 at 05:18:11PM +0100, Luca Capello wrote:
>> On Thu, 07 Sep 2006 22:07:28 +0200, Ola Lundqvist wrote:
>> > Unfortunatly I do not know how to fix that. Patch is appriciated.
>> [...]
>> > On Thu, Sep 07, 2006 at 02:20:48PM +0300, Meelis Roos wrote:
>> >> The 'Authentication failed' messagebox should have keyboard
>> >> focus on the OK button so I could dismiss it with just a
>> >> keypress. Currently the focus is not on the button.
>> 
>> Well, the problem is not only with the error dialog, but also with
>> the IP dialog, i.e. when you start vncviewer with no options and it
>> asks for a VNC server to connect to.  In this window, TAB doesn't
>> work and you're obliged to have a working mouse.  Thus the title
>> should be changed to something like "xvnc4viewer: no keyboard focus
>> in dialog windows".

Mmm, I found that the PasswordDialog shares the problem, too.  I'll try
to find a solution for that: the focus should always be in the dialog
box and TAB should work, no matter if the mouse is available.

This is the first bug (ID -1).

>> IMHO I see two solutions:
>> 
>> 1) the best option will be to have all these dialogs similar to the
>>    password one, i.e. in the console (whose behavior I don't remember
>>    when it changed)
>
> I agree. It changed from vnc3 to vnc4.

Thank you, I didn't know that.

>> 2) while I prefer the console, I can understand someone could want
>>    the X dialogs, thus a command-line option could be the best
>>    solution
>
> I have made a patch so that the password dialog is on the command
> line. The same thing could be done for the other dialogs.

This is the second bug (ID -2).

OK, my fault: I didn't deeply checked the changelog.Debian.gz for
Debian-specific patches.  Anyway, I think that your work (and the
future one to resolve this bug) should be included upstream, but this
is obviously IMHO.

>> I'm not a programmer myself, so patches are a bit of difficult.
>> Even if I'm not promising anything, I'll try to find spare time to
>> dig into the problem.
>
> If you want to take a look at it, then you can look at the changes
> that I made in order to change the password prompt. Similar will be
> done for the other ones.

I'll preparing the patch, but be advised that this will probably take
a bit of time.

Thx, bye,
Gismo / Luca

Attachment: pgpbwDsTY6kdO.pgp
Description: PGP signature

Reply via email to