Skip to content

Conversation

joeljfischer
Copy link
Contributor

Use menu and choice set managers

@NicoleYarroch
Copy link

The link to DarkSky is broken in the README

@NicoleYarroch
Copy link

The Supporting Files folder in the Xcode directory is red.

@NicoleYarroch
Copy link

Works well. I think the 1000 line SmartDeviceLinkService.m file should be broken into smaller, more readable files.

@joeljfischer
Copy link
Contributor Author

joeljfischer commented Oct 25, 2018

@NicoleYarroch fixed those weirdness things.

I don't disagree about breaking it apart...that'll just take a lot more time. It did use to be like 1600 lines before the managers though so...progress lol. I also added some pragmas to help.

@NicoleYarroch
Copy link

Should we add the Pod folder to the .gitignore and add instructions to the README on how to install the SDL library via CocoaPods?

@joeljfischer
Copy link
Contributor Author

🤷🏻‍♂️ We could. I've just been leaving it as is for now.


// Change which config you need based on if you want to connect to a TDK (default) or a wifi based emulator (debug)
SDLLifecycleConfiguration *lifecycleConfig = [SDLLifecycleConfiguration defaultConfigurationWithAppName:@"MobileWeather" appId:@"330533107"];
SDLLifecycleConfiguration *lifecycleConfig = [SDLLifecycleConfiguration defaultConfigurationWithAppName:@"MobileWeather" fullAppId:@"330533107"];
// SDLLifecycleConfiguration *lifecycleConfig = [SDLLifecycleConfiguration debugConfigurationWithAppName:@"MobileWeather" appId:@"330533107" ipAddress:@"192.168.1.61" port:2776];

Choose a reason for hiding this comment

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

The debug configuration is deprecated

@NicoleYarroch
Copy link

Have you tested the app with Manticore? The primary graphic images are barely visible due to the image color being almost identical to the background color. It it possible to template the primary graphic image so it looks good when Manticore is updated to Core 5.0?

self.manager.screenManager.textField4 = [conditions.windSpeed stringValueForUnit:speedType shortened:YES localization:self.localization];

self.manager.screenManager.primaryGraphic = [SDLArtwork artworkWithImage:[[ImageProcessor sharedProcessor] imageFromConditionImage:conditions.conditionIcon] name:conditions.conditionIcon asImageFormat:SDLArtworkImageFormatPNG];
self.manager.screenManager.primaryGraphic = [SDLArtwork artworkWithImage:[[[ImageProcessor sharedProcessor] imageFromConditionImage:conditions.conditionIcon] imageWithRenderingMode:UIImageRenderingModeAlwaysTemplate] name:conditions.conditionIcon asImageFormat:SDLArtworkImageFormatPNG];

Choose a reason for hiding this comment

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

There is another section of the code on L444 where the primary graphic is also set. This image isn't templated.

Choose a reason for hiding this comment

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

The menu icons should be templated too.

@joeljfischer joeljfischer merged commit 9f17b6d into master Oct 30, 2018
@joeljfischer joeljfischer deleted the feature/6.1 branch October 30, 2018 13:24
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.

2 participants