Brion VIBBER has submitted this change and it was merged.

Change subject: Make PageImages fetching adjust to screen density
......................................................................


Make PageImages fetching adjust to screen density

Change-Id: Id8c9078c073f11882674c28a135a4123a2449ede
---
M wikipedia/src/main/java/org/wikimedia/wikipedia/SearchArticlesFragment.java
M wikipedia/src/main/java/org/wikimedia/wikipedia/WikipediaApp.java
2 files changed, 8 insertions(+), 1 deletion(-)

Approvals:
  Brion VIBBER: Verified; Looks good to me, approved



diff --git 
a/wikipedia/src/main/java/org/wikimedia/wikipedia/SearchArticlesFragment.java 
b/wikipedia/src/main/java/org/wikimedia/wikipedia/SearchArticlesFragment.java
index 57c378f..3241121 100644
--- 
a/wikipedia/src/main/java/org/wikimedia/wikipedia/SearchArticlesFragment.java
+++ 
b/wikipedia/src/main/java/org/wikimedia/wikipedia/SearchArticlesFragment.java
@@ -56,7 +56,11 @@
             getActivity().getCurrentFocus().clearFocus();
         } else {
             searchResultsList.setVisibility(View.VISIBLE);
-            PageImagesTask imagesTask = new 
PageImagesTask(app.getAPIForSite(app.getPrimarySite()), app.getPrimarySite(), 
results, 48) {
+            PageImagesTask imagesTask = new PageImagesTask(
+                    app.getAPIForSite(app.getPrimarySite()),
+                    app.getPrimarySite(),
+                    results,
+                    (int)(48f * WikipediaApp.SCREEN_DENSITY)) {
                 @Override
                 public void onFinish(Map<PageTitle, String> result) {
                     for(Map.Entry<PageTitle, String> entry : 
result.entrySet()) {
diff --git a/wikipedia/src/main/java/org/wikimedia/wikipedia/WikipediaApp.java 
b/wikipedia/src/main/java/org/wikimedia/wikipedia/WikipediaApp.java
index 621d9d9..f89d088 100644
--- a/wikipedia/src/main/java/org/wikimedia/wikipedia/WikipediaApp.java
+++ b/wikipedia/src/main/java/org/wikimedia/wikipedia/WikipediaApp.java
@@ -14,6 +14,8 @@
     public static long SHORT_ANIMATION_DURATION;
     public static long MEDIUM_ANIMATION_DURATION;
 
+    public static float SCREEN_DENSITY;
+
     @Override
     public void onCreate() {
         super.onCreate();
@@ -21,6 +23,7 @@
 
         SHORT_ANIMATION_DURATION = 
getResources().getInteger(R.integer.config_shortAnimTime);
         MEDIUM_ANIMATION_DURATION = 
getResources().getInteger(R.integer.config_mediumAnimTime);
+        SCREEN_DENSITY = getResources().getDisplayMetrics().density;
     }
 
     public Bus getBus() {

-- 
To view, visit https://gerrit.wikimedia.org/r/96769
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Id8c9078c073f11882674c28a135a4123a2449ede
Gerrit-PatchSet: 2
Gerrit-Project: apps/android/wikipedia
Gerrit-Branch: master
Gerrit-Owner: Yuvipanda <[email protected]>
Gerrit-Reviewer: Brion VIBBER <[email protected]>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to