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

Issue 29657633: Issue 6188 - Add-ons option is being shown (Closed)

Created:
Jan. 5, 2018, 8:41 p.m. by diegocarloslima
Modified:
Jan. 16, 2018, 4:52 p.m.
Reviewers:
anton, jens
CC:
René Jeschke
Visibility:
Public.

Description

Issue 6188 - Add-ons option is being shown

Patch Set 1 #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -4 lines) Patch
M mobile/android/app/src/main/res/menu-large/browser_app_menu.xml View 1 chunk +0 lines, -1 line 1 comment Download
M mobile/android/app/src/main/res/menu-xlarge/browser_app_menu.xml View 1 chunk +0 lines, -1 line 0 comments Download
M mobile/android/app/src/main/res/menu/browser_app_menu.xml View 1 chunk +0 lines, -1 line 0 comments Download
M mobile/android/base/java/org/mozilla/gecko/BrowserApp.java View 1 chunk +2 lines, -1 line 0 comments Download

Messages

Total messages: 3
diegocarloslima
https://codereview.adblockplus.org/29657633/diff/29657634/mobile/android/app/src/main/res/menu-large/browser_app_menu.xml File mobile/android/app/src/main/res/menu-large/browser_app_menu.xml (left): https://codereview.adblockplus.org/29657633/diff/29657634/mobile/android/app/src/main/res/menu-large/browser_app_menu.xml#oldcode101 mobile/android/app/src/main/res/menu-large/browser_app_menu.xml:101: android:visible="false" Since we're forcing add-ons to be hidden in ...
Jan. 5, 2018, 8:46 p.m. (2018-01-05 20:46:26 UTC) #1
anton
On 2018/01/05 20:46:26, diegocarloslima wrote: > https://codereview.adblockplus.org/29657633/diff/29657634/mobile/android/app/src/main/res/menu-large/browser_app_menu.xml > File mobile/android/app/src/main/res/menu-large/browser_app_menu.xml (left): > > https://codereview.adblockplus.org/29657633/diff/29657634/mobile/android/app/src/main/res/menu-large/browser_app_menu.xml#oldcode101 > ...
Jan. 9, 2018, 6:41 a.m. (2018-01-09 06:41:29 UTC) #2
jens
Jan. 10, 2018, 11:26 a.m. (2018-01-10 11:26:45 UTC) #3
On 2018/01/09 06:41:29, anton wrote:
> On 2018/01/05 20:46:26, diegocarloslima wrote:
> >
>
https://codereview.adblockplus.org/29657633/diff/29657634/mobile/android/app/...
> > File mobile/android/app/src/main/res/menu-large/browser_app_menu.xml (left):
> > 
> >
>
https://codereview.adblockplus.org/29657633/diff/29657634/mobile/android/app/...
> > mobile/android/app/src/main/res/menu-large/browser_app_menu.xml:101:
> > android:visible="false"
> > Since we're forcing add-ons to be hidden in BrowserApp.java, we can revert
the
> > changes we previously made in browser_app_menu.xml
> 
> LGTM

LGTM

Powered by Google App Engine
This is Rietveld