feat(chat): record video messages from the message input by ToteMeiSter · Pull Request #6818 · nextcloud/talk-android

ToteMeiSter · 2026-10-04T19:50:54Z

This is a feature PR; the issue is not approved yet, the PR is opened for discussion.

🖼️ Screenshots

🏚️ Before	🏡 After
The record button records voice only	A tap on the record button switches between voice and video; holding it records a video message with a live preview

Feature

A short tap on the record button in the message input switches between voice and video mode (the mode is stored in the preferences and shown by the icon and a hint popup). In video mode:

holding the button records with CameraX (front camera, 720p with fallback, about 2.5 Mbit/s, at most 120 s, then it stops and is sent);
sliding left cancels, sliding up locks; gestures, lock state, timer and the locked recording view are shared with the voice recording;
a live preview is shown above the input and the camera can be switched during the recording;
the recording is not lost on a screen rotation: the recorder lives in ChatViewModel like the voice recorder and the camera is bound to a lifecycle of the recorder, not to the activity;
a recording which the camera cuts off is shown in the attachment preview instead of being lost;
the finished mp4 goes through the regular file upload (a normal video attachment, not a special message type).

Structure

The PR is a stack. It contains the two voice recording fixes as the first commits:

After they are merged only the four commits of the feature remain:

feat(chat): record video messages from the message input
fix(chat): start recording only on hold and show the record hint as a popup
fix(chat): keep a video recording over a screen rotation
fix(chat): block taps and TalkBack focus on the video recording preview

How to check on a device

Open a chat, tap the record button: the mode switches and a hint is shown.
In video mode hold the button: the preview appears and the recording runs. Release to send, slide left to cancel, slide up to lock.
In a locked recording rotate the device: recording and preview go on. Switch the camera during a recording.
A very short recording shows a hint and is not sent.

Tested by the author on a Huawei DEL-LX9 with a fork build that has this feature together with other fork changes; this branch itself was built and unit-tested.

🚧 TODO

Discuss the feature with the maintainers (issue Record a video message from the chat input (hold to record, like voice messages) #6812 is not approved yet)

🏁 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

The change was written with the help of Claude Code (Anthropic). The author reviewed the code, built it and ran the unit tests, and checked the behavior on a device with the fork build.

coderabbitai · 2026-10-04T19:55:41Z

Note

Reviews paused

It looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the reviews.auto_review.auto_pause_after_reviewed_commits setting.

Last reviewed commit: 8c4e966
Latest commit, not reviewed: 70c586b

Use the following commands to manage reviews:

@coderabbitai resume to resume automatic reviews.
@coderabbitai review to trigger a single review.

Use the checkboxes below for quick actions:

▶️ Resume reviews
🔍 Trigger review

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI
Review profile: CHILL
Plan: Advanced
Run ID: 7752a140-89a1-4732-badc-e705c6546b06

📥 Commits

Reviewing files that changed from the base of the PR and between 62d3140 and 8c4e966.

📒 Files selected for processing: 1

app/src/main/java/com/nextcloud/talk/chat/ChatActivity.kt

Included review availability: This review used your included allowance. Your plan provides up to 4 included reviews per hour; 3 remain after this review.

📝 Walkthrough

The chat recording flow now supports video messages alongside voice recordings. Taps switch between voice and video modes, while holds start recording after a threshold. Video capture uses CameraX and supports camera switching, orientation handling, and activity recreation. The change adds mode-specific permissions and controls, persists the selected input mode, and adds tests for gestures, recording outcomes, lifecycle behavior, and preview placement.

Priority: ➖ Normal

Merge Risk: 🔵 Low · up to 8c4e9

Sending an unlocked video or sufficiently long voice recording can silently switch the saved mode, so the next recording may start in the wrong mode. Users can reselect the mode, making the impact bounded, but this behavior remains a low-level merge risk.

Security architecture risk: 🟡 Moderate · up to 59de4

Recording remains permission-gated and tied to the selected chat. However, sending during a camera switch can leave capture running briefly after the send action, and recording completion during screen recreation can bypass chat-readiness checks. These lifecycle risks affect privacy and recovery of sensitive recordings.

Retained concerns

Medium · security · inferred: Sending during a camera switch records the SEND decision immediately but defers the actual stop until the settling timer expires. Persistent camera/audio capture is not frozen at that decision, and a later background cancellation cannot override SEND. Consequently, the uploaded recording can include audiovisual content captured after the user requested sending, including after leaving the chat. The delay addresses a CameraX shutdown race, but does not preserve the user's capture endpoint.
Medium · reliability · inferred: The recreation flow defers an already-pending result until chat metadata is ready, but attaches an active recorder regardless of readiness. If that recorder finalizes before metadata arrives, its callback immediately uploads or previews the file. The recorder has already cleared its output and does not retain the result when a callback exists. Upload requires a current conversation, and preview requires initialized capabilities, so this ordering can fail after sensitive-file ownership has been relinquished, defeating the intended recovery handoff.

Security review details

Security Blast Radius

inferred — The identified privacy exposure concerns camera and microphone content from a user's recording session, delivered through the selected chat's normal attachment path. Unintended trailing content could reach that chat's recipients. The inspected change does not establish cross-account access, elevated privileges, or a remote capture endpoint.

Security Findings and Attack Paths

inferred — The privacy path requires an authorized user to record, switch cameras, and send during the settling window. SEND is committed before capture stops, so subsequent content may be uploaded without a new confirmation. This is an introduced capture-consent boundary risk, not evidence that an external attacker can bypass Android permissions.

Trust Boundaries and Controls

observed — Local account resolution remains ahead of chat initialization, while capture is reached through the user hold and permission checks. Upload dispatch supplies the activity's room token. The new recording methods are internal application APIs rather than exported Android component entrypoints.

Resilience and Maintainability Implications

observed — Discard-first finalization cannot become SEND, and releasing the recorder deletes pending output and suppresses delivery. These controls contain ordinary cancellation and teardown. They do not freeze capture during deferred stopping or make an attached completion callback readiness-aware.

Hardening Proposals

proposed — Separate the user's capture endpoint from deferred encoder finalization: freeze capture at the accepted stop decision using a CameraX-supported mechanism, and validate send-then-background behavior on devices. Keep completed-file ownership in chat state until a readiness-aware consumer accepts delivery, including when an active recorder finishes during recreation.

Pre-merge checks: 4 · 1

Failed checks: 1 warning

Check name	Status	Explanation	Resolution
Docstring Coverage		Docstring coverage is 22.08% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 231 functions across 26 files.	Write docstrings for the functions missing them to satisfy the coverage threshold.

Passed checks: 4 passed

Check name	Status	Explanation
Title check		The title clearly and concisely describes the main change: recording video messages from the chat message input.
Description check		The description follows the repository template and includes screenshots, feature details, testing information, TODO items, checklist status, dependencies, and AI disclosure. The milestone remains unc…
Linked Issues check		Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check		Check skipped because no linked issues were found for this pull request.

Full details: Description check

Explanation

The description follows the repository template and includes screenshots, feature details, testing information, TODO items, checklist status, dependencies, and AI disclosure. The milestone remains unchecked, but this is a non-critical omission for a discussion-stage feature PR.

✨ Finishing Touches: 💡 1

⚔️ Resolve merge conflicts: 💡

Resolve merge conflict in branch feat/video-message-record

Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

_{Comment @coderabbitai help to get the list of available commands.}

coderabbitai

Actionable comments posted: 1

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI
Review profile: CHILL
Plan: Advanced
Run ID: b493e8b5-ae18-41b3-a265-55028ef6b613

📥 Commits

Reviewing files that changed from the base of the PR and between 1e9c2f1 and 4528d9e.

📒 Files selected for processing (27)

app/build.gradle.kts
app/src/main/java/com/nextcloud/talk/chat/CameraLens.kt
app/src/main/java/com/nextcloud/talk/chat/ChatActivity.kt
app/src/main/java/com/nextcloud/talk/chat/MessageInputFragment.kt
app/src/main/java/com/nextcloud/talk/chat/MessageInputVoiceRecordingFragment.kt
app/src/main/java/com/nextcloud/talk/chat/RecordButtonGesture.kt
app/src/main/java/com/nextcloud/talk/chat/RecordHintPopup.kt
app/src/main/java/com/nextcloud/talk/chat/RecordInputMode.kt
app/src/main/java/com/nextcloud/talk/chat/RecordingStopCoordinator.kt
app/src/main/java/com/nextcloud/talk/chat/VideoMessageRecorder.kt
app/src/main/java/com/nextcloud/talk/chat/viewmodels/ChatViewModel.kt
app/src/main/java/com/nextcloud/talk/utils/preferences/AppPreferences.java
app/src/main/java/com/nextcloud/talk/utils/preferences/AppPreferencesImpl.kt
app/src/main/res/drawable/bg_record_hint.xml
app/src/main/res/drawable/ic_record_hint_arrow.xml
app/src/main/res/layout/activity_chat.xml
app/src/main/res/layout/view_message_input.xml
app/src/main/res/layout/view_record_hint.xml
app/src/main/res/values/dimens.xml
app/src/main/res/values/strings.xml
app/src/test/java/com/nextcloud/talk/chat/RecordButtonGestureTest.kt
app/src/test/java/com/nextcloud/talk/chat/RecordHintPopupPlacementTest.kt
app/src/test/java/com/nextcloud/talk/chat/RecordingStopCoordinatorTest.kt
app/src/test/java/com/nextcloud/talk/chat/ShouldVibrateOnRecordingChangeTest.kt
app/src/test/java/com/nextcloud/talk/chat/VideoMessageRecordingTest.kt
app/src/test/java/com/nextcloud/talk/chat/VideoRecorderLifecycleTest.kt
app/src/test/java/com/nextcloud/talk/chat/VideoRecordingRecreationTest.kt

Included review availability: This review used your included allowance. Your plan provides up to 4 included reviews per hour; 1 remain after this review.

mahibi · 2026-10-05T08:21:06Z

Hi @ToteMeiSter thank you for your contributions.
We will review them soon

coderabbitai

Actionable comments posted: 2

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI
Review profile: CHILL
Plan: Advanced
Run ID: 1db88cfb-cd9e-4493-9d4e-d139a302b51f

📥 Commits

Reviewing files that changed from the base of the PR and between 4528d9e and 2e9ec35.

📒 Files selected for processing (16)

app/src/main/java/com/nextcloud/talk/chat/ChatActivity.kt
app/src/main/java/com/nextcloud/talk/chat/MessageInputFragment.kt
app/src/main/java/com/nextcloud/talk/chat/MessageInputVoiceRecordingFragment.kt
app/src/main/java/com/nextcloud/talk/chat/RecordButtonGesture.kt
app/src/main/java/com/nextcloud/talk/chat/VideoMessageRecorder.kt
app/src/main/java/com/nextcloud/talk/chat/VideoPreviewPlacement.kt
app/src/main/res/drawable/video_recording_dot.xml
app/src/main/res/layout/activity_chat.xml
app/src/main/res/layout/fragment_message_input_voice_recording.xml
app/src/main/res/values/colors.xml
app/src/main/res/values/dimens.xml
app/src/test/java/com/nextcloud/talk/chat/RecordButtonGestureTest.kt
app/src/test/java/com/nextcloud/talk/chat/VideoFrameAspectTest.kt
app/src/test/java/com/nextcloud/talk/chat/VideoPreviewPlacementCompactPanelTest.kt
app/src/test/java/com/nextcloud/talk/chat/VideoPreviewPlacementTest.kt
app/src/test/java/com/nextcloud/talk/chat/VideoRecordingRecreationTest.kt

🚧 Files skipped from review as they are similar to previous changes (2)

app/src/test/java/com/nextcloud/talk/chat/RecordButtonGestureTest.kt
app/src/main/java/com/nextcloud/talk/chat/MessageInputFragment.kt

Included review availability: This review used your included allowance. Your plan provides up to 4 included reviews per hour; 3 remain after this review.

ToteMeiSter · 2026-10-05T10:19:49Z

Update: four commits were added on top of the original change. They refine the video message UI of this PR and were tested on a device (Huawei DEL-LX9) with a fork build:

5af90c0 — dim the chat behind the recording and fit the preview to the recorded frame;
0367437 — center the preview, follow the screen orientation, hide the chat from TalkBack while recording;
c0587e2 — one-line recording panel for video messages with a larger preview;
2e9ec35 — review fixes of that panel.

They stay in this PR instead of a separate one because they change the same recording UI that this PR introduces; a follow-up PR on top of an unmerged one would need the same code reviewed twice. The CodeRabbit findings are fixed in 9447da7, 18b8f2a and 59de428 (answers in the threads). All commits are carried up the stack into #6819 and #6820.

This comment was prepared with the help of an AI assistant (Claude Code) and reviewed before posting.

ToteMeiSter · 2026-10-05T14:43:10Z

@coderabbitai review

coderabbitai · 2026-10-05T14:43:18Z

✅ Action performed

Review finished.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

Copilot

🟡 Changes recommended

Recording finalization and cleanup can lose accepted sends, disrupt subsequent recordings, or deliver results before the chat is ready.

6 open findings

What changed in this PR

Adds inline video messages to Nextcloud Talk’s chat input, using the existing recording controls and attachment upload flow.

Changes:

Adds persistent voice/video mode selection and hold-to-record gestures.
Introduces CameraX recording with live preview, camera switching, and rotation handling.
Includes voice-recording lifecycle fixes, UI resources, and recording helper tests.

File	Description
app/src/test/java/com/nextcloud/talk/chat/VideoRecordingRecreationTest.kt	Tests recording restoration and outcomes.
app/src/test/java/com/nextcloud/talk/chat/VideoRecorderLifecycleTest.kt	Tests recorder-owned lifecycle.
app/src/test/java/com/nextcloud/talk/chat/VideoPreviewPlacementTest.kt	Tests preview sizing.
app/src/test/java/com/nextcloud/talk/chat/VideoPreviewPlacementCompactPanelTest.kt	Tests compact-panel preview sizing.
app/src/test/java/com/nextcloud/talk/chat/VideoMessageRecordingTest.kt	Tests modes and recording outcomes.
app/src/test/java/com/nextcloud/talk/chat/VideoFrameAspectTest.kt	Tests frame orientation calculations.
app/src/test/java/com/nextcloud/talk/chat/ShouldVibrateOnRecordingChangeTest.kt	Tests vibration filtering.
app/src/test/java/com/nextcloud/talk/chat/RecordingStopCoordinatorTest.kt	Tests delayed stop coordination.
app/src/test/java/com/nextcloud/talk/chat/RecordHintPopupPlacementTest.kt	Tests hint positioning.
app/src/test/java/com/nextcloud/talk/chat/RecordButtonGestureTest.kt	Tests tap and hold classification.
app/src/main/res/values/strings.xml	Adds recording labels and hints.
app/src/main/res/values/dimens.xml	Defines preview dimensions.
app/src/main/res/values/colors.xml	Adds recording overlay color.
app/src/main/res/layout/view_record_hint.xml	Defines hint popup layout.
app/src/main/res/layout/view_message_input.xml	Adds record-button tint.
app/src/main/res/layout/fragment_message_input_voice_recording.xml	Adds video progress indicator.
app/src/main/res/layout/activity_chat.xml	Adds preview overlay and camera switch.
app/src/main/res/drawable/video_recording_dot.xml	Adds recording indicator drawable.
app/src/main/res/drawable/ic_record_hint_arrow.xml	Adds hint arrow.
app/src/main/res/drawable/bg_record_hint.xml	Adds hint background.
app/src/main/java/com/nextcloud/talk/utils/preferences/AppPreferencesImpl.kt	Persists recording mode.
app/src/main/java/com/nextcloud/talk/utils/preferences/AppPreferences.java	Exposes recording-mode preference.
app/src/main/java/com/nextcloud/talk/chat/viewmodels/ChatViewModel.kt	Owns video recorder and shared state.
app/src/main/java/com/nextcloud/talk/chat/VideoPreviewPlacement.kt	Calculates preview geometry.
app/src/main/java/com/nextcloud/talk/chat/VideoMessageRecorder.kt	Implements CameraX recording lifecycle.
app/src/main/java/com/nextcloud/talk/chat/RecordInputMode.kt	Defines voice and video modes.
app/src/main/java/com/nextcloud/talk/chat/RecordingStopCoordinator.kt	Coordinates stops during camera switching.
app/src/main/java/com/nextcloud/talk/chat/RecordHintPopup.kt	Displays anchored recording hints.
app/src/main/java/com/nextcloud/talk/chat/RecordButtonGesture.kt	Classifies recording gestures.
app/src/main/java/com/nextcloud/talk/chat/MessageInputVoiceRecordingFragment.kt	Shares locked controls with video.
app/src/main/java/com/nextcloud/talk/chat/MessageInputFragment.kt	Integrates mode switching and hold recording.
app/src/main/java/com/nextcloud/talk/chat/ChatActivity.kt	Integrates preview, permissions, restoration, and uploads.
app/src/main/java/com/nextcloud/talk/chat/CameraLens.kt	Adds camera selection and binding helpers.
app/build.gradle.kts	Adds CameraX dependencies.

🧠 Review effort: Balanced

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

coderabbitai

Actionable comments posted: 1

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI
Review profile: CHILL
Plan: Advanced
Run ID: 6a554354-5aa3-4e60-943f-416c55e3d0b9

📥 Commits

Reviewing files that changed from the base of the PR and between 59de428 and 62d3140.

📒 Files selected for processing: 7

app/src/main/java/com/nextcloud/talk/chat/ChatActivity.kt
app/src/main/java/com/nextcloud/talk/chat/MessageInputFragment.kt
app/src/main/java/com/nextcloud/talk/chat/RecordInputMode.kt
app/src/main/java/com/nextcloud/talk/chat/VideoMessageRecorder.kt
app/src/main/java/com/nextcloud/talk/chat/viewmodels/ChatViewModel.kt
app/src/test/java/com/nextcloud/talk/chat/RecordStartTest.kt
app/src/test/java/com/nextcloud/talk/chat/VideoRecordingCopilotFixesTest.kt

Included review availability: This review used your included allowance. Your plan provides up to 4 included reviews per hour; 2 remain after this review.

A short tap on the record button switches between voice and video mode. The mode is stored in AppPreferences and shown by the button icon and a snackbar hint. The result is a regular video attachment. In video mode holding the button records with CameraX (front camera, 720p with fallback, about 2.5 Mbit/s, at most 120 s, then it stops and is sent). Sliding left cancels, sliding up locks. The gestures, the lock state, the timer and the locked recording view are shared with the voice recording. The preview is shown above the input and the camera can be switched during the recording. An unfinished recording is cancelled and the camera released when the chat is paused. Too short recordings show a hint, camera errors a message. The finished mp4 goes through the regular file upload. Refs nextcloud#6812 Signed-off-by: Krainov Gleb <krajnov.g@kontentplus.ru> Assisted-by: Claude-Code:claude-opus-5-5

… popup The record button used to start a recording on the first touch, so a short tap that should switch the mode started a recording first. Recording now starts only when the button is held. A tap shows the record hint as a popup above the button instead of a snackbar, placed in the window of the button and kept above it after layout changes. Signed-off-by: Krainov Gleb <krajnov.g@kontentplus.ru> Assisted-by: Claude-Code:claude-opus-5-5

The activity is recreated on rotation while the video recorder belonged to it: onPause cancelled the recording and deleted its file, and binding the camera to the next activity made the CameraX recorder configure itself in a state where that is not allowed. - The video recorder lives in ChatViewModel like the voice recorder. The activity attaches its preview and callback, and the recorder lets go of them when the activity is destroyed. A result that arrives without an activity is delivered to the next one. - The camera is bound to a lifecycle of the recorder itself, not to the one of the activity, so it stays bound while the activity is recreated. - A new activity picks up the recording and locks it when it was held, so the timer, stop, send and cancel are shown. - The recording is cancelled when the user leaves the chat, not on a configuration change. - A recording cut off by the camera is not lost but shown in the attachment preview. - The video is recorded in the rotation of the sensor. - A recording is never stopped while a switched camera settles. Signed-off-by: Krainov Gleb <krajnov.g@kontentplus.ru> Assisted-by: Claude-Code:claude-opus-5-5

The container of the video recording preview did not consume touches, so taps on it reached the chat below. Make it consume them and hide it from TalkBack, where it is only decoration. Signed-off-by: Krainov Gleb <krajnov.g@kontentplus.ru> Assisted-by: Claude-Code:claude-opus-5-5

…g start The start time was taken on ACTION_DOWN, but the recording starts only after the 400-600 ms hold threshold, so a voice message was judged too short (or long enough) by a duration which included the hold. Take the time in beginRecording() after the recording has started and use the monotonic clock. Signed-off-by: Krainov Gleb <krajnov.g@kontentplus.ru> Assisted-by: Claude-Code:claude-opus-5-5

…alized it Send or delete of a locked video recording cleared the lock and the in-progress state at once, while CameraX was still finalizing the file. A new recording started in that window was silently dropped in beginRecording(). For video the state is now released in onVideoRecordingFinished(), which CameraX reaches for every outcome (send, cancel, error, too short). Audio, and a video recorder which is already idle, are still cleared at once. Signed-off-by: Krainov Gleb <krajnov.g@kontentplus.ru> Assisted-by: Claude-Code:claude-opus-5-5

…rded frame The locked video recording preview was a fixed 180x240dp box floating over the message bubbles, squashed to about 2:1 in landscape. Now a scrim dims the chat pane (it blocks touches and is hidden from TalkBack), and the preview is centered in it in the aspect of the recorded frame, fitted with margins, at most 75% of the width and 480dp on the longest side. The placement is a pure function (videoPreviewPlacement) with unit tests; it is recomputed when the pane changes size and when the recorder learns the frame aspect from CameraX. Signed-off-by: Krainov Gleb <krajnov.g@kontentplus.ru> Assisted-by: Claude-Code:claude-opus-5-5

…entation, hide chat from TalkBack - Center by Gravity.CENTER and change only width and height: the margin comparison never settled in RTL and re-laid out the chat every frame. - The frame aspect is turned into screen coordinates by the difference of the recording and the display rotation (screenFrameAspect), so the frame matches the picture with auto-rotate off and after a turn during the recording. - While recording, the siblings of the scrim are hidden from TalkBack and get their previous values back afterwards. - The preview size is recalculated after a camera switch. Signed-off-by: Krainov Gleb <krajnov.g@kontentplus.ru> Assisted-by: Claude-Code:claude-opus-5-5

In video mode the locked recording panel is one row: delete, red dot in a progress ring of the 120 s limit, timer, send. The voice panel is unchanged. The preview keeps a smaller gap on a short area. Signed-off-by: Krainov Gleb <krajnov.g@kontentplus.ru> Assisted-by: Claude-Code:claude-opus-5-5

- Hide the whole recording indicator from TalkBack, so the progress ring does not announce its percentage every second. - Relative padding of the timer, so the gap stays between dot and timer in RTL. - No inset around the progress ring, so it fills its 24dp. - The ring scale is set in code only (PROGRESS_MAX). Signed-off-by: Krainov Gleb <krajnov.g@kontentplus.ru> Assisted-by: Claude-Code:claude-opus-5-5

A video recording result that waited for a new activity was delivered from onCreate, before currentConversation and spreedCapabilities were set: the send path and the preview path could throw. The pending result is now delivered once the initial capabilities state has initialized both; a running recording still reattaches at once. The delete and send buttons of the compact video row announced the voice recording labels to TalkBack. Signed-off-by: Krainov Gleb <krajnov.g@kontentplus.ru> Assisted-by: Claude-Code:claude-opus-5-5

- A recording result that arrives before the chat of a recreated activity is loaded is held by the recorder and delivered once it is loaded, as the results that were already pending. - No recording of either kind starts while a video is still being finalized. Its end would clear the state and the audio focus of the new recording. - A video that was sent is finalized and uploaded even if the chat is left before CameraX has finished; clearing the view model gives the audio focus of a video recording back. - The record button switches the mode on a click as well (TalkBack, Switch Access, keyboard); a touch does not switch it twice. - Recording a video no longer asks for access to the media files, it writes to the app cache. Assisted-by: Claude-Code:claude-sonnet-5-5 Signed-off-by: Krainov Gleb <krajnov.g@kontentplus.ru>

…ding After a recreation of the activity the lock observer shows the locked recording panel at once, but the capabilities arrive later and their handler replaced the panel with the ordinary message input. The lock stayed set, so nothing brought the panel back. The handler now keeps the panel of a locked recording. Assisted-by: Claude-Code:claude-sonnet-5-5 Signed-off-by: Krainov Gleb <krajnov.g@kontentplus.ru>

The photo of the system camera is written to the app cache (createAttachmentFile) and handed over through the own file provider, so it needs no access to the media. Drop the gate in onPickCameraResult and the permission callback branch that only resumed that preview; the callback is still used by the voice recording. Assisted-by: Claude-Code:claude-sonnet-5-5 Signed-off-by: Krainov Gleb <krajnov.g@kontentplus.ru>

View.onTouchEvent posts the click of a released touch, so it ran after the touch router had reset its flag and switched the recording mode once more, e.g. right after a recording was sent. The release is now consumed by the touch listener and never reaches the view; the pressed state is cleared by hand. The click listener stays for TalkBack, Switch Access and the keyboard. Replaces RecordButtonClickRouter. Assisted-by: Claude-Code:claude-sonnet-5-5 Signed-off-by: Krainov Gleb <krajnov.g@kontentplus.ru>

…essage records sound The recording asked for the audio focus and opened the microphone in the same call. The player of another app pauses some time after it lost the focus, so the first moments of a video message contained the end of the music from the speaker. The camera and the preview are bound at once, but the recording starts only when AudioManager.isMusicActive is false, for at most one second; without music there is no delay. Stopping in the meantime ends the start as before. MusicSilenceGate is the same file as in the voice message fix. Assisted-by: Claude-Code:claude-sonnet-5-5 Signed-off-by: Krainov Gleb <krajnov.g@kontentplus.ru>

github-actions · 2026-10-19T06:22:38Z

Hello there,
Thank you so much for taking the time and effort to create a pull request to our Nextcloud project.

We hope that the review process is going smooth and is helpful for you. We want to ensure your pull request is reviewed to your satisfaction. If you have a moment, our community management team would very much appreciate your feedback on your experience with this PR review process.

Your feedback is valuable to us as we continuously strive to improve our community developer experience. Please take a moment to complete our short survey by clicking on the following link: https://cloud.nextcloud.com/apps/forms/s/i9Ago4EQRZ7TWxjfmeEpPkf6

Thank you for contributing to Nextcloud and we hope to hear from you soon!

(If you believe you should not receive this message, you can add yourself to the blocklist.)

This was referenced Oct 4, 2026

feat(chat): replace the attachment menu with a media bottom sheet #6819

Open

Attachment menu as a bottom sheet with a recent media grid and a camera tile #6814

Open

Record a video message from the chat input (hold to record, like voice messages) #6812

Open

coderabbitai Bot reviewed Oct 4, 2026

View reviewed changes

Comment thread app/src/main/java/com/nextcloud/talk/chat/MessageInputFragment.kt Outdated

ToteMeiSter mentioned this pull request Oct 4, 2026

feat(chat): Record video messages from the voice message button nextcloud/talk-ios#2745

Draft

coderabbitai Bot reviewed Oct 5, 2026

View reviewed changes

Comment thread app/src/main/java/com/nextcloud/talk/chat/ChatActivity.kt Outdated

Comment thread app/src/main/java/com/nextcloud/talk/chat/MessageInputVoiceRecordingFragment.kt

AndyScherzinger added 3. to review Waiting for reviews AI assisted labels Oct 5, 2026

ToteMeiSter mentioned this pull request Oct 5, 2026

feat(mediaviewer): show the position, add forward and draw actions to the media viewer #6823

Open

5 of 7 tasks

mahibi requested a balanced review from Copilot October 16, 2026 09:19

Copilot started reviewing on behalf of mahibi October 16, 2026 09:19

View session

Copilot AI reviewed Oct 16, 2026

View reviewed changes

coderabbitai Bot reviewed Oct 18, 2026

View reviewed changes

Comment thread app/src/main/java/com/nextcloud/talk/chat/RecordInputMode.kt Outdated

ToteMeiSter added 12 commits October 18, 2026 12:06

ToteMeiSter added 3 commits October 18, 2026 12:06

ToteMeiSter force-pushed the feat/video-message-record branch from 70c586b to 4fddff2 Compare October 18, 2026 12:39

github-actions Bot added the feedback-requested label Oct 19, 2026

Conversation

ToteMeiSter commented Oct 4, 2026

🖼️ Screenshots

Feature

Structure

How to check on a device

🚧 TODO

🏁 Checklist

🤖 AI (if applicable)

Uh oh!

coderabbitai Bot commented Oct 4, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Reviews paused

Uh oh!

coderabbitai Bot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

mahibi commented Oct 5, 2026

Uh oh!

coderabbitai Bot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

ToteMeiSter commented Oct 5, 2026

Uh oh!

ToteMeiSter commented Oct 5, 2026

Uh oh!

coderabbitai Bot commented Oct 5, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

🟡 Changes recommended

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

coderabbitai Bot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

github-actions Bot commented Oct 19, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

coderabbitai Bot commented Oct 4, 2026 •

edited

Loading

coderabbitai Bot commented Oct 5, 2026 •

edited

Loading