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

Issue 82041: Melange Surveys: New and updated JS and CSS 'upstream' files.

Can't Edit
Can't Publish+Mail
Start Review
Created:
14 years, 10 months ago by ajaksu
Modified:
8 years, 4 months ago
Reviewers:
Mario Ferraro
Visibility:
Public.

Patch Set 1 #

Total comments: 10

Patch Set 2 : Moved and renamed files, replaced packed file. #

Total comments: 2

Patch Set 3 : Renamed the growfield file. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+3727 lines, -43 lines) Patch
A app/jquery/jquery-editable-1.3.3.js View 1 chunk +161 lines, -0 lines 0 comments Download
A app/jquery/jquery-growfield.js View 1 chunk +581 lines, -0 lines 0 comments Download
M app/jquery/jquery-ui.core.js View 13 chunks +42 lines, -43 lines 0 comments Download
A app/jquery/jquery-ui.dialog.js View 1 chunk +607 lines, -0 lines 0 comments Download
A app/jquery/jquery-ui.draggable.js View 1 chunk +711 lines, -0 lines 0 comments Download
A app/jquery/jquery-ui.sortable.js View 1 chunk +1015 lines, -0 lines 0 comments Download
A app/soc/content/css/ui.all.css View 1 chunk +610 lines, -0 lines 0 comments Download

Messages

Total messages: 4
Mario Ferraro
All should be moved in the right directories (following current naming path), and also don't ...
14 years, 10 months ago (2009-06-21 10:35:47 UTC) #1
ajaksu
Thanks a lot for the review, Mario! On 2009/06/21 10:35:47, Mario Ferraro wrote: > All ...
14 years, 10 months ago (2009-06-21 20:48:55 UTC) #2
Mario Ferraro
http://codereview.appspot.com/82041/diff/12/1016 File app/jquery/jquery.growfield.js (right): http://codereview.appspot.com/82041/diff/12/1016#newcode1 Line 1: /* Name should be jquery-growfield.js, not jquery.growfield.js :) ...
14 years, 10 months ago (2009-06-23 16:28:42 UTC) #3
ajaksu
14 years, 10 months ago (2009-06-26 14:56:15 UTC) #4
I think we should merge as-is, then rename to versioned on next update if we go
the version in filenames way :)

http://codereview.appspot.com/82041/diff/12/1016
File app/jquery/jquery.growfield.js (right):

http://codereview.appspot.com/82041/diff/12/1016#newcode1
Line 1: /*
On 2009/06/23 16:28:42, Mario Ferraro wrote:
> Name should be jquery-growfield.js, not jquery.growfield.js :)
> I think it's useful to have the version in some way, but adding it in the name
> of the file as we're doing can be quite painful to manage in the long time,
> going to mail the group about it. :)

Done.
Sign in to reply to this message.

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