eduvpn / apple

app for iOS and macOS
Other
62 stars 18 forks source link

Display Let's Connect! as Let's Connect! #303

Closed jeroenleenarts closed 4 years ago

jeroenleenarts commented 4 years ago

Fixes #301

On simulator iPhone SE (1st gen) iOS 13.5 it looks good. We might want to do some more testing/checking with users with iOS 12 and adjusted font sizes.

github-actions[bot] commented 4 years ago

Make sure to keep CHANGES.md up to date!

roop commented 4 years ago

@jeroenleenarts So we keep Info.plist entry as "Let's Connect", but product name as LetsConnect. Is that correct?

jeroenleenarts commented 4 years ago

Yes. Only display name is allowed to contain the Let’s Connect name properly formatted.

roop commented 4 years ago
jeroenleenarts commented 4 years ago

@roop Good feedback. I'll change that later today.

roop commented 4 years ago

@jeroenleenarts Does it work on macOS too? I couldn't find a CFBundleDisplayName entry in the macOS Info.plist, so am a bit confused. (I haven't tested it in iOS or macOS.)