-
Notifications
You must be signed in to change notification settings - Fork 42
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
Home scene rework #39
base: master
Are you sure you want to change the base?
Conversation
…o one definition.
I missinterpreted a couple of lines of code and if I remember correctly, my implementation of the sidebar doesn't let more than 10 channels to load, instead of showing 10 channels and letting you scroll down to them. This will require a quick fix: deleting the "if" in 15th line of Sidebar.brs. Will do this at some other time. |
Hey, thanks for your efforts. I don't think we would've tried cleaning up the code ever, so this is appreciated lol. Hopefully we can check this out soon |
The objective of this update is to mantain similar functionality and graphics, but fixing underlaying code so it's more friendly to continue developing new features.
Changes made:
Code structure:
Behaviour changes:
Issues fixed:
Tests required (this I have not been able to test):
Introduced differences that could be issues: