Skip to content
This repository has been archived by the owner on Oct 16, 2020. It is now read-only.

Mac OS X Support #215

Open
Noitidart opened this issue Oct 22, 2015 · 4 comments
Open

Mac OS X Support #215

Noitidart opened this issue Oct 22, 2015 · 4 comments

Comments

@Noitidart
Copy link

I was hoping to submit a PR to add Mac OS X support. I see you have a specific file structure, and place cetrain things in certain places.

Can you please create the template files and put a comment of what you want me to write where, then I can populate it with the OSX code :)

This is how I typically do things: I have one main chromeworker, and then i import the ostypes to it:

https://github.com/Noitidart/NativeShot/blob/master/modules/workers/MainWorker.js

My ctypes for mac are in ostypes_mac.jsm: https://github.com/Noitidart/NativeShot/blob/master/modules/ostypes_mac.jsm

I basically want to know where I can populate this code in your addon :) I know there are no workers so no problem :)

@foudfou
Copy link
Owner

foudfou commented Oct 22, 2015

hi @Noitidart !
OS-specific code is in modules. So you should probably create modules/darwin and modules/ctypes/darwin.

But, before that, isn't there already dock support for OSX in FFX and TB ?

@Noitidart
Copy link
Author

Ah yes good point there is dock support, but maybe we can make the menus different. And in future if we add customizable menus that would be truly awesome.

The application teamviewer has docksupport but they also have tray support please see image here:

With the trayicon, we can offer option of hiding the dock icon. :)

@foudfou
Copy link
Owner

foudfou commented Oct 22, 2015

Why not. I don't know what OSX user would appreciate it, as I feel the dock is the mainstream way to go, OSX users tend to be mainstream users...
Besides, porting to OSX will probably have impact on the rest of the code. That's what experienced from porting to Windows. Considering the current list of open issues, I'm a bit afraid of such an impact.
All that said, if you want to give it a try, please do so!

@Noitidart
Copy link
Author

Ah I totally understand. Ok Ill do this low priority and just try it out sometme on a PR. Those answers you gave on which files to make helps a lot :)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants