Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(325)

Issue 1677053: code review 1677053: dashboard: use jQuery Autocomplete plugin from Google A... (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
15 years ago by adg
Modified:
14 years, 11 months ago
Reviewers:
CC:
rsc, golang-dev
Visibility:
Public.

Description

dashboard: add jquery autocomplete plugin

Patch Set 1 #

Patch Set 2 : code review 1677053: dashboard: use jQuery Autocomplete plugin from Google A... #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+4 lines, -4 lines) Patch
M misc/dashboard/godashboard/project-edit.html View 2 chunks +4 lines, -4 lines 1 comment Download

Messages

Total messages: 11
adg
Hello rsc (cc: golang-dev@googlegroups.com), I'd like you to review this change.
15 years ago (2010-07-02 03:07:22 UTC) #1
rsc
is there really no copy of that javascript somewhere else on google.com already that we ...
15 years ago (2010-07-02 04:46:37 UTC) #2
rsc
also add a note at the top of the file saying where it was downloaded ...
15 years ago (2010-07-02 04:47:02 UTC) #3
r2
pardon me for asking but what is a jquery autocomplete plugin and why do we ...
15 years ago (2010-07-02 04:54:04 UTC) #4
adg
On 2 July 2010 14:53, Rob 'Commander' Pike <r@google.com> wrote: > pardon me for asking ...
15 years ago (2010-07-02 05:02:57 UTC) #5
r2
On Jul 1, 2010, at 10:02 PM, Andrew Gerrand wrote: > On 2 July 2010 ...
15 years ago (2010-07-02 06:34:01 UTC) #6
adg
Hello rsc (cc: golang-dev@googlegroups.com), I'd like you to review this change.
14 years, 12 months ago (2010-07-05 04:27:05 UTC) #7
adg
On 2 July 2010 16:33, Rob 'Commander' Pike <r@google.com> wrote: > > On Jul 1, ...
14 years, 12 months ago (2010-07-05 04:28:44 UTC) #8
rsc1
LGTM http://codereview.appspot.com/1677053/diff/8001/9001 File misc/dashboard/godashboard/project-edit.html (right): http://codereview.appspot.com/1677053/diff/8001/9001#newcode41 misc/dashboard/godashboard/project-edit.html:41: $('#tags').autocomplete({source:tags}); is this guaranteed to be delayed until ...
14 years, 11 months ago (2010-07-12 18:14:44 UTC) #9
adg
On 13 July 2010 04:14, <rsc@google.com> wrote: > LGTM > > > > http://codereview.appspot.com/1677053/diff/8001/9001 > ...
14 years, 11 months ago (2010-07-13 00:21:15 UTC) #10
adg
14 years, 11 months ago (2010-07-13 00:21:20 UTC) #11
*** Submitted as http://code.google.com/p/go/source/detail?r=680c935d7176 ***

dashboard: use jQuery Autocomplete plugin from Google AJAX Libs API

R=rsc
CC=golang-dev
http://codereview.appspot.com/1677053

Committer: Andrew Gerrand <adg@golang.org>
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b