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

UI/exit node status indicators (emphasizes the added functionality) #177

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

Conversation

aalokkamble
Copy link
Contributor

Refactored code exit node screen UI and improved visual feedback:

  1. Improved clarity: Unavailable nodes are now displayed in gray for better indication of their status.
  2. Enhanced user experience: The selected exit node, if offline, is now highlighted in red on the main screen for clear visual confirmation.

Signed-off-by: Aalok Kamble <aalok.kamble@gmail.com>
Aligned dot vertically with lowercase letters for machine names.
Reverted 'Machine' to 'My devices'

Signed-off-by: Aalok Kamble <aalok.kamble@gmail.com>
Improved clarity: Unavailable nodes are now displayed in gray for better indication of their status.
Enhanced user experience: The selected exit node, if online, is now highlighted in red on the main screen for clear visual confirmation.
This commit message effectively conveys the following:

Action: Refactored the code for the UI.
Specific changes:
Introduced visual cues for unavailable nodes.
Implemented highlighting for the selected exit node, indicating its online status.
Benefit: Improved clarity and user experience.
@aalokkamble
Copy link
Contributor Author

Screenshot_20240229-184652.png

@aalokkamble
Copy link
Contributor Author

@sonovawolf Please review.

I know this is not the place to discuss changes but here is my recommendation:

current single file, ui.go, is getting cumbersome. Splitting it offers advantages:

Readability: Functions grouped by purpose make code easier to understand.
Maintainability: Changes become isolated, reducing unintended side effects.
Collaboration: Developers can work on different parts simultaneously.

Here's the plan:

Analyze: Identify logical boundaries in ui.go to split by features.
Structure: Define a clear file structure for related functionalities.
Docs: Update documentation/comments to reflect the new organization.

Splitting ui.go will create a cleaner, more manageable codebase for future development.

@kari-ts
Copy link
Contributor

kari-ts commented Mar 8, 2024

@sonovawolf Please review.

I know this is not the place to discuss changes but here is my recommendation:

current single file, ui.go, is getting cumbersome. Splitting it offers advantages:

Readability: Functions grouped by purpose make code easier to understand. Maintainability: Changes become isolated, reducing unintended side effects. Collaboration: Developers can work on different parts simultaneously.

Here's the plan:

Analyze: Identify logical boundaries in ui.go to split by features. Structure: Define a clear file structure for related functionalities. Docs: Update documentation/comments to reflect the new organization.

Splitting ui.go will create a cleaner, more manageable codebase for future development.

Hey @aalokkamble, thank you so much for preparing these contributions! We've just started on a large set of changes that will replace the gioui code in this application with a Kotlin implemented GUI instead, which we expect will improve our maintenance costs and improve our ability to ensure that our app keeps up with the latest Android features. We do not yet have an organized way for contributors to follow that work stream, as it's still relatively early, getting the initial IPC and build system wiring in a good state. The existing gioui code base will be entering a bug-fix only phase as we move forward with this new strategy.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants