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

Refactoring & Improvement Suggestions #2

Open
1 of 9 tasks
Ambratolm opened this issue Aug 4, 2021 · 0 comments · Fixed by #1 · May be fixed by #3
Open
1 of 9 tasks

Refactoring & Improvement Suggestions #2

Ambratolm opened this issue Aug 4, 2021 · 0 comments · Fixed by #1 · May be fixed by #3

Comments

@Ambratolm
Copy link

Ambratolm commented Aug 4, 2021

  • Organize project structure into 📁 sub-folders dividing GUI components from core API.

  • Divide API into 📄 simple components following some good practices.

  • Rename variables and members into clear , consistent, and homogeneous names (no more 🇫🇷➕🇬🇧 french-english mixing, 🔤 unclear abbreviations, and 🎲 random letter casing) following some good consistent 📕 naming conventions.

  • Remove useless 🗨️ comments (Code must comment itself or use reasonable comment that tells something unclear from the code itself).

  • Replace many created 🔧 utility functions with better .NET available alternatives.

  • Upgrade .NET Framework from v2.0 to v4.0 allowing the use of 🔩 extension methods for better 🔧 utility functions use.

  • Turn program native language (Text in GUI, Code, and Database) fully to 🇬🇧 english instead of 🇫🇷 french (It should be 🇬🇧 english by default if there's no language resource set).

  • Update the GUI following some good 📕 Windows GUI standards (No more weird inconsistent incoherent colors and fonts on the controls. Either follow the native Windows look or use custom controls library).

  • Update 📖 README.md file with some infos and fancy screenshots 😆.

@AXeL-dev AXeL-dev linked a pull request Aug 7, 2021 that will close this issue
@AXeL-dev AXeL-dev linked a pull request Aug 29, 2021 that will close this issue
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 a pull request may close this issue.

1 participant