Skip to content

[Bugfix:InstructorUI] notebook builder MC vertical resize#12744

Merged
bmcutler merged 4 commits into
Submitty:mainfrom
tijaruS:fix/notebook-builder-multiple-choice-description-container
Apr 10, 2026
Merged

[Bugfix:InstructorUI] notebook builder MC vertical resize#12744
bmcutler merged 4 commits into
Submitty:mainfrom
tijaruS:fix/notebook-builder-multiple-choice-description-container

Conversation

@tijaruS

@tijaruS tijaruS commented Apr 4, 2026

Copy link
Copy Markdown
Contributor

Why is this Change Important & Necessary?

In the Notebook Builder UI, when creating a multiple choice question, the description box
resizes both vertically and horizontally, making the UI look inconsistent.
Using resize: vertical; fixes this issue. This is a simple CSS fix.

What is the New Behavior?

The description box in the multiple choice question now only resizes vertically.

What Steps Should a Reviewer Take to Reproduce or Test?

  1. Open the Notebook Builder UI.
  2. Create a new multiple choice question.
  3. Try resizing the description box — it should only resize vertically(after the change).

Automated Testing & Documentation

  • No automated tests required for this CSS-only fix.
  • No documentation changes needed.

Video

Before

2026-04-04.15-53-51.mp4

After

2026-04-04.16-02-03.mp4

Other Information

  • Breaking changes: No
  • Fix type: Simple CSS change

@GarvitKhandelwal31

GarvitKhandelwal31 commented Apr 4, 2026

Copy link
Copy Markdown
Contributor

I have tested this, works fine as shown in the recording of the description. The changes also looks direct and minimal.
But if I am not wrong you should include all the parts from this in your description-

Why is this Change Important & Necessary?

What is the New Behavior?

What steps should a reviewer take to reproduce or test the bug or new feature?

Automated Testing & Documentation

Other information

@tijaruS

tijaruS commented Apr 4, 2026

Copy link
Copy Markdown
Contributor Author

@GarvitKhandelwal31 Thank you for testing. I will update the description part to include all the part.

@GarvitKhandelwal31

Copy link
Copy Markdown
Contributor

One more thing you can do is that when I resize along the vertical direction the other buttons are also coming down (I mean the buttons in choice control and Up, Down, Remove).
In my opinion that should not happen as it is not looking good. Maybe you can find a way in which we can resize them without shifting the position of other buttons.

@tijaruS

tijaruS commented Apr 4, 2026

Copy link
Copy Markdown
Contributor Author

I fixed it. Now those button stays same while resizing.

Screenshots:

image

@GarvitKhandelwal31

Copy link
Copy Markdown
Contributor

Yep, the up, down, remove buttons are now fixed, but can we also choice control buttons and the whole select multiple [ I mean that whole column], because that is still flowing.

@tijaruS

tijaruS commented Apr 4, 2026

Copy link
Copy Markdown
Contributor Author

Those buttons and others also working as expected. Thank you for pointing it out.

Screenshot

image

@GarvitKhandelwal31

Copy link
Copy Markdown
Contributor

Hey, I have tested again and yes most of the buttons are fixed but the plus (or add button) is still moving.

@tijaruS

tijaruS commented Apr 5, 2026

Copy link
Copy Markdown
Contributor Author

I think that should be the desired behavior. Where the resizing of the description box ends the plus button should appear there so that user can add another mcq. Should this not be the case? what do you think?

  • image

@GarvitKhandelwal31

Copy link
Copy Markdown
Contributor

Yeah, I overlooked that. I think you are right and that should be the correct feature. Let's see what the other reviewers have in mind. Everything looks good to me.

@codecov

codecov Bot commented Apr 6, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 21.64%. Comparing base (05dae18) to head (c5cfd32).
⚠️ Report is 7 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff            @@
##               main   #12744   +/-   ##
=========================================
  Coverage     21.64%   21.64%           
  Complexity     9654     9654           
=========================================
  Files           268      268           
  Lines         36244    36244           
  Branches        487      487           
=========================================
  Hits           7845     7845           
  Misses        27916    27916           
  Partials        483      483           
Flag Coverage Δ
autograder 21.32% <ø> (ø)
js 2.04% <ø> (ø)
migrator 100.00% <ø> (ø)
php 20.66% <ø> (ø)
python_submitty_utils 80.08% <ø> (ø)
submitty_daemon_jobs 91.13% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@dagemcn dagemcn left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

The changes look good! Good work here. The only change that I would request is to link the issue this is referencing, if it exists. That way we can close it when the PR is merged.

@github-project-automation github-project-automation Bot moved this from Seeking Reviewer to Awaiting Maintainer Review in Submitty Development Apr 7, 2026
@tijaruS

tijaruS commented Apr 8, 2026

Copy link
Copy Markdown
Contributor Author

Hi! @dagemcn Thank you for reviewing it. Currently, There is no open issue regarding this.I found it out when i was examining notebook builder ui. Should i open a issue and liked with it? or its just fine?

@bmcutler
bmcutler merged commit aae4388 into Submitty:main Apr 10, 2026
24 of 25 checks passed
@github-project-automation github-project-automation Bot moved this from Awaiting Maintainer Review to Done in Submitty Development Apr 10, 2026
GarvitKhandelwal31 pushed a commit to GarvitKhandelwal31/Submitty that referenced this pull request Apr 13, 2026
…2744)

## Why is this Change Important & Necessary?

In the Notebook Builder UI, when creating a multiple choice question,
the description box
resizes both vertically and horizontally, making the UI look
inconsistent.
Using `resize: vertical;` fixes this issue. This is a simple CSS fix.

## What is the New Behavior?

The description box in the multiple choice question now only resizes
vertically.

## What Steps Should a Reviewer Take to Reproduce or Test?

1. Open the Notebook Builder UI.
2. Create a new multiple choice question.
3. Try resizing the description box — it should only resize
vertically(after the change).

## Automated Testing & Documentation

- No automated tests required for this CSS-only fix.
- No documentation changes needed.

## Video

### Before

https://github.com/user-attachments/assets/a1bf514c-ce3d-481c-a083-2ce0bc0aad9c

### After

https://github.com/user-attachments/assets/66ebd2f2-6ffb-45a4-aa89-5a9827b21af1

## Other Information

- **Breaking changes:** No
- **Fix type:** Simple CSS change

---------

Co-authored-by: dagemcn <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

4 participants