Unified Diff: mobile/android/base/home/PanelLayout.java
Issue 29341383:
Issue 2606 - Firefox logo shown in background on empty Adblock Browser New Tab and Search pages (Closed)
Patch Set: Adjustments in comments
Created Nov. 21, 2016, 12:59 p.m.
Use n/p to move between diff chunks;
N/P to move between comments.
« no previous file with comments
|
« mobile/android/base/home/PanelAuthLayout.java ('k')
|
mobile/android/base/resources/drawable-hdpi/abb_icon_home_empty.png » ('j')
|
no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
| Index: mobile/android/base/home/PanelLayout.java |
| =================================================================== |
| --- a/mobile/android/base/home/PanelLayout.java |
| +++ b/mobile/android/base/home/PanelLayout.java |
| @@ -454,22 +454,23 @@ abstract class PanelLayout extends Frame |
| textView.setText(R.string.home_default_empty); |
| } else { |
| textView.setText(text); |
| } |
| final String imageUrl = (emptyViewConfig == null) ? null : emptyViewConfig.getImageUrl(); |
| final ImageView imageView = (ImageView) view.findViewById(R.id.home_empty_image); |
| + // Changed icon. See https://issues.adblockplus.org/ticket/2606 |
| if (TextUtils.isEmpty(imageUrl)) { |
| - imageView.setImageResource(R.drawable.icon_home_empty_firefox); |
| + imageView.setImageResource(R.drawable.abb_icon_home_empty); |
| } else { |
| ImageLoader.with(getContext()) |
| .load(imageUrl) |
| - .error(R.drawable.icon_home_empty_firefox) |
| + .error(R.drawable.abb_icon_home_empty) |
| .into(imageView); |
| } |
| viewState.setEmptyView(view); |
| } |
| return view; |
| } |
« no previous file with comments
|
« mobile/android/base/home/PanelAuthLayout.java ('k')
|
mobile/android/base/resources/drawable-hdpi/abb_icon_home_empty.png » ('j')
|
no next file with comments »
