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

added single view application and SpriteKit game template #9

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

Wh0ba
Copy link

@Wh0ba Wh0ba commented Sep 22, 2018

Added a template for a single view application which contains a simple UIViewController with a white background
And a SpriteKit template for making games using theos (I've made some games using it ) contains two scnes one for the menu and one for the game

The are both to easy to understand and work with

Hopefully this helps people who want to make apps but doesn't have XCode to do so
I've made a full functionality apps using these templates

@Wh0ba
Copy link
Author

Wh0ba commented Jun 12, 2019

I have updated the code with the changes that you requested

@demhademha
Copy link

demhademha commented Aug 21, 2020

Hi, what happened to this pull request, will it ever be merged?
I say this cause because these look like some good templates.
Thank you

@PoomSmart
Copy link
Contributor

The overall code formatting looks off. Check the other templates so that lots of unnecessary empty lines can be removed and the indentation can be consistent.

Copy link
Member

@uroboro uroboro left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I apologise for taking this long to properly review this PR

Please remove unnecessary empty lines from control files, Makefiles and source files. Rename "menuScene" to "MenuScene" and "myScene" to "MainScene".

What's the origin of the App icon images? Are they public domain or similar attribution?

</array>
<key>UIRequiredDeviceCapabilities</key>
<array>
<string>armv7</string>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You might want to add arm64 to the list

include @@THEOS@@/makefiles/common.mk

APPLICATION_NAME = @@PROJECTNAME@@

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove empty lines 4, 6, 8

@@ -0,0 +1,4 @@

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Import UIKit as we no longer do that implicitly

@@ -0,0 +1,38 @@
#import "ViewController.h"

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove empty lines

@@ -0,0 +1,5 @@
name "iphone/application_spriteKit_game"

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove empty lines 2, 3


#import <SpriteKit/SpriteKit.h>
@interface myScene : SKScene <SKPhysicsContactDelegate>
-(instancetype)initWithSize:(CGSize)size;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please do not indent method protocol lines



#import <SpriteKit/SpriteKit.h>
@interface myScene : SKScene <SKPhysicsContactDelegate>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Class does not conform to convention (PascalCase)

@@ -0,0 +1,3 @@
@interface ViewController : UIViewController
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Import UIKit as we no longer do that implicitly

Description: An awesome Game!
Maintainer: @@USER@@
Author: @@USER@@
Section: Utilities
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe "Games" section?

Description: An awesome single view application!
Maintainer: @@USER@@
Author: @@USER@@
Section: Utilities
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe "Games" section?

@L1ghtmann
Copy link
Member

Is there still a use case for a SpriteKit game template? If so, we can likely go ahead with editing this in-place as the requested changes don't appear to have been addressed. Also, it seems that the application template pr'd is extremely similar to the one introduced in 12a565a except that is presents a UIView instead of a UITableView. Given this, I'm in favor of removing that portion for the pr.

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.

6 participants