Skip to content

Conversation

@AndyScherzinger
Copy link
Member

@AndyScherzinger AndyScherzinger commented Dec 28, 2025

  • 💡 Beware the input "bubble is wider than the chat. I did this to not loose more space since the buttons (attachment, emoji and send/mic are 48dp in height/width for a11y reasons) if I add another 8dp on each side of the bubble the message input text field "looses" another 16dp in width
  • Input field buttons are now bottom-aligned like in other messengers
  • paddings and margins are more unified across the chat input elements in general, same for the grey tones in use

🖼️ Screenshots

🏚️ Before 🌞 🏚️ Before 🌑 🏡 After 🌞 🏡 After 🌑
before_light before_dark after_light after_dark
before_single_line before_single_line_dark after_single_line after_single_line_dark
before_multi_line before_multi_line_dark after_multi_line after_multi_line_dark
before_edit before_edit_dark after_edit after_edit_dark
before_reply before_reply_dark after_reply after_reply_dark
before_thread before_thread_dark after_thread after_thread_dark

🚧 TODO

  • Design review
  • Code review

🏁 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?)

@AndyScherzinger AndyScherzinger added this to the 23.0.0 milestone Dec 28, 2025
@AndyScherzinger AndyScherzinger added design Related to the design 3. to review Waiting for reviews labels Dec 28, 2025
@AndyScherzinger AndyScherzinger force-pushed the style/noid/messageInputChatBubbleStyled branch 6 times, most recently from f05a902 to 59cd18f Compare December 29, 2025 12:59
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
@AndyScherzinger AndyScherzinger force-pushed the style/noid/messageInputChatBubbleStyled branch from 59cd18f to 2236ac0 Compare December 29, 2025 20:41
@github-actions
Copy link
Contributor

APK file: https://www.kaminsky.me/nc-dev/android-artifacts/5671.apk

qrcode

To test this change/fix you can simply download above APK file and install and test it in parallel to your existing Nextcloud app.

@github-actions
Copy link
Contributor

Codacy

Lint

TypemasterPR
Warnings9997
Errors00

SpotBugs

CategoryBaseNew
Bad practice66
Correctness1010
Dodgy code5454
Internationalization33
Malicious code vulnerability33
Performance44
Security11
Total8181

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

Labels

3. to review Waiting for reviews design Related to the design

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants