Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add method getSubcontrolId returning the id used in _createChildControlImpl #9140

Merged
merged 3 commits into from
Sep 1, 2016
Merged

Add method getSubcontrolId returning the id used in _createChildControlImpl #9140

merged 3 commits into from
Sep 1, 2016

Conversation

level420
Copy link
Member

This allows, e.g. while listening to the createChildControl event, retrieval of the used id during child control creation.

@@ -3687,6 +3687,19 @@ qx.Class.define("qx.ui.core.Widget",
},


/**
* Return the ID of the current widget if it is a created as a
Copy link
Member

@oetiker oetiker Aug 29, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If the widget has been created as a ChildControl, this will return the name of the ChildControl.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes. But the name is called id in e.g. _createChildControlImpl https://github.com/qooxdoo/qooxdoo/blob/master/framework/source/class/qx/ui/core/Widget.js#L3629
So I thought it's the right naming

Copy link
Member

@oetiker oetiker Aug 29, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK, then this:

If the widget has been created as a ChildControl, return the ID (name) of the ChildControl.

the problem is that your current sentence does not really work ... it is missing words.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK then It's my bad english. Will try to do it better.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.005%) to 55.735% when pulling 6fd3bfa on level420:level420-subcontrol-id into 4064436 on qooxdoo:master.

@oetiker
Copy link
Member

oetiker commented Aug 29, 2016

much better

@oetiker oetiker added the VOTING label Aug 29, 2016
@oetiker
Copy link
Member

oetiker commented Aug 29, 2016

☑️ CALL FOR VOTES

@coveralls
Copy link

Coverage Status

Coverage increased (+0.03%) to 55.764% when pulling ae728f6 on level420:level420-subcontrol-id into 4064436 on qooxdoo:master.

@voger
Copy link
Contributor

voger commented Aug 29, 2016

How do I vote? LGTM

@level420
Copy link
Member Author

@voger Sorry, voting is only possible for the core-team members.

@voger
Copy link
Contributor

voger commented Aug 29, 2016

Oh I didn't know that. Still looks good. :)

@oetiker oetiker merged commit f74ba77 into qooxdoo:master Sep 1, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants