bookworm icon indicating copy to clipboard operation
bookworm copied to clipboard

Add the ability to restore settings to default

Open DraganRatkovich opened this issue 2 years ago • 20 comments

There is currently no way to restore the default settings in each category.

Describe the solution you'd like

Add a button to restore default settings in each category, and if possible, also add a button to completely reset Bookworm settings for all categories.

I think adding these two buttons will help users a lot in different scenarios, for example, if they want to reset only reading preferences or another specific category, they can just go to that category and restore default category settings or, if there a full reset is required, they can press the corresponding button and the settings will be reset.

In both cases, a dialog box should also appear informing users that the reset was successful, with an OK button or a close button to close the dialog box.

DraganRatkovich avatar May 05 '22 16:05 DraganRatkovich

Hi there. As said on the tellegram group, I'd love to take on this issue. I've taken a look at the code I might be changing. Just to make sure, each pannel basicly requires a "reset to defaults" button right? then mabey just before the "Apply" button, a button that says "reset all settings"?

ashleygrobler04 avatar May 07 '22 18:05 ashleygrobler04

Hello @ashleygrobler04 Yes, each category requires a button to reset only a certain category and, in addition, a complete reset, which can basically have a name, reset all settings, and which allows you to completely reset Bookworm settings.

DraganRatkovich avatar May 07 '22 18:05 DraganRatkovich

Hello @ashleygrobler04 @DraganRatkovich

I don't think we need to have a reset all button. We can just add a menu item to the file menu that can revert all config values to their defaults, and I think even that is unnecessary.

Best Musharraf

mush42 avatar May 09 '22 17:05 mush42

@ashleygrobler04

You don't need to add the 'Reset' button to each SettingsPanel subclass.

Just add it to the base class of all settings panels (bookworm.gui.settings.SettingsPanel), and it will be inherited by all its subclasses.

In the event handler, you can retrieve the current config section by using self.config_section.

mush42 avatar May 09 '22 17:05 mush42

Hi @mush42 At least one of the options must be enabled, because if the user only wants to reset certain category options, well, the option to do so will be in that category or, if the user needs to do a full program reset, again the corresponding button must exist. Not sure, choose the right one.

DraganRatkovich avatar May 09 '22 17:05 DraganRatkovich

@DraganRatkovich the category reset button is the primary feature. But the question is: do we need a reset all button?

mush42 avatar May 09 '22 18:05 mush42

@mush42 Yes, you need to add it to reset all settings to default, but again, it's entirely up to you.

DraganRatkovich avatar May 09 '22 18:05 DraganRatkovich

I don't think the ability to reset all config is much necessary. Of course this is just my personal opinion.

cary-rowen avatar May 10 '22 03:05 cary-rowen

Thinking about this, I don't think a reset settings button is needed, since the user can just go ahead and delete the config files if needed. If a reset settings button is required, not 1 button for each category, but rather 1 button that resets all the settings entirely.

ashleygrobler04 avatar May 10 '22 06:05 ashleygrobler04

@ashleygrobler04 No, deleting configuration files is not suitable for most users. Why do they have to deal with files when a button can do the rest of the work for them? Okay, I understand that you can't just add a reset all settings button, but resetting a category is important!

DraganRatkovich avatar May 10 '22 06:05 DraganRatkovich

@cary-rowen Due to the fact that every program these days supports resetting to default values, I thought it was important.

DraganRatkovich avatar May 10 '22 06:05 DraganRatkovich

@DraganRatkovich maby just the reset all settings option then? if it is ok with @mush42 .

ashleygrobler04 avatar May 10 '22 07:05 ashleygrobler04

@ashleygrobler04 I'm not sure, it's entirely up to you and @mush42 because you can add it as a developer. For me, each setting is important from the point of view of the user, who needs to either reset all settings to default values, or individual category settings.

DraganRatkovich avatar May 10 '22 07:05 DraganRatkovich

@ashleygrobler04 Let's implement the reset function for each category.

mush42 avatar May 10 '22 13:05 mush42

Ok, no probs. will do so. I don't know by when I will be done since I got college keeping me busy. Tomorrow is the last day of the week that I will have class, after that I'll start work on it if that is ok with you. Or if there is a spessific deadline I need to meet let me know.

ashleygrobler04 avatar May 10 '22 13:05 ashleygrobler04

@ashleygrobler04 Take your time.

mush42 avatar May 16 '22 16:05 mush42

@ashleygrobler04, any progress on this issue? I'd definitely use this feature.

TheQuinbox avatar Jun 12 '22 03:06 TheQuinbox

Hello, @ashleygrobler04 Do you have news on this issue?

DraganRatkovich avatar Aug 29 '22 19:08 DraganRatkovich

Hi there. I am sorry for not doing mutch. I haven't had the time to provide any contributions. College keeps me quite busy and life is just a little too hectic at the moment. Would it be possible for someone else to take care of this issue?

ashleygrobler04 avatar Aug 30 '22 07:08 ashleygrobler04

@mush42 Now all eyes are on you🙂

DraganRatkovich avatar Aug 30 '22 10:08 DraganRatkovich