Fix profile fields width by adding modifier support to BorderedTextField#1151
Fix profile fields width by adding modifier support to BorderedTextField#1151EngrTaofeek merged 3 commits intoGreenstand:masterfrom
Conversation
- Added modifier parameter to BorderedTextField component - Updated ProfileField to pass fillMaxWidth() modifier - Cleaned up redundant padding application in BorderedTextField - Resolves issue where profile fields didn't utilize full screen width
|
@muhAzri thanks for this looks like great progress just one tiny detail left. The green underline does not extend to the end as shown I the screenshot below. |
|
Hi @muhAzri , To ensure you have everything you need to run the application smoothly, I'd like to share the tree_tracker.keys.properties file with you. Please join the Greenstand Slack community, and I'll send it to you there. This will eliminate the need to comment/uncomment lines in your configuration. Also, to keep our workflow organized, I recommend creating a new branch for each feature, task, or pull request (PR) instead of working directly on the master branch of your fork. This practice helps maintain a clean and manageable version history. Thanks |
|
Hi @EngrTaofeek ✅ I've fixed the reported issues the green underline not extending fully — it should now match the expected result. 🙏 Noted on the suggestion to use a separate branch for future PRs. I’ll definitely follow that practice going forward to keep things more organized. Looking forward to your review! |
|
Looks good. Thank you for your contribution I have merged it in. |

Fix profile fields width by adding modifier support to BorderedTextField
Thank you for opening a Pull Request!
Before submitting your PR, there are a few things you can do to make sure it goes smoothly:
./codeAnalysisto automatically apply formatting/linting)Description
This PR fixes an issue where the profile fields in the ProfileScreen weren't filling the maximum width of the screen, leaving unused space on the right side. The root cause was that the BorderedTextField component didn't accept a modifier parameter, preventing it from receiving layout directives from its parent components.
Changes
Visual Changes
Before: Profile fields didn't utilize the full width of the screen, leaving empty space on the right

After: Profile fields now properly extend to fill the entire width of the screen

Fixes #1138 🦕