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

[CLOSED] Dark Mac #247

Open
core-ai-bot opened this issue Aug 17, 2021 · 3 comments
Open

[CLOSED] Dark Mac #247

core-ai-bot opened this issue Aug 17, 2021 · 3 comments

Comments

@core-ai-bot
Copy link
Member

Issue by JeffryBooher
Tuesday Nov 12, 2013 at 20:23 GMT
Originally opened as adobe#380


FIXED

UTR

BOLO

  • Intermittently going full screen will produce a blank screen. I thought this was fixed with a few cocoa calls to unhide and activate the app before going full screen. If you click on the brackets dock icon the appear will appear after going full screen. At that point it is no longer a problem. But the problem is intermittent. I had 10-15 successful launches to full screen without this problem so hopefully the additional NSView setNeedsDisplay:YES call after entering full screen mode fixed it but it's hard to say. I'd say since there is a work-around then let's roll with it and, if it's reproducible, then file a bug and we'll address it in the usual way. I don't think it's a show stopper unless it reproduces with great frequency.

JeffryBooher included the following code: https://github.com/adobe/brackets-shell/pull/380/commits

@core-ai-bot
Copy link
Member Author

Comment by jasonsanjose
Wednesday Nov 13, 2013 at 02:17 GMT


I did a high level review, but I can't say much about the Cocoa usage.

@core-ai-bot
Copy link
Member Author

Comment by bobeast
Wednesday Nov 13, 2013 at 19:04 GMT


Once the dust settles, I highly recommend refactoring such that each class gets its own file pair, as opposed to placing multiple classes in a single file.

@core-ai-bot
Copy link
Member Author

Comment by redmunds
Thursday Nov 14, 2013 at 04:50 GMT


Merging.

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

No branches or pull requests

1 participant