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

Changed user interface #21

Closed
wants to merge 28 commits into from
Closed

Changed user interface #21

wants to merge 28 commits into from

Conversation

Urdzik
Copy link
Owner

@Urdzik Urdzik commented Mar 13, 2020

No description provided.

@Urdzik Urdzik added the enhancement New feature or request label Mar 13, 2020
@Urdzik Urdzik requested a review from KostiaLeo March 13, 2020 20:25
@Urdzik Urdzik added this to Pull requests in Movie App via automation Mar 13, 2020
@KostiaLeo
Copy link
Collaborator

The issue to fix:
Remove enum-class MainScreen. Since it can cause memory overload due to storing fragment-objects. Enum classes stay alive all app life, though fragments shouldn't be always in memory.

@KostiaLeo KostiaLeo closed this Mar 14, 2020
@Urdzik Urdzik mentioned this pull request Mar 15, 2020
@Urdzik Urdzik reopened this Mar 16, 2020
Movie App automation moved this from Pull requests to Done Mar 16, 2020
@Urdzik Urdzik closed this Mar 17, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: Done
Movie App
  
Done
Development

Successfully merging this pull request may close these issues.

None yet

2 participants