Skip to content

fix: remove hardcoded 700px width from question editing distribution chart#4335

Merged
aseckin merged 5 commits intomainfrom
claude/issue-3655-fix-question-editing-ui-for
Feb 19, 2026
Merged

fix: remove hardcoded 700px width from question editing distribution chart#4335
aseckin merged 5 commits intomainfrom
claude/issue-3655-fix-question-editing-ui-for

Conversation

@aseckin
Copy link
Contributor

@aseckin aseckin commented Feb 12, 2026

Removes the hardcoded style={{ width: 700 }} from the distribution chart preview container in numeric_question_input.tsx. The chart now fills its parent container responsively, fixing the broken layout on narrow screens.

Closes #3655

Generated with Claude Code

Summary by CodeRabbit

  • Style
    • Improved the example input chart layout flexibility by removing a fixed width constraint, allowing better responsiveness across different screen sizes.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 12, 2026

📝 Walkthrough

Walkthrough

Removed a fixed 700px width container wrapping the Example input chart in the numeric question input component. No behavioral or logic changes; rendering flow and conditions remain unchanged.

Changes

Cohort / File(s) Summary
UI Layout Adjustment
front_end/src/app/(main)/questions/components/numeric_question_input.tsx
Removed fixed 700px width container around the Example input chart to improve responsive behavior on narrow screens.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Suggested reviewers

  • elisescu
  • lsabor
  • hlbmtc

Poem

🐰 A rigid constraint fell away,
Like carrots on a narrow tray,
The width that bound now set it free,
On small screens, responsive glee!

🚥 Pre-merge checks | ✅ 4
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately and specifically describes the main change: removing a hardcoded 700px width from the distribution chart in question editing.
Linked Issues check ✅ Passed The PR directly addresses the primary objective from issue #3655 by removing the hardcoded 700px width from the numeric question input chart, fixing the broken UI on narrow screens.
Out of Scope Changes check ✅ Passed The change is narrowly scoped to removing the fixed width constraint from the distribution chart in numeric_question_input.tsx, with no unrelated modifications present.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch claude/issue-3655-fix-question-editing-ui-for

No actionable comments were generated in the recent review. 🎉

Tip

Issue Planner is now in beta. Read the docs and try it out! Share your feedback on Discord.


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 and usage tips.

@github-actions
Copy link
Contributor

github-actions bot commented Feb 12, 2026

Cleanup: Preview Environment Removed

The preview environment for this PR has been destroyed.

Resource Status
🌐 Preview App Deleted
🗄️ PostgreSQL Branch Deleted
⚡ Redis Database Deleted
🔧 GitHub Deployments Removed
📦 Docker Image Retained (auto-cleanup via GHCR policies)

Cleanup triggered by PR close at 2026-02-19T11:51:15Z

@aseckin
Copy link
Contributor Author

aseckin commented Feb 12, 2026

this is super straightforward but tagged you anyway @elisescu

Copy link
Contributor

@elisescu elisescu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@aseckin aseckin merged commit 0736985 into main Feb 19, 2026
12 of 13 checks passed
@aseckin aseckin deleted the claude/issue-3655-fix-question-editing-ui-for branch February 19, 2026 11:51
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.

Fix Question Editing UI for narrow screens

2 participants

Comments