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 GLViewImpl::getMonitorSize() for desktop platforms #18141

Merged
merged 1 commit into from
Aug 22, 2017

Conversation

MrCapone
Copy link
Contributor

When application was created windowed glfwGetWindowMonitor may return NULL. So return of getMonitorSize() will be ZERO. glfwGetPrimaryMonitor was added on glfw 3.0 so you can use it.

@dumganhar
Copy link

@MrCapone
Copy link
Contributor Author

@minggo
Copy link
Contributor

minggo commented Aug 22, 2017

When application was created windowed glfwGetWindowMonitor may return NULL

What does application was created windowed mean? Did you mean not full screen mode?

@MrCapone
Copy link
Contributor Author

I mean GLViewImpl::createWithRect

@minggo minggo added this to the 3.16 milestone Aug 22, 2017
@minggo minggo merged commit 6e013b5 into cocos2d:v3 Aug 22, 2017
stevetranby added a commit to stevetranby/cocos2d-x that referenced this pull request Aug 23, 2017
* commit 'e79f21cec02ebbffbf08150708ed26e524c1faf2':
  add new CMakeLists.txt to cpp-template-default (https://gist.github.com/MrCapone/ccdc1826cb7362086107e8096bfe6f90) (cocos2d#18167)
  Fixes compilation errors for android audio module while building with x86 arch. (cocos2d#18150)
  fixed GLViewImpl::getMonitorSize() for desktop platforms (cocos2d#18141)
  [ci skip][AUTO]: updating luabinding & jsbinding & cocos_file.json automatically (cocos2d#18183)
  Fix android problem (cocos2d#18142)
  fix for ios audio engine (cocos2d#18146)
  fix android problem with 24 bit depth buffer on some devices (cocos2d#18148)
  Blend function for UIImageView (cocos2d#18149)
  Fix alpha blending during the animation of TransitionCrossFade. (cocos2d#18175)
  fixed cocos2d#17989: touch stops responding on android devices. (cocos2d#18165)
  [ci skip]update version and readme (cocos2d#18169)
  fix FontAtlas crash on some Android devices (cocos2d#18145)
  Add game-controller-test for win32 (cocos2d#18164)
  Fix android compile fail bug (cocos2d#18166)

# Conflicts:
#	cocos/audio/android/cutils/bitops.h
#	cocos/audio/apple/AudioEngine-inl.mm
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

3 participants