fix: display selectable prompt text on detail pages - #124
Conversation
Strix Security ReviewWarning This pull request has 3 commits after the last Strix review ( No security issues found. Updated for Reviewed by Strix |
|
I noticed #128 reverted this change. Could you share whether it was a regression or a preference for a different prompt-reading interaction? I'll keep the revert intact. If there was a bug, the affected device or steps would help me reproduce it and prepare the right correction. |
|
@sridharkalaibala I reverted that feature we didn't wanted to go with that after few discussions (also this has been reverted via #128 got merged) |
|
Thanks for clarifying. Understood—this was a product decision after discussion. I'll leave the feature reverted and keep the follow-up fixes aligned with the current main branch. |
|
Thanks for all of this @sridharkalaibala , the 4 PRs I merged were genuinely good work. I can't review this many large changes at once, so I'm going to take them one at a time. I'll start with this one (#126), since it's the P0. Please hold off on claiming anything new until these are through. Going forward, please wait until I've assigned an issue to you before starting on it, especially P0 and P1 ones. #126 changes how likes, saves and follows work across 11 files, and I'd have liked to agree on the approach in #80 before the code was written. I'm adding this to CONTRIBUTING.md so it's clear for everyone. |
|
Understood, and thank you for being clear about the review workload. I’m sorry for sending several large changes before we had agreed on their approaches. I’ll stop claiming or starting new Paro issues while the current queue is reviewed, wait for explicit assignment going forward, and agree the approach first for P0/P1 or larger changes. I’ve read the updated CONTRIBUTING.md, including the two-open-PR limit. I’ll focus on answering your review of #126 and keep the other existing PRs available for your chosen order. |
Fixes #78.
Display the full prompt above the Copy button in a labelled, selectable region. Preserve line breaks, wrap long unbroken text, and constrain the block height with scrolling. The region is keyboard focusable with a visible focus ring; the content flex item can shrink at narrow widths. Copy behavior is unchanged.
Validation:
git diff --checkpasses.No live backend or production account was used. Prepared with AI assistance.