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

Issue 29334188: Issue 3023 - Returning to the homepage resets the language on adblockbrowser.org (Closed)

Created:
Jan. 21, 2016, 3:49 p.m. by juliandoucette
Modified:
Jan. 26, 2016, 10:54 a.m.
Reviewers:
saroyanm
CC:
Sebastian Noack, Thomas Greiner
Visibility:
Public.

Description

Issue 3023 - Returning to the homepage resets the language on adblockbrowser.org - Added locale to main logo href with linkify filter

Patch Set 1 #

Total comments: 2

Patch Set 2 : Changed "" to "index" #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
M templates/default.tmpl View 1 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 4
juliandoucette
Jan. 21, 2016, 3:52 p.m. (2016-01-21 15:52:37 UTC) #1
saroyanm
https://codereview.adblockplus.org/29334188/diff/29334189/templates/default.tmpl File templates/default.tmpl (right): https://codereview.adblockplus.org/29334188/diff/29334189/templates/default.tmpl#newcode35 templates/default.tmpl:35: {{""|linkify(id="logo")}} Seems like we will still need to use ...
Jan. 22, 2016, 4:26 p.m. (2016-01-22 16:26:25 UTC) #2
juliandoucette
https://codereview.adblockplus.org/29334188/diff/29334189/templates/default.tmpl File templates/default.tmpl (right): https://codereview.adblockplus.org/29334188/diff/29334189/templates/default.tmpl#newcode35 templates/default.tmpl:35: {{""|linkify(id="logo")}} On 2016/01/22 16:26:25, saroyanm wrote: > Seems like ...
Jan. 25, 2016, 2:56 p.m. (2016-01-25 14:56:05 UTC) #3
saroyanm
Jan. 25, 2016, 4:38 p.m. (2016-01-25 16:38:16 UTC) #4
LGTM

Powered by Google App Engine
This is Rietveld