Bug 1379066
- 4. Readermode icon use new resource r=jwu
In Photon, the reader-mode icon should support private mode. Therefore we rename resource in Australis to *ic_readermode* for consistent. MozReview-Commit-ID: 1ejwk3OfUex --HG-- rename : mobile/android/app/src/main/res/drawable-hdpi/reader.png => mobile/android/app/src/australis/res/drawable-hdpi/ic_readermode.png rename : mobile/android/app/src/main/res/drawable-hdpi/reader_active.png => mobile/android/app/src/australis/res/drawable-hdpi/ic_readermode_on.png rename : mobile/android/app/src/main/res/drawable-xhdpi/reader.png => mobile/android/app/src/australis/res/drawable-xhdpi/ic_readermode.png rename : mobile/android/app/src/main/res/drawable-xhdpi/reader_active.png => mobile/android/app/src/australis/res/drawable-xhdpi/ic_readermode_on.png extra : rebase_source : 15e62aafe7e895805db10b3d7d7bfb4a641be123
До Ширина: | Высота: | Размер: 315 B После Ширина: | Высота: | Размер: 315 B |
До Ширина: | Высота: | Размер: 316 B После Ширина: | Высота: | Размер: 316 B |
До Ширина: | Высота: | Размер: 355 B После Ширина: | Высота: | Размер: 355 B |
До Ширина: | Высота: | Размер: 360 B После Ширина: | Высота: | Размер: 360 B |
|
@ -0,0 +1,15 @@
|
|||
<?xml version="1.0" encoding="utf-8"?>
|
||||
|
||||
<!-- This Source Code Form is subject to the terms of the Mozilla Public
|
||||
- License, v. 2.0. If a copy of the MPL was not distributed with this
|
||||
- file, You can obtain one at http://mozilla.org/MPL/2.0/. -->
|
||||
|
||||
<selector xmlns:android="http://schemas.android.com/apk/res/android" xmlns:gecko="http://schemas.android.com/apk/res-auto">
|
||||
|
||||
<!-- private browsing mode -->
|
||||
<item android:drawable="@drawable/ic_readermode_pm" gecko:state_private="true" />
|
||||
|
||||
<!-- normal mode -->
|
||||
<item android:drawable="@drawable/ic_readermode_nm" />
|
||||
|
||||
</selector>
|
|
@ -44,12 +44,12 @@ final class UnusedResourcesUtil {
|
|||
R.drawable.casting_active,
|
||||
R.drawable.close,
|
||||
R.drawable.homepage_banner_firstrun,
|
||||
R.drawable.ic_readermode,
|
||||
R.drawable.ic_readermode_on,
|
||||
R.drawable.icon_openinapp,
|
||||
R.drawable.pause,
|
||||
R.drawable.phone,
|
||||
R.drawable.play,
|
||||
R.drawable.reader,
|
||||
R.drawable.reader_active,
|
||||
R.drawable.sync_promo,
|
||||
R.drawable.undo_button_icon,
|
||||
};
|
||||
|
|
|
@ -179,7 +179,7 @@ var Reader = {
|
|||
|
||||
let browser = tab.browser;
|
||||
if (browser.currentURI.spec.startsWith("about:reader")) {
|
||||
showPageAction("drawable://reader_active", Strings.reader.GetStringFromName("readerView.close"));
|
||||
showPageAction("drawable://ic_readermode_on", Strings.reader.GetStringFromName("readerView.close"));
|
||||
// Only start a reader session if the viewer is in the foreground. We do
|
||||
// not track background reader viewers.
|
||||
UITelemetry.startSession("reader.1", null);
|
||||
|
@ -193,7 +193,7 @@ var Reader = {
|
|||
UITelemetry.stopSession("reader.1", "", null);
|
||||
|
||||
if (browser.isArticle) {
|
||||
showPageAction("drawable://reader", Strings.reader.GetStringFromName("readerView.enter"));
|
||||
showPageAction("drawable://ic_readermode", Strings.reader.GetStringFromName("readerView.enter"));
|
||||
UITelemetry.addEvent("show.1", "button", null, "reader_available");
|
||||
this._sendMmaEvent("reader_available");
|
||||
} else {
|
||||
|
|