Issue 548:Remove Calendar Popup and replace with jQuery UI

Issue 548:Remove Calendar Popup and replace with jQuery UI

[damith, 2014-03-28 10:24:59]

https://codereview.appspot.com/81500043/diff/20001/src/main/webapp/stylesheet...
File src/main/webapp/stylesheets/datepicker.css (right):

https://codereview.appspot.com/81500043/diff/20001/src/main/webapp/stylesheet...
src/main/webapp/stylesheets/datepicker.css:1: /* DatePicker Container */
Do we need this file?

[franklingujunchao, 2014-03-28 11:15:13]

Empty?
https://codereview.appspot.com/81500043/patch/20001/30012
this file right?




Gu Junchao
Major in Computer Engineering, National University of Singapore
Primary Email: *franklingujunchao@gmail.com <franklingujunchao@gmail.com>*
NUS Email: *A0105750@nus.edu.sg <A0105750@nus.edu.sg>*




On Fri, Mar 28, 2014 at 6:25 PM, <damith@gmail.com> wrote:

>
> https://codereview.appspot.com/81500043/diff/20001/src/
> main/webapp/stylesheets/datepicker.css
> File src/main/webapp/stylesheets/datepicker.css (right):
>
> https://codereview.appspot.com/81500043/diff/20001/src/
> main/webapp/stylesheets/datepicker.css#newcode1
> src/main/webapp/stylesheets/datepicker.css:1: /* DatePicker Container */
> Do we need this file?
>
> https://codereview.appspot.com/81500043/
>

[damith, 2014-03-28 11:17:43]

Yes, in the code review system it shows only one comment line. May be
something wrong with the code review system.


On Fri, Mar 28, 2014 at 7:15 PM, Junchao gu <franklingujunchao@gmail.com>wrote:

> Empty?
> https://codereview.appspot.com/81500043/patch/20001/30012
> this file right?
>
>
>
>
> Gu Junchao
> Major in Computer Engineering, National University of Singapore
> Primary Email: *franklingujunchao@gmail.com <franklingujunchao@gmail.com>*
> NUS Email: *A0105750@nus.edu.sg <A0105750@nus.edu.sg>*
>
>
>
>
> On Fri, Mar 28, 2014 at 6:25 PM, <damith@gmail.com> wrote:
>
>>
>> https://codereview.appspot.com/81500043/diff/20001/src/
>> main/webapp/stylesheets/datepicker.css
>> File src/main/webapp/stylesheets/datepicker.css (right):
>>
>> https://codereview.appspot.com/81500043/diff/20001/src/
>> main/webapp/stylesheets/datepicker.css#newcode1
>> src/main/webapp/stylesheets/datepicker.css:1: /* DatePicker Container */
>> Do we need this file?
>>
>> https://codereview.appspot.com/81500043/
>>
>
>

[damith, 2014-03-28 11:18:54]

It is shown as empty in the 'side by side' view.


On Fri, Mar 28, 2014 at 7:17 PM, Damith C. Rajapakse <damith@gmail.com>wrote:

> Yes, in the code review system it shows only one comment line. May be
> something wrong with the code review system.
>
>
> On Fri, Mar 28, 2014 at 7:15 PM, Junchao gu
<franklingujunchao@gmail.com>wrote:
>
>> Empty?
>> https://codereview.appspot.com/81500043/patch/20001/30012
>> this file right?
>>
>>
>>
>>
>> Gu Junchao
>> Major in Computer Engineering, National University of Singapore
>> Primary Email: *franklingujunchao@gmail.com
>> <franklingujunchao@gmail.com>*
>> NUS Email: *A0105750@nus.edu.sg <A0105750@nus.edu.sg>*
>>
>>
>>
>>
>> On Fri, Mar 28, 2014 at 6:25 PM, <damith@gmail.com> wrote:
>>
>>>
>>> https://codereview.appspot.com/81500043/diff/20001/src/
>>> main/webapp/stylesheets/datepicker.css
>>> File src/main/webapp/stylesheets/datepicker.css (right):
>>>
>>> https://codereview.appspot.com/81500043/diff/20001/src/
>>> main/webapp/stylesheets/datepicker.css#newcode1
>>> src/main/webapp/stylesheets/datepicker.css:1: /* DatePicker Container */
>>> Do we need this file?
>>>
>>> https://codereview.appspot.com/81500043/
>>>
>>
>>
>

[damith, 2014-04-01 15:10:30]

https://codereview.appspot.com/81500043/diff/70001/src/main/webapp/jsp/instru...
File src/main/webapp/jsp/instructorEvalEdit.jsp (right):

https://codereview.appspot.com/81500043/diff/70001/src/main/webapp/jsp/instru...
src/main/webapp/jsp/instructorEvalEdit.jsp:26: <script
src="//code.jquery.com/ui/1.10.4/jquery-ui.js"></script>
isn't there a minified version of this?