aheze/Multiliner

Update built-in docs

orchetect opened this issue · 5 comments

I just noticed that the GitHub docs / App Built-In Docs are out of sync.

  • GitHub README shows setting up a Key Binding in Xcode,
  • built-in docs of the MultilinerExtension app shows setting up an App Shortcut in System Preferences

Cheers, and thanks for your hard work

Updated and pull request done.

  • Updated built-in docs to show Xcode key binding suggestion (screenshot and text)
  • Bumped both App and Extension marketing version to 1.2 (need to be identical) in preparation for eventual next release (before this, the app was 1.1 and the extension was 1.0 which Xcode throws a built warning at because it requires the app and extension to have identical version numbers)
  • Added gitignore (keep spicy things out of the repo, like xcode user data and .DS_Store file junk)

#17

aheze commented

Hey, thanks for the PR! Yeah the app is outdated right now... actually both the README and the app are old. Personally I prefer setting the shortcut in System Prefs, but maybe Xcode is easier — what do you think?

Screen Shot 2022-07-09 at 11 38 17 AM

aheze commented

Thanks for adding the gitignore btw!

I think my gut instinct is 1) go for consistency and 2) since Xcode natively offers key bindings, that's probably where most people would be setting it.

aheze commented

Aight makes sense