Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Windows version #46

Open
simonwjackson opened this issue Sep 8, 2016 · 8 comments
Open

Windows version #46

simonwjackson opened this issue Sep 8, 2016 · 8 comments

Comments

@simonwjackson
Copy link

I'd like to contribute to getting this working on windows. What do you think the biggest hurdles will be for getting this to work cross platform?

@vutran
Copy link
Member

vutran commented Sep 8, 2016

Hi @simonwjackson I don't think it would be too difficult port it to Windows since a lot of the internals are handled within Node. We'll just have to make sure some of the core plugins such as dext-core-plugin-bookmarks support Window paths since it currently is reading to Mac specific directories.

I haven't tried launching this on Windows yet but I do have a Windows machine I can review any PR's you submit.

I would first try to get the core app launching first with the core calculator plugin and solve the bookmarks plugin afterwards.

@vutran
Copy link
Member

vutran commented Sep 8, 2016

I've actually just cloned the repo on my Windows 7 machine and it actually launched without any hiccups. 😎

Perhaps you can help by enabling the packaging scripts to build an executable for Windows.

@adnasa
Copy link
Member

adnasa commented Oct 2, 2016

Is this still an issue?

@simonwjackson
Copy link
Author

Wow, i only just got notified of a reply to my issue just now (almost a month later)

I'll install this now, and see how far i can push it

@simonwjackson
Copy link
Author

@vutran I just cloned the latest version, ran npm run dev and got the following error:

capture

@vutran
Copy link
Member

vutran commented Oct 3, 2016

Looks like an issue in repositionWindow where height is not specified.

@jonathanlaf
Copy link

Is that still an issue ? I would like to jump in and make it windows compatible.

Thanks.

@adnasa
Copy link
Member

adnasa commented Apr 6, 2018

@jonathanlaf thanks for jumping in. I welcome help on this, since I don't have a windows OS myself 😢

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

No branches or pull requests

4 participants