Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(390)

Issue 273473002: Display function signatures instead of just "Function". (Closed)

Created:
6 years, 7 months ago by Emily Fortuna
Modified:
6 years, 7 months ago
Reviewers:
Alan Knight
CC:
reviews_dartlang.org
Base URL:
https://github.com/dart-lang/dartdoc-viewer.git@master
Visibility:
Public.

Description

Display function signatures instead of just "Function". BUG= R=alanknight@google.com Committed: https://github.com/dart-lang/dartdoc-viewer/commit/7e501cd

Patch Set 1 : #

Patch Set 2 : #

Patch Set 3 : #

Unified diffs Side-by-side diffs Delta from patch set Stats (+116 lines, -11 lines) Patch
M client/lib/components/category.dart View 1 2 2 chunks +4 lines, -1 line 0 comments Download
M client/lib/components/category.html View 1 2 1 chunk +3 lines, -0 lines 0 comments Download
A client/lib/components/closure.dart View 1 chunk +46 lines, -0 lines 0 comments Download
A client/lib/components/closure.html View 1 chunk +3 lines, -0 lines 0 comments Download
M client/lib/components/library.html View 1 2 1 chunk +1 line, -1 line 0 comments Download
M client/lib/components/method.html View 1 chunk +1 line, -1 line 0 comments Download
M client/lib/components/parameters.dart View 3 chunks +18 lines, -6 lines 0 comments Download
M client/lib/components/parameters.html View 1 chunk +1 line, -1 line 0 comments Download
M client/lib/components/typedef.html View 1 2 1 chunk +5 lines, -1 line 0 comments Download
M client/lib/item.dart View 1 3 chunks +34 lines, -0 lines 0 comments Download

Messages

Total messages: 6 (0 generated)
Emily Fortuna
6 years, 7 months ago (2014-05-06 22:40:46 UTC) #1
Alan Knight
One issue. If I click on a reference to a parameter in the comment, normally ...
6 years, 7 months ago (2014-05-07 16:57:08 UTC) #2
Emily Fortuna
PTAL -- I fixed the URLs for the function parameters.
6 years, 7 months ago (2014-05-07 17:57:14 UTC) #3
Alan Knight
lgtm
6 years, 7 months ago (2014-05-07 18:09:27 UTC) #4
Alan Knight
lgtm
6 years, 7 months ago (2014-05-07 19:51:34 UTC) #5
Emily Fortuna
6 years, 7 months ago (2014-05-07 19:53:10 UTC) #6
Message was sent while issue was closed.
Committed patchset #3 manually as r7e501cd (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698