Android: Sharing: app icons are not sized equally

Created on 28 Feb 2018  路  5Comments  路  Source: nextcloud/android

Actual behaviour

The app icons in the sharing menu are not equally sized, instead they are displayed in their original size.
2 screenshots:
screenshot_20180228-110941
screenshot_20180228-112346

Expected behaviour

The app icons in the sharing menu should be equally sized.

Steps to reproduce

  1. Click 'Share' icon on any file

Environment data

Android version: 8.1.0

Device model: Google Pixel 2

Stock or customized system: stock

Nextcloud app version: 3.0.1

Nextcloud server version: 13.0.0

Logs

(n/a)

approved bug design pr exists

All 5 comments

This needs to be fixed for next release.
If anyone wants to pick it up, please feel free.
Otherwise I'll do it close to feature freeze.

@tobiasKaminsky I am not even sure what the correct size is. Looking at Twitter and Tumbler they are the same and both ship the Android 8 defined launcher icon sizes. But yeah should probably be smaller for the moment but we might have to enlarge it again at a later point in time in case more apps support Android 8 icons.

Without looking at the code, I assume that a defined size is missing, e.g. 32dp.

I will look into it and try my best ;)

@tobiasKaminsky
Can you explain me how the images are set there without an "< ImageView/>"
The library for that is still there, but never declared. I waysted a lot of time (confused about the 2nd system share layout [pr exist] and the missing ImageView.
Maybe you can give me a hint and I can learn more about android ;)
Edit: No solution found :D Maybe someone can give me hints.
image is set via sendButton.setCompoundDrawablesWithIntrinsicBounds(null, item.getDrawable(), null, null);
in SendButtonAdapter

Was this page helpful?
0 / 5 - 0 ratings