Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Issue 29753563: Noissue - Renamed "forward" message to "composer.forward" (Closed)

Created:
April 16, 2018, 11:32 a.m. by Sebastian Noack
Modified:
April 16, 2018, 11:49 a.m.
Reviewers:
kzar
Visibility:
Public.

Description

Noissue - Renamed "forward" message to "composer.forward"

Patch Set 1 #

Patch Set 2 : Changed message name in composer.js #

Unified diffs Side-by-side diffs Delta from patch set Stats (+6 lines, -6 lines) Patch
M composer.js View 1 1 chunk +1 line, -1 line 0 comments Download
M composer.postload.js View 4 chunks +4 lines, -4 lines 0 comments Download
M lib/filterComposer.js View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 4
Sebastian Noack
Following up on https://codereview.adblockplus.org/29752571.
April 16, 2018, 11:32 a.m. (2018-04-16 11:32:59 UTC) #1
kzar
What about adblockpluschrome/composer.js?
April 16, 2018, 11:38 a.m. (2018-04-16 11:38:38 UTC) #2
Sebastian Noack
On 2018/04/16 11:38:38, kzar wrote: > What about adblockpluschrome/composer.js? Well spotted. Done.
April 16, 2018, 11:43 a.m. (2018-04-16 11:43:33 UTC) #3
kzar
April 16, 2018, 11:45 a.m. (2018-04-16 11:45:58 UTC) #4
LGTM

Powered by Google App Engine
This is Rietveld