adding some of the tooltips#150
Conversation
📝 WalkthroughWalkthroughAdds conditional rendering for the "Comparison matrix" help text: the hint "A value of −1 represents the control group, and a value of 1 represents the treatment group" is shown only when Changes
Sequence Diagram(s)(omitted — changes are a small UI conditional tweak without multi-component sequential flow) Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 inconclusive)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches🧪 Generate unit tests (beta)
No actionable comments were generated in the recent review. 🎉 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. Comment |
|
Failed to generate code suggestions for PR |
to only apply to response curves
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Fix all issues with AI agents
In `@R/module-statmodel-server.R`:
- Around line 679-691: Remove the stray leading space in the tooltip string so
it reads "A value of −1 represents the control group…" (update the string in the
block that checks NAMESPACE_STATMODEL$comparison_mode against
CONSTANTS_STATMODEL$comparison_mode_all_pairwise,
CONSTANTS_STATMODEL$comparison_mode_all_vs_one, and
CONSTANTS_STATMODEL$comparison_mode_custom_pairwise). Also, since the table is
editable per editable_options (see editable_options handling near lines
660–667), add the same "This table is interactive. Click values to edit." hint
(from the response_curve branch that checks
CONSTANTS_STATMODEL$comparison_mode_response_curve) to these pairwise/all-vs-one
branches so users are informed in all editable modes.
|
Caution Failed to replace (edit) comment. This is likely due to insufficient permissions or the comment being deleted. Error details |
1 similar comment
|
Caution Failed to replace (edit) comment. This is likely due to insufficient permissions or the comment being deleted. Error details |
adding tool tips for the matrix
Summary by CodeRabbit