Skip to content

Conversation

@Maclean-D
Copy link
Contributor

@Maclean-D Maclean-D commented Dec 29, 2025

Home

  • Removed ghost button when tapping on home button:
ScreenRecording_12-28-2025.22-30-55_1.mov
  • Hide goal tracker widget when searching:
Simulator.mp4

Chat

  • Now it's own tab (Memories is now with speech profile and identifying others)
    • Updated way to send messages through (currently only used in goal widget)
    • separated the chat bar out and rewrote movement logic to handle all four states (microphone shown/not shown and hardware keyboard connected/disconnected)
  • removed apps button in favor of new chat app dropdown and new chat button
  • Now using "Chat Apps" instead of "Apps" and "Chat Assistants" to be less confusing.
Simulator.mp4

Copy link
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request successfully refactors the application to feature a dedicated "Chat" tab, improving user navigation and experience. The changes are comprehensive, including updates to the home page, chat page, and navigation logic from other parts of the app like the goal tracker and app details. The introduction of a new chat app selector in the app bar is a nice touch.

I've identified a critical bug related to a type mismatch that could cause a crash, a high-severity design issue regarding state management that could impact maintainability, and a violation of a repository rule for color opacity. Addressing these points will improve the code's correctness and robustness.

Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
@Maclean-D Maclean-D changed the title feat: Tab and Chat Improvements feat: Home and Chat Improvements Dec 29, 2025
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