-
Notifications
You must be signed in to change notification settings - Fork 45
Conversation
Don't be scared by the number of lines changed, a lot of it is just the licences' text :) |
protected void onStart() { | ||
super.onStart(); | ||
|
||
licensesList.setAdapter(new LibrariesAdapter(this)); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
just asking: why are we creating the LibrariesAdapter
every time the Activity
is started? This is a fixed list which is never going to change at runtime so wouldn't it be better to set it in the onCreate
? Same for the LayoutManager
and the ItemDecorator
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yep that's actually a bug, I created onStart
by mistake and didn't notice. Will fix asap.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🎉
This PR closes #27 by implementing the last bit, the licences screen. As a side note I realised we don't need Android-Apt as since the Android Gradle Plugin 2.2 there is a built-in annotation processor, so I switched to that one and removed one dependency.
TODO:
FOSS licencesAbout screenUse CoordinatorLayout in settingsPhoto borderObtain cover photodefo not worth it