[bug][7.7] Readonly password field DNU

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
3 messages Options
Reply | Threaded
Open this post in threaded view
|

[bug][7.7] Readonly password field DNU

Joerg Beekmann, DeepCove Labs (YVR)

In VW 7.7 if a password field is installed as read only there will be a DNU because initializeForPassword is not understood by ReadOnlyTextEditorController.

 

Admittedly a read only password field is a bit of a strange beast but I’d argue it should not fail in this way and it didn’t in 7.6. Adding initializeForPassword as a no-op avoids the DNU.

 

 

-----
Joerg Beekmann
DeepCove Labs
4th floor 595 Howe Street
Vancouver, BC, V6C 2T5
voice +1.604.689.0322
fax   +1.604.689.0311
<a href="blocked::mailto:joerg@deepcovelabs.com" title="blocked::mailto:joerg@deepcovelabs.com">joerg@...


CONFIDENTIALITY NOTICE - This email contains private and confidential
information. If it is not intended for you, delete it and any attachments.

 

 


_______________________________________________
vwnc mailing list
[hidden email]
http://lists.cs.uiuc.edu/mailman/listinfo/vwnc
Reply | Threaded
Open this post in threaded view
|

Re: [bug][7.7] Readonly password field DNU

Holger Kleinsorgen-4
Am 29.04.2010 21:29, schrieb Joerg Beekmann, DeepCove Labs (YVR):
> In VW 7.7 if a password field is installed as read only there will be a
> DNU because initializeForPassword is not understood by
> ReadOnlyTextEditorController.
>
> Admittedly a read only password field is a bit of a strange beast but
> I’d argue it should not fail in this way and it didn’t in 7.6. Adding
> initializeForPassword as a no-op avoids the DNU.

Implementing initializeForPassword as a no-op allows copy/cut of the
passord.

suggestion:

initializeForPassword
   self menuHolder: (ValueHolder with: Menu new).
   dispatchTable := self getDispatchTable.
   dispatchTable bindValue: #ignoreInputKey: to: Ctrlx modifiers:
#(#control).
   dispatchTable bindValue: #ignoreInputKey: to: Ctrlc modifiers:
#(#control).
_______________________________________________
vwnc mailing list
[hidden email]
http://lists.cs.uiuc.edu/mailman/listinfo/vwnc
Reply | Threaded
Open this post in threaded view
|

Re: [bug][7.7] Readonly password field DNU

Joerg Beekmann, DeepCove Labs (YVR)
Thank you I should have looked at this more closely. Doing as you
suggest works perfectly.

Joerg

-----Original Message-----
From: [hidden email] [mailto:[hidden email]] On
Behalf Of Holger Kleinsorgen
Sent: Friday, April 30, 2010 4:43 AM
To: [hidden email]
Subject: Re: [vwnc] [bug][7.7] Readonly password field DNU

Am 29.04.2010 21:29, schrieb Joerg Beekmann, DeepCove Labs (YVR):
> In VW 7.7 if a password field is installed as read only there will be
a
> DNU because initializeForPassword is not understood by
> ReadOnlyTextEditorController.
>
> Admittedly a read only password field is a bit of a strange beast but
> I'd argue it should not fail in this way and it didn't in 7.6. Adding
> initializeForPassword as a no-op avoids the DNU.

Implementing initializeForPassword as a no-op allows copy/cut of the
passord.

suggestion:

initializeForPassword
   self menuHolder: (ValueHolder with: Menu new).
   dispatchTable := self getDispatchTable.
   dispatchTable bindValue: #ignoreInputKey: to: Ctrlx modifiers:
#(#control).
   dispatchTable bindValue: #ignoreInputKey: to: Ctrlc modifiers:
#(#control).
_______________________________________________
vwnc mailing list
[hidden email]
http://lists.cs.uiuc.edu/mailman/listinfo/vwnc

_______________________________________________
vwnc mailing list
[hidden email]
http://lists.cs.uiuc.edu/mailman/listinfo/vwnc