Page MenuHomePhabricator

[BUG] Invisible icons on toolbar when trying to long press and select texts in <= API 21
Closed, ResolvedPublic

Description

Steps to reproduce

  1. Run app in <= API 21 devices
  2. Read any article and try to long press to select texts

Expected

Able to see action buttons on the tool bar

Actual

Unable to see action buttons on the tool bar

1.png (1×700 px, 553 KB)

Event Timeline

Restricted Application added a subscriber: Aklapper. · View Herald Transcript
cooltey renamed this task from [BUG] Invisible icons on toolbar when trying to long press and select texts in < API 21 to [BUG] Invisible icons on toolbar when trying to long press and select texts in <= API 21.Nov 16 2017, 7:23 AM
cooltey updated the task description. (Show Details)

Change 391759 had a related patch set uploaded (by Cooltey; owner: Cooltey):
[apps/android/wikipedia@master] Fix: Invisible toolbar icons in <= API21 - Update some icons with themed attribute - For v19 and v21 styles, srcCompact and android:src cannot be existed in the same time (otherwise will be crashed or does not work)

https://gerrit.wikimedia.org/r/391759

Change 391759 merged by jenkins-bot:
[apps/android/wikipedia@master] Fix: Invisible toolbar icons in <= API21 - Update some icons with themed attribute - For v19 and v21 styles, srcCompact and android:src cannot be existed in the same time (otherwise will be crashed or does not work)

https://gerrit.wikimedia.org/r/391759

ABorbaWMF subscribed.

Tested on 2.6.206-alpha-2017-11-29

I am not seeing the issue on the Pixel with Android 8.0, but I am able to reproduce it on the Nexus 4 with Android
Pixel

Screenshot_20171129-114808.png (1×1 px, 582 KB)

Nexus 4
Screenshot_2017-11-29-11-47-49.png (1×768 px, 261 KB)

@ABorbaWMF
Thanks for the testing. Looks like you have different icons when selecting texts.

Could you please provide the Android version of the Nexus 4?

Yes, it is on Android 5.1.1. I am trying to update it now, but it does not seem to want to update.

@ABorbaWMF Thanks, I will check it on Android 5.1 devices

Tested on AWS cloud testing service on the following devices:

  1. Samsung Galaxy S6 (Verizon) - Android 5.02=> able to reproduce
  2. Samsung Galaxy Note 4 (AT&T) - Android 5.01 => not able to reproduce

Updated:

Test again on Samsung Galaxy S6 (Verizon) - Android 5.02, this time I am not able to reproduce.

Not sure what kind of situation will let the issue happen.

@ABorbaWMF
(No rush) If it is possible, could you please test it again on Nexus 4? (restart the phone and wipe the app)

@cooltey Just to put in a passer by observation. Testing this on the latest alpha build (build dated 30-03-2018) on a Samsung Galaxy j2 running Android 5.1.1, I still observe the text selection icons becoming invisible issue. To be precise, the icons are visible when you select text but become invisible when you change the selection area using the selection pointers.

Just to post another update I also observe the behaviour described in my previous comment on a Galaxy j1 ace (Android 5.1.1) when using the alpha version of the app dated 21-05-2018,

I would like to add another update. on a Galaxy j1 ace.

Redundanct in text-selection menu

There is a redundancy in the text selection menu when the menu becomes transparent. There are two-ways to Share i) using the transparent icon ii) using the menu item from 'More options' overflow menu.

  1. Version 2.7.234-beta-2018-05-30

    The transparent icon triggers the text sharing alone.
    Screenshot_2018-06-09-00-13-06.png (800×480 px, 112 KB)
    While the one hidden in the 'More options' menu triggers the usual image-cum-text sharing.
    Screenshot_2018-06-09-00-23-38.png (800×480 px, 81 KB)
  1. Version 2.7.234-alpha-2018-06-07
    • In the File: pages in particular, only the transparent icon seems to work. The menu item doesn't work. The transparent icon triggers the text-sharing menu. The overflow-menu item doesn't trigger the image-cum-text sharing menu.
    • In the File: pages in particular, the 'Define' overflow menu doesn't work. So, I'm unable to see definitions for words when I'm in File: pages. This doesn't happen in the beta app.