Skip to content
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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improved resource error handling and translations, improved side button layout, fixed rescan duplicates, and more #26

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

click4dylan
Copy link

-Improved handling of missing resources so the user knows what file is missing.
-Improved some misspelled english words and changed some text to be more clear
-Added file path to error message when vital resources are missing
-Moved side buttons to the bottom for much better visual clarity

…ore clear

-Added file path to error message when vital resources are missing
-Moved side buttons to the bottom for much better visual clarity
@click4dylan click4dylan changed the title Improved resource error handling and translations, improved side button layout Improved resource error handling and translations, improved side button layout, fixed rescan duplicates, and more Nov 10, 2023
@Speedvicio
Copy link
Owner

I want to congratulate you for having the courage to understand such a messed up source code.
I still have to test the changes you've made and figure out whether to merge your code or create a dedicated branch on github. thank you for your interest.

Speedvicio added a commit that referenced this pull request Jan 6, 2024
You need to set it to false because whit your changed Resource code MgrSetting will not load the icons
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants