oppia-android icon indicating copy to clipboard operation
oppia-android copied to clipboard

Admin Controls icon color incorrect in API 19 [Blocked on #3853]

Open rt4914 opened this issue 4 years ago • 22 comments

In the screenshot we can notice that the icon color of Administrator Controls is orange even when its not selected. This needs to be fixed.

Expected behaviour: Icon + Text color when selected -> Orange Icon + Text color when not selected -> black

Take exact color code from colors.xml file and from other similar icons.

rt4914 avatar Aug 24 '21 16:08 rt4914

Hey, I think the expected API level is 21+ right?

aadityaguptaa avatar Aug 30 '21 17:08 aadityaguptaa

Hey, I think the expected API level is 21+ right?

@aadityaguptaa Actually for the minimum API version is 19.

rt4914 avatar Aug 30 '21 17:08 rt4914

Ohh okay, my bad. But the project uses OkHttp which works on API 21+. That's why my app keeps crashing.

aadityaguptaa avatar Aug 30 '21 17:08 aadityaguptaa

Ohh okay, my bad. But the project uses OkHttp which works on API 21+. That's why my app keeps crashing.

@aadityaguptaa Yes that's correct. I think I will have to solve this as there is one branch of mine in which I am not facing these crashes so I can leverage that to showcase that my solution works. @aadityaguptaa Can you pick some other issue to work on ?

rt4914 avatar Aug 30 '21 18:08 rt4914

sure

aadityaguptaa avatar Aug 30 '21 20:08 aadityaguptaa

please assign me this issue @rt4914 @bkaur-bkj

Rishavgupta12345 avatar Nov 27 '21 06:11 Rishavgupta12345

@Rishavgupta12345 Done

bkaur-bkj avatar Nov 29 '21 04:11 bkaur-bkj

can someone tell me on which file I can find this issue @rt4914

Rishavgupta12345 avatar Nov 29 '21 13:11 Rishavgupta12345

@Rishavgupta12345 this is something that you'll need to investigate. One thing that can be really helpful is to look at strings on the screen that you're trying to find the code for, then look up those strings in code (which will correspond to string IDs), and then find which layouts/Kotlin files those strings are referenced in (which will either provide the exact files responsible for that screen, or the files responsible for a constituent piece of the screen).

BenHenning avatar Dec 07 '21 20:12 BenHenning

This issue is specific to API 19 only. So please create an emulator accordingly.

rt4914 avatar Dec 08 '21 19:12 rt4914

@Rishavgupta12345 I have unassigned this issue as per your request, do let me know in case you want to pick it up again

bkaur-bkj avatar Dec 09 '21 16:12 bkaur-bkj

pl assign me this issue @rt4914 @bkaur-bkj

shubhanshu-coder avatar Jan 07 '22 11:01 shubhanshu-coder

@bkaur-bkj Can you please assign me this issue?

ArchitJain1201 avatar Jan 10 '22 17:01 ArchitJain1201

@bkaur-bkj Can you please assign me this issue?

as this issue is currently begin assigned to Shubhanshu you can try picking up other issues

bkaur-bkj avatar Jan 10 '22 17:01 bkaur-bkj

can u pl guide me how to start , very eager to contribute. currently figuring out where the issue code .

shubhanshu-coder avatar Jan 10 '22 18:01 shubhanshu-coder

can u pl guide me how to start , very eager to contribute. currently figuring out where the issue code .

@shubhanshu-coder Please checkout our wiki first. This will help you in setting up the project and once you sign the CLA and fill the survey form we can assign you issues to work on.

rt4914 avatar Jan 19 '22 10:01 rt4914

ok

shubhanshu-coder avatar Jan 27 '22 11:01 shubhanshu-coder

i have filled the CLA and the survey form . now i can start next ?

shubhanshu-coder avatar Jan 27 '22 11:01 shubhanshu-coder

@shubhanshu-coder You can find CLA here: https://github.com/oppia/oppia-android/wiki/Contributing-to-Oppia-android#onboarding-instructions

rt4914 avatar Feb 02 '22 07:02 rt4914

I have filled it now

shubhanshu-coder avatar Feb 02 '22 08:02 shubhanshu-coder

@shubhanshu-coder Are you still working on this issue or Can I take it?

ArchitJain1201 avatar Feb 10 '22 05:02 ArchitJain1201

@ArchitJain1201 Yes I am working on it.

shubhanshu-coder avatar Feb 10 '22 18:02 shubhanshu-coder

This issue seems to have been fixed, I checked the develop branch and its been fixed

deonwaju avatar Dec 08 '22 08:12 deonwaju

Yes this issue is fixed. I would like someone from community to close this issue.

YesShubham01 avatar Jan 03 '23 07:01 YesShubham01

@rt4914 is going to verify this and report back. Thanks for the ping.

seanlip avatar Jan 03 '23 11:01 seanlip

@YesShubham01 I did check this on latest develop code and it is still not fixed.

Device Config: API 19 Kitkat Pixel 2 Android 4.4 armeabi-v7a

Screenshot_20230103_171334

rt4914 avatar Jan 03 '23 11:01 rt4914

@seanlip @rt4914 I think this is happening because the brown icon is selected for administrator controls. I am able to change it to grey icon. But I'm not able to change the color according to whether the button is selected or not.

I get the error "This feature requires ASM7" when I try to put conditionals - image

Can you please help me, and if possible assign me the issue?

sumanbmondal avatar Feb 02 '23 17:02 sumanbmondal

@rt4914 I want to work on this issue. Please assign it to me.

nawaz-anwar avatar Feb 07 '23 12:02 nawaz-anwar

@nawaz-anwar Please provide an explanation of what your PR will do (with names of files you're changing, what you plan to change in each file, etc.). If it looks good, we can assign you to this issue.

seanlip avatar Feb 08 '23 05:02 seanlip

@seanlip @rt4914 I think this is happening because the brown icon is selected for administrator controls. I am able to change it to grey icon. But I'm not able to change the color according to whether the button is selected or not.

I get the error "This feature requires ASM7" when I try to put conditionals - image

Can you please help me, and if possible assign me the issue?

@seanlip @rt4914 Can you please comment on this?

sumanbmondal avatar Feb 08 '23 18:02 sumanbmondal