Skip to content
This repository has been archived by the owner on Sep 6, 2018. It is now read-only.

Font relative paths #71

Merged
merged 7 commits into from
Aug 20, 2017
Merged

Font relative paths #71

merged 7 commits into from
Aug 20, 2017

Conversation

djbe
Copy link
Member

@djbe djbe commented Aug 16, 2017

Fixes SwiftGen/SwiftGen#324.

SwiftGenKit related PR: SwiftGen/SwiftGenKit#52
StencilSwiftKit related PR: SwiftGen/StencilSwiftKit#60

  • Will need to undo the podfile changes once the StencilSwiftKit PR is merged.

@djbe djbe added this to the 2.1.0 milestone Aug 16, 2017
@djbe djbe mentioned this pull request Aug 16, 2017
1 task
Copy link
Collaborator

@AliSoftware AliSoftware left a comment

Choose a reason for hiding this comment

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

We should (manually) test at some point that paths are correct and fonts are registered properly by using an actual dummy project that would include some Project/FooBrand/Resources/UI/Fonts folder, added as a folder reference, and itself containing subfolders like DarkTheme/Article/MyFont.ttf and look at the bundle structure and what we got once compiled.

@@ -19,6 +19,7 @@ You can customize some elements of this template by overriding the following par
| Parameter Name | Default Value | Description |
| -------------- | ------------- | ----------- |
| `enumName` | `FontFamily` | Allows you to change the name of the generated `enum` containing all font families. |
| `preservePath` | N/A | Setting this parameter will disable the basename filter applied to all font paths. Use this if you are adding fonts to your target in a specific folder, and need to preserve those folder names when registering fonts. The path will be relative to the folder you provided to SwiftGen. |
Copy link
Collaborator

Choose a reason for hiding this comment

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

Use this if you added your font folder as a "folder reference" in your Xcode project, making that folder hierarchy preserved once copied in the build app bundle

@@ -18,6 +18,7 @@ You can customize some elements of this template by overriding the following par
| Parameter Name | Default Value | Description |
| -------------- | ------------- | ----------- |
| `enumName` | `FontFamily` | Allows you to change the name of the generated `enum` containing all font families. |
| `preservePath` | N/A | Setting this parameter will disable the basename filter applied to all font paths. Use this if you are adding fonts to your target in a specific folder, and need to preserve those folder names when registering fonts. The path will be relative to the folder you provided to SwiftGen. |
Copy link
Collaborator

Choose a reason for hiding this comment

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

Ditto

@@ -18,6 +18,7 @@ You can customize some elements of this template by overriding the following par
| Parameter Name | Default Value | Description |
| -------------- | ------------- | ----------- |
| `enumName` | `FontFamily` | Allows you to change the name of the generated `enum` containing all font families. |
| `preservePath` | N/A | Setting this parameter will disable the basename filter applied to all font paths. Use this if you are adding fonts to your target in a specific folder, and need to preserve those folder names when registering fonts. The path will be relative to the folder you provided to SwiftGen. |
Copy link
Collaborator

Choose a reason for hiding this comment

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

Ditto

@djbe djbe force-pushed the feature/font-relative-paths branch from a086a7f to 558dab6 Compare August 17, 2017 10:54
@@ -19,6 +19,7 @@ You can customize some elements of this template by overriding the following par
| Parameter Name | Default Value | Description |
| -------------- | ------------- | ----------- |
| `enumName` | `FontFamily` | Allows you to change the name of the generated `enum` containing all font families. |
| `preservePath` | N/A | Setting this parameter will disable the basename filter applied to all font paths. Use this if you added your font folder as a "folder reference" in your Xcode project, making that folder hierarchy preserved once copied in the build app bundle. The path will be relative to the folder you provided to SwiftGen. |
Copy link
Collaborator

Choose a reason for hiding this comment

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

👌

@djbe djbe force-pushed the feature/font-relative-paths branch from 558dab6 to 1439a37 Compare August 20, 2017 15:31
@djbe djbe merged commit 52c4ff8 into master Aug 20, 2017
@djbe djbe deleted the feature/font-relative-paths branch August 20, 2017 15:36
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants