MalloyDelacroix / DownloaderForReddit

The Downloader for Reddit is a GUI application with some advanced features to extract and download submitted content from reddit.
GNU General Public License v3.0
511 stars 48 forks source link

Import list #99

Closed DankMemeGuy closed 4 years ago

DankMemeGuy commented 4 years ago

Would be nice to import a user or subreddit list through a textfile

zacker150 commented 4 years ago

If you right click the add user dialog, the capability is there. image

That being said, we should probably update the dialog to include buttons for the import options. I propose we split the Add User/Subreddit Dialog into two dialogs:

MalloyDelacroix commented 4 years ago

I agree, these options should be presented as buttons. I also like the idea of single and bulk add dialogs.

If we have two completely separate dialogs, we will have to add another button to the main GUI for bulk adds. What if instead of doing that, we used the current add user dialog as the single add dialog and left it as the default when the '+' button is clicked. Then add a 'Bulk Add' button to this dialog which then transforms it into a bigger dialog with all of the import options.

I know it already somewhat has this functionality by transforming into the list version, but it needs a lot of changes and improvements.

zacker150 commented 4 years ago

Let's keep this issue open until the new UI has been implemented.

MalloyDelacroix commented 4 years ago

I agree. We'll keep it open.

zacker150 commented 4 years ago

Here is a proposed design for the new UI: image image

Comments?

MalloyDelacroix commented 4 years ago

I like the tabbed window and everything still looks pretty clean.

The only thing that I would add is an import button. I'm thinking we just need one button that opens a file select dialog and then handle selecting the importer type in code based on the file extension. That way we don't have to have four different buttons.

add_users_dialog
MalloyDelacroix commented 4 years ago

This dialog rework as been incorporated into the database_backend branch and will be merged when the database integration is complete.

I am leaving this here so no one goes down the path of reworking this dialog because the new version is not added to the master branch.