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

Issue 29430593: Issue 5192 - Add notification for users without Samsung Internet (Closed)

Created:
May 5, 2017, 2:50 p.m. by jens
Modified:
May 9, 2017, 3:45 p.m.
Reviewers:
diegocarloslima
CC:
Felix Dahlke
Visibility:
Public.

Description

Issue 5192 - Add notification for users without Samsung Internet

Patch Set 1 #

Total comments: 2

Patch Set 2 : Removed CDATA section from string resource #

Unified diffs Side-by-side diffs Delta from patch set Stats (+3 lines, -3 lines) Patch
M adblockplussbrowser/res/values/strings.xml View 1 1 chunk +3 lines, -3 lines 0 comments Download

Messages

Total messages: 4
jens
May 5, 2017, 3:21 p.m. (2017-05-05 15:21:42 UTC) #1
diegocarloslima
https://codereview.adblockplus.org/29430593/diff/29430594/adblockplussbrowser/res/values/strings.xml File adblockplussbrowser/res/values/strings.xml (right): https://codereview.adblockplus.org/29430593/diff/29430594/adblockplussbrowser/res/values/strings.xml#newcode24 adblockplussbrowser/res/values/strings.xml:24: <string name="sbrowser_dialog_message"><![CDATA[Before you can use Adblock Plus, you must ...
May 5, 2017, 5:46 p.m. (2017-05-05 17:46:25 UTC) #2
jens
https://codereview.adblockplus.org/29430593/diff/29430594/adblockplussbrowser/res/values/strings.xml File adblockplussbrowser/res/values/strings.xml (right): https://codereview.adblockplus.org/29430593/diff/29430594/adblockplussbrowser/res/values/strings.xml#newcode24 adblockplussbrowser/res/values/strings.xml:24: <string name="sbrowser_dialog_message"><![CDATA[Before you can use Adblock Plus, you must ...
May 9, 2017, 1:50 p.m. (2017-05-09 13:50:27 UTC) #3
diegocarloslima
May 9, 2017, 1:53 p.m. (2017-05-09 13:53:23 UTC) #4
On 2017/05/09 13:50:27, jens wrote:
>
https://codereview.adblockplus.org/29430593/diff/29430594/adblockplussbrowser...
> File adblockplussbrowser/res/values/strings.xml (right):
> 
>
https://codereview.adblockplus.org/29430593/diff/29430594/adblockplussbrowser...
> adblockplussbrowser/res/values/strings.xml:24: <string
> name="sbrowser_dialog_message"><![CDATA[Before you can use Adblock Plus, you
> must install Samsung Internet 4.0 or above.]]></string>
> On 2017/05/05 17:46:25, diegocarloslima wrote:
> > Since the html tags were removed, we can remove the CDATA section
> 
> Acknowledged.

LGTM

Powered by Google App Engine
This is Rietveld