feat: use system browser for oauth flow

Signed-off-by: Adam Setch <[email protected]>

Author: setchy

package.json

@@ -80,6 +80,12 @@
       "package.json"
     ],
     "electronLanguages": ["en"],
+    "protocols": [
+      {
+        "name": "Gitify",
+        "schemes": ["gitify"]
+      }
+    ],
     "mac": {
       "category": "public.app-category.developer-tools",
       "icon": "assets/images/app-icon.icns",

src/main/main.ts

@@ -20,7 +20,6 @@ const browserWindowOpts = {
   resizable: false,
   skipTaskbar: true, // Hide the app from the Windows taskbar
   webPreferences: {
-    enableRemoteModule: true,
     nodeIntegration: true,
     contextIsolation: false,
   },

src/renderer/components/settings/NotificationSettings.tsx

@@ -1,7 +1,7 @@
 import { type FC, type MouseEvent, useContext } from 'react';
 
 import { BellIcon } from '@primer/octicons-react';
-import { Stack, Text } from '@primer/react';
+import { Box, Stack, Text } from '@primer/react';
 
 import { APPLICATION } from '../../../shared/constants';
 import { AppContext } from '../../context/App';
@@ -61,9 +61,8 @@ export const NotificationSettings: FC = () => {
           tooltip={
             <Text>
               See{' '}
-              <button
-                type="button"
-                className="text-gitify-link"
+              <Box
+                className="text-gitify-link cursor-pointer"
                 title="Open GitHub documentation for participating and watching notifications"
                 onClick={(event: MouseEvent<HTMLElement>) => {
                   // Don't trigger onClick of parent element.
@@ -72,7 +71,7 @@ export const NotificationSettings: FC = () => {
                 }}
               >
                 official docs
-              </button>{' '}
+              </Box>{' '}
               for more details.
             </Text>
           }

src/renderer/routes/Accounts.tsx

@@ -107,7 +107,6 @@ export const AccountsRoute: FC = () => {
                     title="Open account profile"
                     onClick={() => openAccountProfile(account)}
                     data-testid="account-profile"
-                    className="pb-2"
                   >
                     <AvatarWithFallback
                       src={account.user.avatar}
@@ -130,10 +129,10 @@ export const AccountsRoute: FC = () => {
                         </Stack>
                       </Box>
 
-                      <button
+                      <Box
                         title="Open host"
-                        type="button"
                         onClick={() => openHost(account.hostname)}
+                        className="cursor-pointer"
                         data-testid="account-host"
                       >
                         <Stack
@@ -144,12 +143,12 @@ export const AccountsRoute: FC = () => {
                           <PlatformIcon />
                           <Text>{account.hostname}</Text>
                         </Stack>
-                      </button>
+                      </Box>
 
-                      <button
+                      <Box
                         title="Open developer settings"
-                        type="button"
                         onClick={() => openDeveloperSettings(account)}
+                        className="cursor-pointer"
                         data-testid="account-developer-settings"
                       >
                         <Stack
@@ -160,7 +159,7 @@ export const AccountsRoute: FC = () => {
                           <AuthMethodIcon />
                           <Text>{account.method}</Text>
                         </Stack>
-                      </button>
+                      </Box>
                     </Stack>
                   </Box>
                 </Stack>