Is your demo down? http://codereview.appspot.com/1095041/diff/2001/3001 File gtfsscheduleviewer/files/transit_editor.js (right): http://codereview.appspot.com/1095041/diff/2001/3001#newcode1725 gtfsscheduleviewer/files/transit_editor.js:1725: * @param {Boolean} show Show ...
The CL looks good to me, but found a couple of bugs in the demo.
1) Edit leg -> click on a leg -> sometimes the infowindow won't be opened, very
randomly. I don't know how to regularly reproduce it yet.
2) Edit leg -> drag the red point to get directions -> save all -> nothing is
saved. Refresh the page, then you will see the change is not taking effect.
3) Edit leg -> drag the red point to get directions -> generate shape -> the
generated shape will take over the previously modified leg, but the anchor point
is still there.
You can submit it now, and fix bugs in other CLs.
updated to rev 77
2010/5/5 <baiming@google.com>
> The CL looks good to me, but found a couple of bugs in the demo.
> 1) Edit leg -> click on a leg -> sometimes the infowindow won't be
> opened, very randomly. I don't know how to regularly reproduce it yet.
> 2) Edit leg -> drag the red point to get directions -> save all ->
> nothing is saved. Refresh the page, then you will see the change is not
> taking effect.
> 3) Edit leg -> drag the red point to get directions -> generate shape ->
> the generated shape will take over the previously modified leg, but the
> anchor point is still there.
>
> You can submit it now, and fix bugs in other CLs.
>
>
> http://codereview.appspot.com/1095041/show
>
Issue 1095041: enabled multi points gdirections
Created 14 years ago by calidion
Modified 7 years, 5 months ago
Reviewers: lychen, xan_google.com, weiliu, leio.chen, baiming, qiaojian, quguangfan, heian911_gmail.com
Base URL: http://scheduleeditor.googlecode.com/svn/trunk/python/
Comments: 11