-
Notifications
You must be signed in to change notification settings - Fork 7k
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 #4163: add cceditbox to winrt #5805
Conversation
Sorry I have not got to this review yet. I am close to finishing cocos2d-x 3.0 for wp8-xaml. I will review this early next week. Thanks, Dale From: Jingchen [mailto:notifications@github.com] @stammenhttps://github.com/stammen @dumganharhttps://github.com/dumganhar You can merge this Pull Request by running git pull https://github.com/koowolf/cocos2d-x CCEditBox_winrt Or view, comment on, or merge it at: Commit Summary
File Changes
Patch Links:
— |
@stammen no problem,Take your time -- there is no rush. |
This Pr could not be automatically merged. |
…EditBox_winrt Conflicts: extensions/proj.winrt/libExtensions.vcxproj.filters samples/Cpp/TestCpp/proj.winrt/TestCpp.vcxproj.filters
can auto merged now. |
anyone have any idea when you will do the merge |
@stammen can you please help review? then we can start to merge it. Thanks. |
Good job on the code! The code builds and runs in the cocos2d-winrt.vc2012.sln project but not the cocos2d-winrt.vc2013.sln winrt project. Please fix and test the VS2013 project by adding the missing files. Also, please verify that ARM versions build correctly. |
closed #4163: add cceditbox to winrt
closed cocos2d#4163: add cceditbox to winrt
closed cocos2d#4163: add cceditbox to winrt
closed cocos2d#4163: add cceditbox to winrt
@stammen @dumganhar
please review it.