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

Bugfixes #124

Open
wants to merge 14 commits into
base: main
Choose a base branch
from
Open

Bugfixes #124

wants to merge 14 commits into from

Conversation

AliA5y
Copy link

@AliA5y AliA5y commented Aug 28, 2024

In this pull request, I am fixing some bugs and applying small enhancements to the UI. Here is a summary of changes and adjustments I made to the project:

  1. Enhance the UI of the intro screen.
  2. Fix issues of content overflow, due different screen size, in some UI components.
  3. Fix some bugs, that cause the app to crush, such as late initialization error and calling setState after a widget is disposed.
  4. Add scrolling to some screens to fit with small screens and avoid overflow.
  5. Fix receiving cancel issue whenever a file name contains unknown characters such as Arabic letters.
  6. Fix the error of being unable to start a server in some devices.
  7. Fix app crash issue when attempting to open file in windows.
  8. Get rid of addresses duplicates for faster scanning, and include the first interface in the scanning process.
    Attached some sample images of issues UI changes.
    photon-1
    Screenshot_20240824-165105
    Screenshot_20240824-165045
    Screenshot_20240824-165230
    Screenshot_20240824-165222
    Screenshot_20240824-165114

@AliA5y AliA5y force-pushed the bugfixes branch 2 times, most recently from ca6ac0c to d30a395 Compare October 27, 2024 11:03
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.

1 participant