-
Notifications
You must be signed in to change notification settings - Fork 8
Settings page #3
Comments
We should check out https://github.com/gabrielbull/react-desktop - prebuilt react components with a native look / feel for OSX and Win10. |
And we also need to implement a persistence layer with one of the existing solutions (there are several existing modules that coordinate a backing JSON file for app settings - we should pick one) and wire it up to the IPC framework. |
Ah jeez, don't be forgetting the linux lot. You'll get some serious grief if you do 😉 |
They're just react components, so technically they would work with Linux. They're just not gonna integrate with gtk or anything so it would look out of place. An alternative could be to do one in a page in the app, similar to how GPMDP does it? |
One note on Win10 / Linux - thus far I have written the app to target only OSX since the immediate target is to replace Radiant 1.x. The keyboard shortcuts, menu items, and some window behaviors expect OSX, and most importantly, the custom title bar uses a native (C++) interface to the system mouse events so that it can performantly implement the drag regions, and that native module only works on OSX. So, if / when we decide to go cross-platform, those elements will need to be conditionally modified to behave as expected on the other platforms, and the mouse module will need to be rewritten for Windows and Linux in addition to theming considerations like the settings dialog. |
That mouse stuff is issue #2 |
Need a generic, tabbed (?) settings page.
Ideally accessible via the context menu, but I guess we're not tied that that if we have a better idea!
Note - if anyone fancies taking this on but doesn't fancy tackling the styling, feel free to add the functionality and ping me and I'll pretty it up.
The text was updated successfully, but these errors were encountered: