Skip to content

UI Improvement & Design Fixes#387

Open
Rnk56 wants to merge 91 commits into
mainfrom
ronik/improvement-ui
Open

UI Improvement & Design Fixes#387
Rnk56 wants to merge 91 commits into
mainfrom
ronik/improvement-ui

Conversation

@Rnk56
Copy link
Copy Markdown
Collaborator

@Rnk56 Rnk56 commented Apr 3, 2026

Description

  • Resolved an issue with the Training button icon by adding the correct drawable resource to the res/drawable folder. Previously, the button displayed as a plain white square due to a missing asset. After adding and correctly referencing the icon, the button now displays as intended. (Care Taker Page)

  • Improved the visual distinction between the Task List and Patient List buttons. Initially, both buttons used the same icon, which could cause confusion for users. I introduced a new drawable resource specifically for the Task List button and updated its XML reference. This ensures that each button now has a unique and easily identifiable icon. (Care Taker Page)

  • Refined the content of Patient Manual 1 by updating the wording to be more professional and clear. These changes improve readability and ensure the information is presented in a more formal and user-friendly manner.

Checks Done:

  • Tested and working locally
  • Code follows the style guidelines of this project
  • I have performed a self-review of my code
  • Code changes documented
  • Requested review from >= 2 devs on the team

JohnAgaga and others added 30 commits November 25, 2024 02:46
Add Rate Limiting to PIN Verification
Fix application crash in patient list screen
Security & Privacy Documentation
* Enhance user role definition

* Add Nurse name to title

* Remove unused import
…ention

Add Feature Engineering Techniques and BILSTM + Attention hybrid model
implementation of Leave-One-Subject-Out Cross Validation to current l…
Implement K-fold cross-validation for BiLSTM and Combine the models to achieve higher accuracy.
BhuvanPS and others added 27 commits August 9, 2025 21:05
changes made
Added my admin login code with OTP verification and redirection to dashboard
Complete Phase 1 -> annotation guidelines, Doccano label configs, synthetic dataset
…rofile_chamoth

Created Sample Dataset according to given Schema
Web Admin Dashboard Shell Layout and Auth Flow UI
Copy link
Copy Markdown
Collaborator

@KudratAroraa KudratAroraa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The work completed within the stated PR scope looks good. The fixes for the Caretaker page icons and the refinement of Patient Manual 1 are meaningful improvements, and the intended changes themselves seem appropriate and useful. That can be approved easily.

However, this PR currently includes many unrelated file changes outside its stated scope, including AI-related files and Guardian admin dashboard files that do not appear to belong to this task. Because of that, the branch needs cleanup before it can be merged into the main repository. I would recommend removing all unrelated changes and keeping only the files directly relevant to the Caretaker page icon fixes and the Patient Manual wording updates. Once that cleanup is done, the PR should be in a much better state to review and merge confidently.

@KudratAroraa KudratAroraa self-assigned this May 28, 2026
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.