Skip to content

Check for iOS#168

Merged
lervag merged 3 commits intolervag:masterfrom
anjiro:master
Jul 7, 2021
Merged

Check for iOS#168
lervag merged 3 commits intolervag:masterfrom
anjiro:master

Conversation

@anjiro
Copy link
Copy Markdown
Contributor

@anjiro anjiro commented Jul 7, 2021

Explicitly check for iOS to fix terrychou/iVim#205

Explicitly check for iOS too fix terrychou/iVim/#205
return 'win'
elseif has('unix')
if system('uname') =~# 'Darwin'
if has{'ios') || system('uname') =~# 'Darwin'
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

It seems you've used { instead of ( here. Also, perhaps we should also add has('mac')?

return 'win'
elseif has('unix')
if has('mac') || system('uname') =~# 'Darwin'
if has('mac') || has{'ios') || system('uname') =~# 'Darwin'
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

{ should be (

@lervag
Copy link
Copy Markdown
Owner

lervag commented Jul 7, 2021

Thanks; I'll merge this after you address my two comments.

anjiro added 2 commits July 7, 2021 16:46
iPhone screen with tiny font…
@anjiro
Copy link
Copy Markdown
Contributor Author

anjiro commented Jul 7, 2021

Whoops, tiny font on iPhone screen leads to typos. Fixed!

@lervag lervag merged commit 523cc52 into lervag:master Jul 7, 2021
@lervag
Copy link
Copy Markdown
Owner

lervag commented Jul 7, 2021

Thanks!

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.

wiki.vim prevents iVim from working

2 participants