Skip to content

fix(chat): stack emoji button below attachment button on multi-line messages - #6673

Open
rapterjet2004 wants to merge 1 commit into
masterfrom
issue-6652-optimize-emoji-picker
Open

fix(chat): stack emoji button below attachment button on multi-line messages#6673
rapterjet2004 wants to merge 1 commit into
masterfrom
issue-6652-optimize-emoji-picker

Conversation

@rapterjet2004

@rapterjet2004 rapterjet2004 commented Sep 9, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

The emoji button sitting next to the attachment button wasted about a third of the input's width once a message wrapped to multiple lines. Move it below the attachment button in that case, freeing that width for the text, while keeping the original layout for short messages.

🖼️ Screenshots

Shorter messages Long messages
Screenshot 2026-09-09 at 10 08 44 AM Screenshot 2026-09-09 at 10 56 24 AM

🏁 Checklist

  • ⛑️ Tests (unit and/or integration) are included or not needed
  • 🔖 Capability is checked or not needed
  • 🔙 Backport requests are created or not needed: /backport to stable-xx.x
  • 📅 Milestone is set
  • 🌸 PR title is meaningful (if it should be in the changelog: is it meaningful to users?)

🤖 AI (if applicable)

  • The content of this PR was partly or fully generated using AI

@rapterjet2004 rapterjet2004 self-assigned this Sep 9, 2026
@rapterjet2004 rapterjet2004 added the 3. to review Waiting for reviews label Sep 9, 2026
@rapterjet2004
rapterjet2004 requested a review from mahibi September 9, 2026 15:11
@github-actions

github-actions Bot commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

APK file: https://github.com/nextcloud/talk-android/actions/runs/34368485904/artifacts/10111192965
To test this change/fix you can simply download above APK file and install and test it in parallel to your existing Nextcloud app.
qrcode (please click on link to get QR code displayed)

@mahibi
mahibi requested a review from nimishavijay September 9, 2026 15:17
@github-actions

github-actions Bot commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

APK file: https://github.com/nextcloud/talk-android/actions/runs/34373647424/artifacts/10113544567
To test this change/fix you can simply download above APK file and install and test it in parallel to your existing Nextcloud app.
qrcode (please click on link to get QR code displayed)

@rapterjet2004
fix(chat): stack emoji button below attachment button on multi-line m…
9d19f3c 
…essages

The emoji button sitting next to the attachment button wasted about a
third of the input's width once a message wrapped to multiple lines.
Move it below the attachment button in that case, freeing that width
for the text, while keeping the original layout for short messages.

Assisted-By: Claude Code:claude-sonnet-5

Signed-off-by: rapterjet2004 <juliuslinus1@gmail.com>
@rapterjet2004
rapterjet2004 force-pushed the issue-6652-optimize-emoji-picker branch from 5843f45 to 9d19f3c Compare September 10, 2026 14:10
@github-actions

github-actions Bot commented Sep 10, 2026

Copy link
Copy Markdown
Contributor

APK file: https://github.com/nextcloud/talk-android/actions/runs/34487308808/artifacts/10156852866
To test this change/fix you can simply download above APK file and install and test it in parallel to your existing Nextcloud app.
qrcode (please click on link to get QR code displayed)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mahibi mahibi Awaiting requested review from mahibi
@nimishavijay nimishavijay Awaiting requested review from nimishavijay

At least 1 approving review is required to merge this pull request.

Assignees

@rapterjet2004 rapterjet2004

Labels

3. to review Waiting for reviews AI assisted

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Emoji picker could suit a few adjustements

1 participant

@rapterjet2004