Skip to content

[Feature:TAGrading] highlight pos/neg marks w/ count up/down #4170

Merged
bmcutler merged 7 commits intomasterfrom
countUpDown
Jul 24, 2019
Merged

[Feature:TAGrading] highlight pos/neg marks w/ count up/down #4170
bmcutler merged 7 commits intomasterfrom
countUpDown

Conversation

@thef1stofhername
Copy link
Copy Markdown
Contributor

@thef1stofhername thef1stofhername commented Jul 22, 2019

What is the current behavior?

Closes #3413

What is the new behavior?

Now, when you change the count up/count down grading guideline, any component score that doesn't comply is highlighted. It becomes unhighlighted when you change it to comply.

Other information?

Testing:

  1. Sign in as instructor
  2. Go to Gradeables
  3. Click on the edit pencil
  4. Click on rubric and select a component
  5. Play around with possible scores and count up/down configurations
    If the score is negative on count up, it should be highlighted and vice versa. Changing the score to comply should remove the highlight.
    Instructor highlight 2019-07-23 15-03
  6. Sign in as TA and edit rhe grading rubric when you grade a student
    TAGrading highlight2019-07-23 11-46

@bmcutler
Copy link
Copy Markdown
Member

Please add a couple screenshots to illustrate the changes.

Also we should add some screenshots/documentation on submitty.org :)
[note: this could be a big undertaking, instructor config of ta grading is under-documented & stale]

Thanks!

@bmcutler bmcutler changed the title [Feature: Instructor/TA UI] when the grading count up/down is changed, it indicates if the previous values entered fit the grading format [Feature:TAGrading] highlight positive & negative marks w/ count up/down Jul 22, 2019
@bmcutler bmcutler changed the title [Feature:TAGrading] highlight positive & negative marks w/ count up/down [Feature:TAGrading] highlight pos/neg marks w/ count up/down Jul 22, 2019
@bmcutler bmcutler requested a review from Kevin-MM July 24, 2019 03:24
Copy link
Copy Markdown
Contributor

@Kevin-MM Kevin-MM left a comment

Choose a reason for hiding this comment

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

Works well on Firefox, but not working on Chrome =(.

@thef1stofhername
Copy link
Copy Markdown
Contributor Author

What component because I tested it in both?

@Kevin-MM
Copy link
Copy Markdown
Contributor

I tried in Future (No TAs) Homework in the Edit Gradeable page and the TA grading interface. No luck.

@thef1stofhername
Copy link
Copy Markdown
Contributor Author

ok i will go back and see what is up

@Kevin-MM Kevin-MM self-requested a review July 24, 2019 16:58
Copy link
Copy Markdown
Contributor

@Kevin-MM Kevin-MM left a comment

Choose a reason for hiding this comment

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

My mistake chrome was caching so I didn't get the updates immediately. Everything works well!

@bmcutler bmcutler merged commit abc014c into master Jul 24, 2019
@bmcutler bmcutler deleted the countUpDown branch July 24, 2019 17:08
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.

Configuration of count up/count down for ta grading

3 participants