Skip to content
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

[DO NOT MERGE UNTIL 0.16.1] Fix Screen Flickering in Libarary page #11853

Merged
merged 2 commits into from
Feb 21, 2024

Conversation

iskipu
Copy link
Contributor

@iskipu iskipu commented Feb 10, 2024

Summary

The issue got fixed after converting the margins into to padding.

screen-flicker.mp4

References

fix #11838

Reviewer guidance


@github-actions github-actions bot added APP: Learn Re: Learn App (content, quizzes, lessons, etc.) DEV: frontend SIZE: small labels Feb 10, 2024
@iskipu iskipu changed the title change margin to padding Fix Screen Flickering in Libarary page Feb 10, 2024
@MisRob MisRob self-requested a review February 12, 2024 05:54
@MisRob MisRob self-assigned this Feb 12, 2024
@MisRob MisRob added the TODO: needs review Waiting for review label Feb 12, 2024
@pcenov
Copy link
Member

pcenov commented Feb 12, 2024

Hi @iskipu the Android app keeps force stopping when I launch it so there must be something wrong with the .apk build. Could you check why is this so?

@iskipu
Copy link
Contributor Author

iskipu commented Feb 12, 2024

Hi @iskipu the Android app keeps force stopping when I launch it so there must be something wrong with the .apk build. Could you check why is this so?

Hi @pcenov i checked few other apk builds from other PRs and they were not working too !!

@pcenov
Copy link
Member

pcenov commented Feb 13, 2024

@rtibbles I need to test this in the Android app - could you please have a look and advise what needs to be done so that the app installer is not broken. Thanks!

@iskipu
Copy link
Contributor Author

iskipu commented Feb 14, 2024

Hi @pcenov, @rtibbles had made a fix to the issue, so i rebased the commit and force pushed but not sure why kolibri build is failing any idea how to fix this ?

@rtibbles
Copy link
Member

Not your fault, there's another open PR to fix this new issue! Never boring around here!

@iskipu
Copy link
Contributor Author

iskipu commented Feb 14, 2024

Not your fault, there's another open PR to fix this new issue! Never boring around here!

Oh Cool !! i was very confused why it was not working and also this is my first time trying git rebase so i was thinking i am doing something wrong 🤣 🤣

@MisRob
Copy link
Member

MisRob commented Feb 16, 2024

Thanks @iskipu and @rtibbles. Seems builds are now working, @pcenov.

@pcenov pcenov self-requested a review February 16, 2024 13:05
Copy link
Member

@pcenov pcenov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you @iskipu - I confirm that the flickering issue is fixed now!

@MisRob
Copy link
Member

MisRob commented Feb 20, 2024

Hi @iskipu, thanks for the contribution and @pcenov for testing. I haven't noticed any issue related to the change from margin to padding so I think this is good. We will need to wait for merging though because we're about to release 0.16.0 but this change is planned for 0.16.1.

@MisRob MisRob changed the title Fix Screen Flickering in Libarary page [DO NOT MERGE UNTIL 0.16.1] Fix Screen Flickering in Libarary page Feb 20, 2024
@MisRob
Copy link
Member

MisRob commented Feb 21, 2024

0.16.0 is out so we can merge

@MisRob MisRob merged commit 1f5fa7c into learningequality:release-v0.16.x Feb 21, 2024
34 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
APP: Learn Re: Learn App (content, quizzes, lessons, etc.) DEV: frontend SIZE: small TODO: needs review Waiting for review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants