Repository navigation
Add mobile message conversation menu and copy Concept ID - #672
Merged
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Messaging rows currently expose a separate pin/hide dialog and lack a touch menu button or copyable conversation Concept ID. This change uses the shared, mobile-width-bounded menu with open, pin/hide, participant profile, structured reference and Copy Concept ID actions.
Copy Concept ID lazily creates a private reference through the canonical concept service after verifying the authenticated actor can read the exact participant/organisation exchange. The stable per-actor reference retains the retrieval locator, copies no transcript, and is read back before success. Reuse checks source access again.
Validation: 21 targeted backend tests and 30 frontend tests passed; static JS lint and diff checks passed. Production UI modules/styles in a synthetic HTTP browser fixture passed touch, keyboard/focus, both clipboard actions and no menu overflow at 320, 448 and 1280px. Authenticated live storage and physical Android clipboard were not exercised.
Related screenshot paste investigation found the ordinary chat extraction already handles items/files, while message attachments remain separate work in #650 (mobile picker/recovery in #658). Screenshot file copies were not found in the worker actor scope; no screenshot bytes or native Android failure reproduction were available. This PR does not claim to fix that incident. No deployment requested.
Von task: #V#task_agent_64049c78fe7f3128c5f655c6073a792c