Allow to add directories #138

Merged
kernelwhisperer merged 2 commits from bugfix/156-allow-to-upload-directories-on-win into master 2018-06-15 04:23:33 -04:00
kernelwhisperer commented 2018-06-13 11:38:40 -04:00 (Migrated from github.com)

What changed?

Added two new sub menus under File: Add File and Add Directory.

## What changed? Added two new sub menus under `File`: Add File and Add Directory.
kernelwhisperer (Migrated from github.com) reviewed 2018-06-13 11:42:50 -04:00
@ -6,4 +6,2 @@
const { app, dialog, shell } = remote
/**
* Returns `true` if the user wants to wrap all files under a single dir,
kernelwhisperer (Migrated from github.com) commented 2018-06-13 11:42:50 -04:00

remote is undefined when is called from the main process (which is happening now since I used this in the menu), but importing { app, dialog, shell } directly seems to work.

`remote` is `undefined` when is called from the main process (which is happening now since I used this in the menu), but importing `{ app, dialog, shell }` directly seems to work.
koalalorenzo (Migrated from github.com) requested changes 2018-06-13 12:43:30 -04:00
@ -2,16 +2,41 @@
* Import/require this file to set up the Menu of the windows
koalalorenzo (Migrated from github.com) commented 2018-06-13 12:42:54 -04:00

Why this doesn't have app.mainWindow and the other has?

Why this doesn't have app.mainWindow and the other has?
kernelwhisperer (Migrated from github.com) reviewed 2018-06-13 12:46:28 -04:00
@ -2,16 +2,41 @@
* Import/require this file to set up the Menu of the windows
kernelwhisperer (Migrated from github.com) commented 2018-06-13 12:46:28 -04:00

This is ran before the StorageList is created. Maybe is worth considering moving this to StorageList and creating the menu afterwards? We disable the menu for any other window anyway...

This is ran before the StorageList is created. Maybe is worth considering moving this to StorageList and creating the menu afterwards? We disable the menu for any other window anyway...
koalalorenzo (Migrated from github.com) reviewed 2018-06-13 14:36:49 -04:00
@ -2,16 +2,41 @@
* Import/require this file to set up the Menu of the windows
koalalorenzo (Migrated from github.com) commented 2018-06-13 14:36:49 -04:00

I see, as we discussed maybe move the creation of the Menu into the Storage Window only.

I see, as we discussed maybe move the creation of the Menu into the Storage Window only.
kernelwhisperer (Migrated from github.com) reviewed 2018-06-14 08:28:06 -04:00
@ -2,16 +2,41 @@
* Import/require this file to set up the Menu of the windows
kernelwhisperer (Migrated from github.com) commented 2018-06-14 08:28:06 -04:00

Turns out we can only create the menu from the main process. Nonetheless, I added the missing app.mainWindow to the dialog and it seems to work (StorageList is unusable while the dialog is open).

The warning I was getting: "Gtk-Message: GtkDialog mapped without a transient parent. This is discouraged." seems to be unrelated to this.

Turns out we can only create the menu from the main process. Nonetheless, I added the missing `app.mainWindow` to the dialog and it seems to work (StorageList is unusable while the dialog is open). The warning I was getting: "Gtk-Message: GtkDialog mapped without a transient parent. This is discouraged." seems to be unrelated to this.
koalalorenzo (Migrated from github.com) approved these changes 2018-06-15 04:20:23 -04:00
Sign in to join this conversation.
No reviewers
No labels
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
Siderus/Orion!138
No description provided.