JackMcGrath / slms

0 stars 0 forks source link

[MEDIUM] Teacher. Attendance #67

Closed weavora closed 11 years ago

weavora commented 11 years ago

@getzerebral we are about to work on attendance logic from tomorrow and I'd like to discuss a few things with you before that.

  1. I assume this is available for teacher only, students should not see this page? (just to confirm)
  2. On the right hand side i suggest to have same calendar for easy navigation. We may rethink this approach in case we'd have more columns and space shortage
  3. How often comments will be used? Do we need more space for them or one line text field with tooltip on hover will be enough?
  4. The typical scenario should be as follows:
    1. Teacher click "Attendance" page
    2. We show's editable form for today by default (no matter if it was filled earlier today, i assume teacher can correct any previous entry and update it, correct?)
    3. Teacher set class attendance
    4. Teacher click "Submit attendance button"
    5. ?? After page refresh we show a notification that form was successfully saved. Should we redirect teacher to some other page?
  5. I assume in future we may have "Journal" logic so that teacher could see class attendance for past week or month. What do you think?
  6. I was thinking if we need a separate UI for edit attendance and preview past entries? Maybe teacher should not be able to change attendance for yesterday or earlier? Maybe after attendance submit we should show another UI without checkboxes? What do you think?

1360ad565db1476dbdd9a765ed8081d3

JackMcGrath commented 11 years ago

@weavora

  1. Teachers are the only users that need this level of access. I could see students and parents benefitting from simple reporting (we can talk about this later as its not critical at this point)
  2. Not sure if the calendar will be required for attendance entry (the date selector should be enough) maybe use the calendar for the journal for easy date changes.
  3. I'm not fully sure how much room we will need here or how often they will be used. If we remove the calendar we should have some more space though.
  4. It all sounds good, nice and easy. 4.2 correct 4.5 I'm don't think a redirect is necessary
  5. I think this a great idea
  6. I think teachers will still need to edit for past events (just in case of an entry error) I think that the journal would work well as the layout is now without the checkboxes and the single date picker. It's is where the calendar comes in useful for changing dates easily.
weavora commented 11 years ago

@alex-shevchuk lets start with the following:

staeron commented 11 years ago

Looks good

JackMcGrath commented 11 years ago

@weavora The only issue I see is that in using radio buttons vs. checkboxes is that we cannot record if a student is absent. I see either two options. Use a checkbox on Present/Absent or more fitting add an absent option radio button for selection.

weavora commented 11 years ago

Few UI corrections

Screenshot 1:28:13 10:29 AM

alex-shevchuk commented 11 years ago

TODO:

staeron commented 11 years ago

Looks good