-
-
Notifications
You must be signed in to change notification settings - Fork 443
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
Bottom toolbar hide text on help screen fixed #604
Conversation
I see there are certain conflicts that are needed to be resolved. |
@jb1998 What is the status of this PR? |
@brijeshshah13 I am doing a straight forward thing. So to solve it I come up with the solution to remove the top space dynamically. I would like to make some changes in help.xml to see the cause. I will let you know in a day or two if I could make it from help.xml Here is the screenshot displaying the weird space added at top |
@jb1998 Alright. Also, you might want to search for alternative solutions, just in case this approach doesn't work out. |
Yeah will definitely do that :) |
I am not able to build the project in android studio. |
@jb1998 Please try to |
I did Clean project and Invalidate caches but that did not work. Finally changing build variant to debug worked. :) |
@jb1998 You mean |
@brijeshshah13 Yes kiwixDebug build variant |
@brijeshshah13 I have fixed the issue Please review it :) |
@abdulwd Thanks for informing :) |
Fixes #[555]
Changes: [I gave id to last cardview and I am then dynamically adding space when toolbar is enabled,the check is also made in onResume() of KiwixMobileActivity and if bottom toolbar is enabled then space is added accordingly]
Screenshots/GIF for the change: Below is the gif showing the changes.The text is now visible when we open the app and bottom toolbar is enabled as well as when we navigate from one screen in app to help screen.