search

mit-eecs / issue-tracker

Scripts Issue Tracker
0 stars 0 forks source link

Exceptions editable in eduoffice--> planning classes --> TA offer process #71

Closed samanamarasinghe closed 11 years ago

pluskid commented 11 years ago

Currently both student.exception and student.curr_stta.course_term.staff_exception is displayed in the table cell. Which attribute shall we make editable?

pluskid commented 11 years ago

@samanamarasinghe Currently both student.exception and student.curr_stta.course_term.staff_exception is displayed in the table cell. Which attribute shall we make editable?

samanamarasinghe commented 11 years ago

Can you create an editing dialog box that displays both on separate boxes? So you can edit each separately?

-Saman

From: Chiyuan Zhang [mailto:notifications@github.com] Sent: Thursday, December 27, 2012 6:17 PM To: mit-eecs/issue-tracker Cc: samanamarasinghe Subject: Re: [issue-tracker] Exceptions editable in eduoffice--> planning classes --> TA offer process (#71)

@samanamarasinghe https://github.com/samanamarasinghe Currently both student.exception and student.curr_stta.course_term.staff_exception is displayed in the table cell. Which attribute shall we make editable?

— Reply to this email directly or view it on GitHub https://github.com/mit-eecs/issue-tracker/issues/71#issuecomment-11721025 .

https://github.com/notifications/beacon/3TegWssTPzG9ShRILOhjVYYik6DGwh1HOutG756YRdU24wAWaN-ZOTullIzWh38n.gif

pluskid commented 11 years ago

Fixed in revision 326.

samanamarasinghe commented 11 years ago

The pencil graphic is too much. Can you just leave it same as done with editing FTE?

pluskid commented 11 years ago

@samanamarasinghe Sure. But I need to put something there so that when the exception is empty string (""), we can still have something to click on.

samanamarasinghe commented 11 years ago

Can you do that only on an empty string? I just put “.”

In Planning à Summary by Class the field CEO-only Notices. The pencil is too distracting.

-Saman

From: Chiyuan Zhang [mailto:notifications@github.com] Sent: Sunday, December 30, 2012 10:02 AM To: mit-eecs/issue-tracker Cc: samanamarasinghe Subject: Re: [issue-tracker] Exceptions editable in eduoffice--> planning classes --> TA offer process (#71)

@samanamarasinghe https://github.com/samanamarasinghe Sure. But I need to put something there so that when the exception is empty string (""), we can still have something to click on.

— Reply to this email directly or view it on GitHub https://github.com/mit-eecs/issue-tracker/issues/71#issuecomment-11765260 .

https://github.com/notifications/beacon/3TegWssTPzG9ShRILOhjVYYik6DGwh1HOutG756YRdU24wAWaN-ZOTullIzWh38n.gif

pluskid commented 11 years ago

Now pencil is replaced with a dot, and only displays when the text is empty.