Issue 6808 in pharo: Remove EncryptedMultiDisplayScanner and users

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

Issue 6808 in pharo: Remove EncryptedMultiDisplayScanner and users

pharo
Status: Accepted
Owner: [hidden email]
Labels: Type-Bug Milestone-2.0

New issue 6808 by [hidden email]: Remove  
EncryptedMultiDisplayScanner and users
http://code.google.com/p/pharo/issues/detail?id=6808

An EncryptedMultiDisplayScanner is a MultiDisplayScanner displaying
stars instead of characters (for password by example)

Wow... this is soooo overkill!

Look at the amount of extra code we have just to simply displaying
stupid stars ... ************

See the:
EncryptedTextMorphForFieldView>>drawOn:
...
aCanvas encryptedParagraph: self paragraph bounds: fauxBounds color: color.

ehh?.. Since when canvas (a tool for drawing graphic primitives and
text) needs to know anything like that?

I'm not sure if there could be better/simpler solution (considering
that we don't have a simple line editor and need to go over
overcomplicated text->composer->scanner->displayscanner chain to be
able to edit even single string..
but that's really, really stupid to penetrate multiple layers of
responsibility just to display stars instead of characters...

What if i would like to display '%' instead of chars, should i also go
and repeat same way??

We should get rid of this thing..

--
Best regards,
Igor Stasenko.



_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker
Reply | Threaded
Open this post in threaded view
|

Re: Issue 6808 in pharo: Remove EncryptedMultiDisplayScanner and users

pharo

Comment #1 on issue 6808 by [hidden email]: Remove  
EncryptedMultiDisplayScanner and users
http://code.google.com/p/pharo/issues/detail?id=6808

We should think what changes needed to preserve functionality, but avoid  
such mess.


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker
Reply | Threaded
Open this post in threaded view
|

Re: Issue 6808 in pharo: Remove EncryptedMultiDisplayScanner and users

pharo
Updates:
        Labels: -Milestone-2.0 Milestone-3.0

Comment #2 on issue 6808 by [hidden email]: Remove  
EncryptedMultiDisplayScanner and users
http://code.google.com/p/pharo/issues/detail?id=6808

(No comment was entered for this change.)


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker