Skip to content

Conversation

@MCCLUCKY415
Copy link
Collaborator

Pull Request Title

What’s this PR about? 🤔

Describe briefly what this Pull Request is doing. Explain the problem it solves or the feature it adds to the project.

  • Description:
  • What did you change? - (e.g., added a new feature, fixed a bug, updated something)

Why are we doing this? 💡

Explain why these changes are necessary.

  • Reason: - (e.g., fixing a bug, improving functionality, adding a new feature)

Type of Change 🛠️

What kind of change does this Pull Request introduce? Check the box that applies.

  • 🐛 Bug fix (non-breaking change which fixes an issue)
  • ✨ New feature (non-breaking change which adds functionality)
  • 📚 Documentation update (non-code changes, such as readme or comments)
  • ⚠️ Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • 🧹 Code clean-up/refactoring (basic code cleaning and or variable renaming)

How did you test it? 🧪

Explain how you tested your changes.

  • What did you test?
    • Tested locally
    • Wrote automated tests
    • I ran the code and it didn't break
    • Other: ______

Screenshots (if helpful) 📸

If your changes include something visual, like UI changes, add screenshots here!

Checklist ✅

Make sure you’ve done these things before submitting.

  • Code is running without errors
  • Everything works as expected
  • No unnecessary files or code
  • Documentation is updated if necessary
  • Pull Request title is clear and makes sense
  • All checkboxes are checked off!

Anything else? 💬

Any extra notes, questions, or concerns for your partner or reviewers?

MCCLUCKY415 and others added 26 commits June 7, 2025 16:32
- Applied a consistent toolbar style (`ToolBarTitle`, `ToolButtonIcon`) to `fragment_wallet_add.xml`, `fragment_budget_main.xml`.
- Updated `fragment_wallet_add.xml`:
    - Replaced existing toolbar content with a standardized `LinearLayout` containing back button, title, and a placeholder ImageButton.
    - Adjusted margins for the toolbar content.
- Updated `fragment_budget_main.xml`:
    - Replaced the existing title `TextView` with a new `AppBarLayout` and `Toolbar`.
    - Implemented the standardized toolbar structure with a back button, "Current Budgets" title, and a placeholder ImageButton.
    - Adjusted constraints to position content below the new app bar.
- Updated `fragment_achievements.xml`:
    - Changed the `src` of two `ImageView`s from `ic_ui_circle_check` to `ic_ui_security_lock`.
- Applied a consistent toolbar style (`ToolBarTitle`, `ToolButtonIcon`) to `fragment_wallet_add.xml`, `fragment_budget_main.xml`.
- Updated `fragment_wallet_add.xml`:
    - Replaced existing toolbar content with a standardized `LinearLayout` containing back button, title, and a placeholder ImageButton.
    - Adjusted margins for the toolbar content.
- Updated `fragment_budget_main.xml`:
    - Replaced the existing title `TextView` with a new `AppBarLayout` and `Toolbar`.
    - Implemented the standardized toolbar structure with a back button, "Current Budgets" title, and a placeholder ImageButton.
    - Adjusted constraints to position content below the new app bar.
- Updated `fragment_achievements.xml`:
    - Changed the `src` of two `ImageView`s from `ic_ui_circle_check` to `ic_ui_security_lock`.
…ps/dhiren

# Conflicts:
#	app/src/main/assets/version.txt
#	app/src/main/res/layout/fragment_wallet_add.xml
- Added a comprehensive list of new `IconItem` entries to the `_icons` list in `CategoryAddNewViewModel.kt`. This significantly increases the variety of icons users can choose from when creating new categories.
- Removed a specific `app` run configuration selection state from `.idea/deploymentTargetSelector.xml`.
- Added a comprehensive list of new `IconItem` entries to the `_icons` list in `CategoryAddNewViewModel.kt`. This significantly increases the variety of icons users can choose from when creating new categories.
- Removed a specific `app` run configuration selection state from `.idea/deploymentTargetSelector.xml`.
…ps/dhiren

# Conflicts:
#	app/src/main/assets/version.txt
- Set `android:visibility="invisible"` for `btnGoBack` in `fragment_budget_main.xml`.
- Set `android:visibility="invisible"` for the placeholder `ImageButton` in the toolbar in `fragment_budget_main.xml`.
…ps/dhiren

# Conflicts:
#	app/src/main/assets/version.txt
@MCCLUCKY415 MCCLUCKY415 requested a review from AlphaSweater June 9, 2025 21:08
@MCCLUCKY415 MCCLUCKY415 self-assigned this Jun 9, 2025
@AlphaSweater AlphaSweater deleted the feature/touchUps/dhiren branch June 9, 2025 21:49
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.

3 participants