#190 CPTextField's 'controlTextDidChange:' delegate doesn't work - Cappuccino

Type	To find
responsible:me	tickets assigned to you
tagged:"@high"	tickets tagged @high
milestone:next	tickets in the upcoming milestone
state:invalid	tickets with the state invalid
created:"last week"	tickets created last week
sort:number, importance, updated	tickets sorted by #, importance or updated
Combine keywords for powerful searching.
Use advanced searching »

#190 open

CPTextField's 'controlTextDidChange:' delegate doesn't work

Reported by Jakub Pawlowicz | December 31st, 2008 @ 02:21 AM | in 0.7

Looks like providing controlTextDidChange: method by CPTextField's delegate doesn't work as expected as delegation is not being sent at all.

It is caused by CPTextField not updating the current value (stringValue) while editing. Once fixed (see attachment) it started working but required another fix as then the control's static text were shown under the input field.

I've proposed a solution to this (hiding static text field while editing) but somebody may come up with a better approach.

cptextfield-didchange-fix.patch 1.2 KB

Comments and changes to this ticket

You flagged this item as spam.
Jakub Pawlowicz December 31st, 2008 @ 02:28 AM
Adding a test case. Updating the CPTextField's stringValue while typing can be seen via Firebug/Webkit debug console.
- AppController.j 1.1 KB
You flagged this item as spam.
ronin-756 (at lighthouseapp) January 31st, 2009 @ 01:29 AM
- Tag changed from “appkit, cptextfield” to “appkit, cptextfield, patch”
I've been able to test this patch and it works as expected.
You flagged this item as spam.
ronin-756 (at lighthouseapp) April 3rd, 2009 @ 08:21 AM
- Tag changed from “appkit, cptextfield, patch” to “@to-review, appkit, cptextfield, patch”
Here is a proposed fix for 0.7b : http://github.com/suitmymind/cap...

(it also propose a solution for #260)
Ross Boucher April 4th, 2009 @ 01:02 PM
- Milestone set to “0.7”
- Assigned user changed from “admin (at 280north)” to “Francisco Tolmasky”
- State changed from “new” to “open”
Giving this to Francisco, since he just changed a bunch of textfield stuff.

Please Sign in or create a free account to add a new ticket.

With your very own profile, you can contribute to projects, track your activity, watch tickets, receive and update tickets through your email and much more.