Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[BUG] Batch grader button on the home screen does not work #255

Closed
charlestian23 opened this issue Aug 5, 2022 · 1 comment
Closed

[BUG] Batch grader button on the home screen does not work #255

charlestian23 opened this issue Aug 5, 2022 · 1 comment
Assignees
@wei-jun7 @littlebizaizi @BoZhiDeng
Labels
batch grader bug Something isn't working good first issue Good for newcomers ui

Comments

@charlestian23
Copy link
Collaborator

Description

The "Batch Grader" button on the home screen does not work

Steps to Reproduce

  1. Grab the most recent version of Legup from dev and compile a jar
  2. Run the jar file
  3. Click on "Batch Grader"
  4. Notice that nothing happens

Expected Behavior

This button should have the same functionality as the "Check All" button in the Legup proof editor.

Screenshots

See the "Batch Grader" button on the home screen:
image

See the "Check All" button on the Legup proof editor:
image

Puzzle Files

No response

Additional Context

No response
@charlestian23 charlestian23 added bug Something isn't working good first issue Good for newcomers batch grader ui labels Aug 5, 2022
@littlebizaizi littlebizaizi self-assigned this Aug 8, 2022
@wei-jun7 wei-jun7 self-assigned this Sep 30, 2022
@BoZhiDeng BoZhiDeng self-assigned this Oct 4, 2022
@BoZhiDeng BoZhiDeng mentioned this issue Oct 4, 2022
Closed
4 tasks
BoZhiDeng added a commit to BoZhiDeng/Legup that referenced this issue Oct 4, 2022
@BoZhiDeng
Update for issues Bram-Hub#255
7e44c7f
BoZhiDeng added a commit to BoZhiDeng/Legup that referenced this issue Oct 7, 2022
@BoZhiDeng
Update HomePanel.java
9fe44a7 
Fix Bram-Hub#255 by apply the action listener of "toolBarButtons[ToolbarName.CHECK_ALL.ordinal()]" in ProofEditorPanel
@BoZhiDeng BoZhiDeng mentioned this issue Oct 7, 2022
Merged
10 tasks
Chase-Grajeda added a commit that referenced this issue Nov 15, 2022
@Chase-Grajeda
Merge pull request #344 from wei-jun7/batch-grader
e4d04ea 
Batch grader test
Merging, closes #255 and #301
@Chase-Grajeda
Copy link
Collaborator

Fixed by #344

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@wei-jun7 wei-jun7

@littlebizaizi littlebizaizi

@BoZhiDeng BoZhiDeng

Labels
batch grader bug Something isn't working good first issue Good for newcomers ui
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@charlestian23 @Chase-Grajeda @wei-jun7 @littlebizaizi @BoZhiDeng