Skip to content
This repository has been archived by the owner on Aug 5, 2022. It is now read-only.

added additional superclass call #7

Closed
wants to merge 1 commit into from

Conversation

russ1985
Copy link

Not sure if you ran into this with Sencha Touch 2.4.* but I had to add an additional superclass call to get it to work

thanks,

@vadimpopa
Copy link
Owner

So what's the issue ? I've tried the fiddle https://fiddle.sencha.com/#fiddle/r9 and seems ok

@russ1985
Copy link
Author

Maybe it was just my setup, I am using Sencha Touch 2.4.1 and I have a field that is extending from your field. Maybe because I was extending the field I needed the additional superclass call.

@vadimpopa
Copy link
Owner

Right, if you extend it, then you need to call one more superClass. Because this is a component which is rarely extended, I will just add a comment in the code, don't want to write a loop and get the Ext.field.Text superclass which actually ends the updateValue. Thanks for raising this.

@vadimpopa vadimpopa closed this Jul 4, 2015
@russ1985
Copy link
Author

russ1985 commented Jul 4, 2015

Makes sense, thanks!

Sent via the Samsung Galaxy S™ III, an AT&T 4G LTE smartphone

-------- Original message --------
From: Vadim Popa [email protected]
Date:07/04/2015 16:09 (GMT-05:00)
To: "vadimpopa/Ux.field.Multiselect" [email protected]
Cc: Russell Holmes [email protected]
Subject: Re: [Ux.field.Multiselect] added additional superclass call (#7)
Right, if you extend it, then you need to call one more superClass. Because this is a component which is rarely extended, I will just add a comment in the code, don't want to write a loop and get the Ext.field.Text superclass which actually ends the updateValue. Thanks for raising this.


Reply to this email directly or view it on GitHub.

@waqarprojectinertia
Copy link

Hey
Does it work in extjs 6 modern toolkit? or any other plugin

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants