-
-
Couldn't load subscription status.
- Fork 33
Installer ui #30
Installer ui #30
Conversation
browser.js
Outdated
| const getInstalledVersion = require('./lib/check-node') | ||
| const loadVersions = require('./lib/load.js') | ||
| // const install = require('./lib/install') | ||
| // for testing purpose |
This comment was marked as off-topic.
This comment was marked as off-topic.
Sorry, something went wrong.
This comment was marked as off-topic.
This comment was marked as off-topic.
Sorry, something went wrong.
This comment was marked as off-topic.
This comment was marked as off-topic.
Sorry, something went wrong.
This comment was marked as off-topic.
This comment was marked as off-topic.
Sorry, something went wrong.
This comment was marked as off-topic.
This comment was marked as off-topic.
Sorry, something went wrong.
This comment was marked as off-topic.
This comment was marked as off-topic.
Sorry, something went wrong.
|
LOVE the look of this :) |
|
I was wondering if could be a good idea add a |
That's a great idea! But that shouldn't hold up this commit, let's get this merged as soon as you think it's ready :) |
|
For me this first implementation of the UI is ready, I've removed the testing code and added another code example. Later I will open an issue for the If you think there are other things to change before merging, please let me know. |
|
LGTM. Merging in. |

Hi all, I've implemented the UI proposal as seen in #26.


I basically rewrited all the browser part.
Here's some screenshots:
During installation:
What do you think?
Still missing few things, such as the update of the code example and the error management if the installation fails.
For every question/suggestion I'm here.
If you want to try it by yourself clone this and run
npm start.- Update -
NOTE: the PR is not ready to be merged, I've opened early to get suggestions and feedback and allow you to try it. The final version will not have the testing code (see fakeInstall).