_CPAttachedWindow is not defined #1634

Closed
ggsato opened this Issue Jul 26, 2012 · 10 comments

Projects

None yet

6 participants

@ggsato
Contributor
ggsato commented Jul 26, 2012

CPAlert window running as a modal(by runModal in my case) produces an error as you move your mouse over it.

This is the error message produced at line 1193.

_CPAttachedWindow is not defined

It seemed brought by this commit, Fixed: popovers in modal windows couldn't be interacted with.

But this happens only with jake deployed build.

And simply adding the following import to my app solves this.

@import <AppKit/_CPAttachedWindow.j>

Perhaps, something is left undone?

cappbot commented Jul 26, 2012

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

Contributor

Could you please verify that this is still a problem and, if possible, write a small application that exhibits this problem?

+#accepted
+#needs-reduction

cappbot commented Feb 8, 2013

Milestone: Someday. Labels: #accepted, #needs-reduction. What's next? A minimal test app should be created which demonstrates the concern of this issue in isolation.

Contributor

Could you please verify that this is still a problem and, if possible, write a small application that exhibits this problem?

That's your job as reviewer. :-)

-#accepted

Owner

Well, it's still up to the submitter of an issue to make their case that the issue really is an issue. We can't be writing test cases for them all the time. If one of our reviewers or core devs want to, great, but I think the expectation should be that the issue submitter builds the case.

In this case the question is moot though since the issue submitter also happens to be a Cappuccino Reviewer. :)

cappbot commented Feb 8, 2013

Milestone: Someday. Label: #needs-reduction. What's next? A minimal test app should be created which demonstrates the concern of this issue in isolation.

Contributor

I didn't think it had enough information on the ticket to make a reduction on my own, and with the re-working of all the imports after Obj2, I suspect it's no longer a problem.

Contributor

Its no longer an issue since there is no longer a _CPAttachedWindow...

Contributor

Too bad there isn't a "#can't-fix-because-its-outdated" tag... :) Oh well. Marking it fixed instead.

+#fixed

cappbot commented Feb 9, 2013

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

@cappbot cappbot closed this Feb 9, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment