Skip to content

Night mode: Category pages #2088

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
millbittencourt opened this issue Dec 8, 2018 · 9 comments · Fixed by #2213
Closed

Night mode: Category pages #2088

millbittencourt opened this issue Dec 8, 2018 · 9 comments · Fixed by #2213
Labels

Comments

@millbittencourt
Copy link

millbittencourt commented Dec 8, 2018

Summary:

When clicking on a category in night mode, the app does not show the "Parent Categories", there's a blank space instead.

Steps to reproduce:

How can we reproduce the issue?
Settings > Night Mode;
Explore > Click any image > Click on a random categorie > Parent Categories

What did you expect the app to do, and what did you see instead?
The names of the parent categories should be shown.

Device and Android version:

Found on Xiaomi A1, Android 8.1.0

Commons app version:
2.8.5

Screen-shots:
9a656a2e-8237-4dbe-b4b0-16f5b277ae89

Would you like to work on the issue?

Yess.

@vanshikaarora
Copy link
Contributor

Hey!!
Have you made any changes to your code. My app crashes while trying to open this activity. It gives null pointer exception.

java.lang.NullPointerException: Attempt to invoke virtual method 'android.support.v4.app.FragmentActivity android.support.v4.app.Fragment.getActivity()' on a null object reference
at fr.free.nrw.commons.media.MediaDetailPagerFragment$MediaDetailAdapter.getItem(MediaDetailPagerFragment.java:401)

@domdomegg
Copy link
Member

@vanshikaarora I think that's unrelated. I fixed that in 46db38b

@domdomegg
Copy link
Member

Just checked, this also happens in subcategories. So basically any categories page

@domdomegg domdomegg changed the title Parent Categories on Night Mode Night mode: Category pages Dec 19, 2018
@domdomegg domdomegg mentioned this issue Dec 19, 2018
20 tasks
@vanshikaarora
Copy link
Contributor

Ok @domdomegg I'll check that out

@domdomegg
Copy link
Member

@vanshikaarora are you working on this? Just want it to be clear for other developers looking at issues.

@Divyanshu2132
Copy link
Contributor

I am working on this issue

@domdomegg
Copy link
Member

Okay. I'm going to do the tabs separately in #2186, so please don't change them if possible (to avoid merge conflicts)

@Divyanshu2132
Copy link
Contributor

Sure
I'll talk to you before changing or creating pull request

Divyanshu2132 added a commit to Divyanshu2132/apps-android-commons that referenced this issue Dec 20, 2018
@Divyanshu2132
Copy link
Contributor

@domdomegg I have created a pull request look into the files changed to avoid merge conflicts

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants