-
Notifications
You must be signed in to change notification settings - Fork 69
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
This library broke alacritty and deleted my config.. 馃憥馃憥馃憥馃憥 #131
Comments
hey, @joe-narvaez I'm sorry about that 馃檹 , we may solve it did you remember what did you do when it happened? |
I've had the same issue where it replaced my whole config with the alacritty-themes colors |
I think we should give some sort of escape hatch to the users if we have problems like these. May be something like always creating a backup for the config file, before modifying it for preview and apply themes. So the users can just rename/move/copy the backup file to restore the settings. |
Just outlining the steps here:
The backup file name could be something like
The downside is, we will be creating a lot of backup copies. But the user can still be able to clear out these files. |
What if you create a backup file and delete the older versions of the backup each time. |
yes, I agreed. creating a backup could work. Aside note:I think this error is showing up when you are moving between themes fast. |
Yes, I changed the theme a few times before this happened |
@rajasegar I'm thinking users will end up in the same scenario if we create a backup each time they select a new theme. We may create corrupt backups. What about to add the option: |
No no, not each time they select a new theme, but only one time when they invoke the program even before showing the prompts. |
sounds good! I'll check it |
This will be called once the `alacritty-themes` command gets executed. Related rajasegar#131
This will be called once the `alacritty-themes` command gets executed. Related rajasegar#131
This will be called once the `alacritty-themes` command gets executed. Related rajasegar#131
This will be called once the `alacritty-themes` command gets executed. Related #131
:(
The text was updated successfully, but these errors were encountered: