Skip to content

Change button order#11

Open
rozeappletree wants to merge 1 commit intoLiquid4All:mainfrom
rozeappletree:change-btn-order
Open

Change button order#11
rozeappletree wants to merge 1 commit intoLiquid4All:mainfrom
rozeappletree:change-btn-order

Conversation

@rozeappletree
Copy link
Copy Markdown
Contributor

prev oder (send, stop, clean) was not ux friendly because I was clicking on "clean" multiple times instead of send - because it was in right most side instead of "send" button.

New order is: clean, stop, send.

prev oder (send, stop, clean) was not ux friendly because I was clicking
on "clean" multiple times instead of send - because it was in right most
side instead of "send" button.

New order is: clean, stop, send.
iamstuffed added a commit that referenced this pull request Feb 9, 2026
Phase 3 Critical Review Issues:
- Issue #11: Fix FloatArray equality in AudioDemoMessage
  - Override equals() and hashCode() for proper structural comparison
  - Use contentEquals() for FloatArray comparison
  - Prevents Compose recomposition issues

- Issue #12: Add audioData.isNotEmpty() validation
  - Validate audio array is not empty before showing play button
  - Prevents UI crashes from attempting to play empty audio

- Issue #13: Add double-tap protection on FAB
  - Debounce FAB clicks with 500ms delay
  - Prevents race conditions from rapid start/stop cycles
  - Improves UX reliability

Also completed:
- Add onCleared() to ShareAI/AIChatViewModel
- Add onCleared() to RecipeGenerator/MainActivityViewModel
- All Android examples now have proper async model cleanup

All tests passing (59 tests).
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.

2 participants