Skip to content

Improve pull request conversations#47

Merged
danielss-dev merged 2 commits into
mainfrom
developments/pr-conversation-comments
Jul 13, 2026
Merged

Improve pull request conversations#47
danielss-dev merged 2 commits into
mainfrom
developments/pr-conversation-comments

Conversation

@danielss-dev

Copy link
Copy Markdown
Owner

Summary

  • redesign the hosted pull-request conversation as an avatar-led review timeline
  • add a keyboard-operable Markdown composer with Write/Preview modes, formatting actions, character count, and clearer submission feedback
  • support hosted screenshot/image Markdown with explicit click-to-load rendering
  • show GitHub and Azure DevOps profile avatars with initials as a no-layout-shift fallback
  • update the task tracker, roadmap, README, user guide, and durable implementation guidance

Why

The existing conversation view was visually sparse and offered only a plain textarea. Review discussions now have clearer author/context separation and a composer that supports the Markdown workflows people expect without leaving Strand.

User impact

Comments are easier to scan and compose. Provider avatars load lazily with no referrer; arbitrary images in PR content remain inert until the user explicitly reveals them. Direct local binary attachment upload is intentionally not presented because the supported GitHub and Azure CLI paths do not share a stable attachment API.

Validation

  • cargo check -p strand-core -p strand-tauri
  • cargo test -p strand-tauri (25 passed)
  • pnpm --filter ./ui exec vitest run (212 passed)
  • pnpm --filter ./ui exec tsc --noEmit

@danielss-dev
danielss-dev marked this pull request as ready for review July 13, 2026 17:35
@danielss-dev
danielss-dev merged commit dda45ec into main Jul 13, 2026
2 checks passed
@danielss-dev
danielss-dev deleted the developments/pr-conversation-comments branch July 14, 2026 13:52
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.

1 participant