Skip to content

Comments

Add sending freeform reactions#1

Open
frebib wants to merge 6 commits intoChelseaDH:feature/emoji-searchfrom
frebib:frebib/freeform-reaction
Open

Add sending freeform reactions#1
frebib wants to merge 6 commits intoChelseaDH:feature/emoji-searchfrom
frebib:frebib/freeform-reaction

Conversation

@frebib
Copy link
Collaborator

@frebib frebib commented Apr 7, 2024

No description provided.

ChelseaDH and others added 5 commits March 3, 2024 21:09
Changes:
- Ensures search state is cleared when the bottom sheet is dismissed
- Adds padding below search bar
- Moves search placeholder into string resource
- Moves emoji grid into composable for reuse within the EmojiPicker
- Only displays existing emoji picker tabs when search is not active
@github-actions
Copy link

github-actions bot commented Apr 7, 2024

Thank you for your contribution! Here are a few things to check in the PR to ensure it's reviewed as quickly as possible:

  • Your branch should be based on origin/develop, at least when it was created.
  • There is a changelog entry in the changelog.d folder with the Towncrier format.
  • The test pass locally running ./gradlew test.
  • The code quality check suite pass locally running ./gradlew runQualityChecks.
  • If you modified anything related to the UI, including previews, you'll have to run the Record screenshots GH action in your forked repo: that will generate compatible new screenshots. However, given Github Actions limitations, it will prevent the CI from running temporarily, until you upload a new commit after that one. To do so, just pull the latest changes and push an empty commit.

@frebib frebib force-pushed the frebib/freeform-reaction branch from 3c1a677 to b8e6e4f Compare April 7, 2024 13:54
Signed-off-by: Joe Groocock <me@frebib.net>
@frebib frebib force-pushed the frebib/freeform-reaction branch from b8e6e4f to d78dfbe Compare April 7, 2024 14:07
@frebib frebib force-pushed the feature/emoji-search branch 4 times, most recently from ab4f208 to 7ae884b Compare May 21, 2024 21:08
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