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

Fixed: CPWindow keeps a reference of its firstResponder when closing a window #2229

Merged
merged 1 commit into from Oct 23, 2014

Conversation

Dogild
Copy link
Member

@Dogild Dogild commented Oct 23, 2014

Previously, when closing a CPWindow, the window kept a reference of the firstResponder. This things raised an issue when opening a window again and asking the window to make the first responder to the same element as the one we had.

Now when closing a window, the window will update its firstResponder to nil.

…a window

Previously, when closing a CPWindow, the window kept a reference of the firstResponder. This things raised an issue when opening a window again and asking the window to make the first responder to the same element as the one we had.

Now when closing a window, the window will update its firstResponder to nil.
@cappbot cappbot added this to the Someday milestone Oct 23, 2014
@cappbot cappbot added the #new label Oct 23, 2014
@cappbot
Copy link

cappbot commented Oct 23, 2014

Milestone: Someday. Label: #new. What's next? A reviewer should examine this issue.

primalmotion added a commit that referenced this pull request Oct 23, 2014
Fixed: CPWindow keeps a reference of its firstResponder when closing a window
@primalmotion primalmotion merged commit 2e3c658 into cappuccino:master Oct 23, 2014
@primalmotion
Copy link
Member

Thanks,

+#fixed

@cappbot
Copy link

cappbot commented Oct 23, 2014

Milestone: Someday. Label: #fixed. What's next? This issue is considered successfully resolved.

@cappbot cappbot added #fixed and removed #new labels Oct 23, 2014
@Dogild Dogild deleted the WindowCloseFirstResponder branch October 25, 2014 06:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants