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

Storyboard names #16

Closed
Ashton-W opened this issue Jan 29, 2015 · 7 comments · Fixed by #20
Closed

Storyboard names #16

Ashton-W opened this issue Jan 29, 2015 · 7 comments · Fixed by #20

Comments

@Ashton-W
Copy link

I don't fully understand the template format yet, so I don't know if this only requires a template update.

Could we include constants for the storyboard name?
As used in + (UIStoryboard *)storyboardWithName:(NSString *)name bundle:(NSBundle *)storyboardBundleOrNil

@orta
Copy link
Collaborator

orta commented Jan 29, 2015

You could run the ruby gem per storyboard and name the file differently?

@Ashton-W
Copy link
Author

I don't follow @orta? How would that create constants with the name of the *.storyboard?

@orta
Copy link
Collaborator

orta commented Jan 29, 2015

storyboard_ids:
    bundle exec sbconstants Kiosk/Storyboards/MainStoryboardIdentifiers.swift --source-dir Kiosk/Storyboards/Main --swift
    bundle exec sbconstants Kiosk/Storyboards/OtherStoryboardIdentifiers.swift --source-dir Kiosk/Storyboards/Other --swift

@Ashton-W
Copy link
Author

I'm using Objective-C if that makes a difference

@Ashton-W
Copy link
Author

@orta, maybe the Swift templates are different, but the ObjC template does not include the name of the storyboard

@orta
Copy link
Collaborator

orta commented Jan 29, 2015

Looks like it does to me in the README, I just used that as I had it to hand.

@orta
Copy link
Collaborator

orta commented Jan 29, 2015

Note, mine doesn't include the storyboard name either

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 a pull request may close this issue.

2 participants