fix: checking navigator type before to determine isMacOs

[rolancia]

This PR contains:

• bugfix
• feature
• refactor
• tests
• documentation
• others

Breaking Changes?

• yes
• no

Checklist

• commit messages follow the Conventional Commits pattern
  • A feature commit message is prefixed "feat:"
  • A bugfix commit message is prefixed "fix:"
  • A document related commit is prefixed "docs:"
• docs have been added / updated (for bug fixes / features)

Describe Your Changes

Adds checking for /Mac/.test(navigator.platform) to avoid the error 'navigator is not defined' with SSR.

Does this PR affects any existing issues?

• yes
• no

[sibiraj-s]

This will fix the crash, but it will always use the other shortcut in MacOs. That is fine?

[rolancia]

i don't understand why it makes the problem.
(sorry, am not familiar with Angular, other web side neither)

but if it does, using Angular core library also is a good way to detect platform?

[sibiraj-s]

You should be able to detect platform from the request body. You can read more information regarding that here https://angular.io/guide/universal.

I will merge this.

[rolancia]

thank you!

[rolancia]

oh. you deployed it.. appreciate

[github-actions]

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs and link to relevant comments in the thread.