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.

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

All Column Headers in question tab and student data should be sortable after the teacher clicks on "Show Results" #17

Open
truedat101 opened this issue Jun 4, 2013 · 19 comments
Assignees

Comments

@truedat101
Copy link
Contributor

Please implement sortable tabs.

@ghost ghost assigned thiagofnogueira Jun 26, 2013
@truedat101
Copy link
Contributor Author

Please prioritize this before working on anything else. Idea is to have sortable column headers for the scores and ratings in the students tab

@truedat101
Copy link
Contributor Author

Presumably the student names should be sortable as well.

@thiagofnogueira
Copy link
Contributor

Let me know if I am right. Would you like the "Students Tab" to be sorted by Score in the Results phase, and the "Question Tab" sorted by "rating", right? Because in the "Questions Tab" it is always sorted by the "%Correct" column and in the "Students Tab" it is always already sorted by the "Score" column.

@truedat101
Copy link
Contributor Author

                                                                                                                                                                                                                                                                      Sent from my BlackBerry 10 smartphone.                                                                                                                                                                                                                                        From: thiagofnogueiraSent: Wednesday, July 3, 2013 6:43 AMTo: SMILEConsortium/smile_teacher_androidReply To: SMILEConsortium/smile_teacher_androidCc: truedat101Subject: Re: [smile_teacher_android] All Column Headers in question tab and student  data should be sortable after the teacher clicks on "Show Results" (#17)After show results is sent, we don't need to auto sort, so columns should be sortable.  Let me know if I am right. Would you like the "Students Tab" to be sorted  by Score in the Results phase, and the "Question Tab" sorted by "rating", right? Because in the "Questions Tab" it is always sorted by the "%Correct" column.

—Reply to this email directly or view it on GitHub.

@thiagofnogueira
Copy link
Contributor

Sorry, the answer don't came.

@thiagofnogueira
Copy link
Contributor

Fixes #17

Please, check out if you agree!

@truedat101
Copy link
Contributor Author

Charles will verify.

@thiagofnogueira
Copy link
Contributor

Ok

@chrqls
Copy link

chrqls commented Jul 3, 2013

Ok , I confirm.
Just after pressing "Show results", list of students is auto-sorted by score and list of questions is auto-sorted by average rating. The best is on the top, the crapest on the bottom.

@truedat101
Copy link
Contributor Author

Hmm, sortable columns means that if you click the column header, you should
be able to sort by any of the column headers. For names, it sorts
alphabetically (ascending/descending) and for ratings
(ascending/descending), and for scores (ascending descending). If you
click heading once, it should reverse direction, click again, back to the
other way. Does it do this?

On Wed, Jul 3, 2013 at 12:03 PM, Charles Quelos [email protected]:

Ok , I confirm.
Just after pressing "Show results", list of students is auto-sorted by
score and list of questions is auto-sorted by average rating. The best is
on the top, the crapest on the bottom.


Reply to this email directly or view it on GitHubhttps://github.com//issues/17#issuecomment-20437968
.

@chrqls
Copy link

chrqls commented Jul 12, 2013

No, it does not. After pressing "Show results", tables are sorted but there is any possibility to click on column headers.
For example, if two students have a different score, clicking several times on "Score" header won't change anything. The best score will keep the first position.
FYI, I am on dev branch.

@truedat101
Copy link
Contributor Author

I'll reopen. This pisses me off. It was clear that we wanted click on column headers to resort. commit b67bead was supposed to "fix" this.

@truedat101 truedat101 reopened this Jul 15, 2013
@ghost ghost assigned truedat101 Jul 15, 2013
@truedat101
Copy link
Contributor Author

This turned out to be extremely half-baked. @chrqls what I meant was that if I had requested sortable columns, that was not how this got implemented. I think I'd like to turn off whatever automatic sorting is there, because it is confusing. It's better to list the questions in order.

@truedat101
Copy link
Contributor Author

I think what this does is dynamically sort while the session is active. I think it sorts on the highest scored, but not clear ... For now, the desired result is that it stops sorting after we hit show results. At that point, I don't want sorting. Will leave this open, but I think there's nothing to fix before this release.

@truedat101
Copy link
Contributor Author

So I was wrong. After the show results is clicked, it switches to some other dumb sort. I think it sorts by the actual rating. Just stop sorting I think.

@truedat101
Copy link
Contributor Author

I'll fix this.

@truedat101
Copy link
Contributor Author

Ok so all this gets handled in QuestionsFragment. There is a routine called SortQuestionList() which is called. Added in comit ref: b67bead

@truedat101
Copy link
Contributor Author

Just dump the line that calls this. It's not doing what I asked for. We wanted sortable columns.

@truedat101
Copy link
Contributor Author

This is good enough for the release. Leave issue open so that someone eventually fixes this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants