silverandroid / friendstagram-android Goto Github PK

When using a Bundle to pass the arguments, it'll be annoying because it'll require to make 2 variables instead of 1 for a new constructor parameter but it'll make sure they aren't static variables.

On the random occurrence that a post does fail to be posted, instead of uploading the image to Cloudinary again, just reuse the URL returned

If a fragment with a Subject has been subscribed to, and the fragment is resumed, it'll get something twice.

Add an extension method for EditTexts where it allows for validation of them

For example, instead of:

if (!Util.isPasswordValid(password)) 
    editPassword.setError(getString(R.string.error_password_short))

it can be:

editPassword.addValidation({ value -> return Util.isPasswordValid(value) }, R.string.error_password_short)

// ...

editPassword.validate()

When using the fade animation, if the fade animation is run again while there is one currently running, it will set the visibility of both views to invisible.

First, I need to figure out how to do them between Fragments. Then, use them with Fragments like from Profile fragment to Picture fragment.

For most screens (such as profile view or edit profile), a local SQLite database can be used to grab that information directly after loading from the network and refreshing the data occasionally.

Improve code architecture using:

• RxJava

• New architecture components announced at Google I/O

However, this should be done after the app is complete with all the features added.

Toolbar title is not centering even though it is specified in the layout. It only seems to happen inside ProfileFragment.java so it seems to have something to do with any of the other layouts such as CoordinatorLayout or AppBarLayout or CollapsingToolbarLayout.

When account fragment loaded, profile picture does not load because Fresco does not submit a request to retrieve the image and just removes the image out of memory. However, if the fragment were replaced a second time (ie. rotating the screen), the profile picture loads (by sending a request).

Only way to leave PicturesActivity is to press the X in the top left corner. Can't leave by clicking on the Back button and it'll just restart the Activity again.

Best practices indicate to separate the 2

Find way to use both in the same adapter

Use Dagger to inject the listeners by putting them in their own module. This way, there won't be a need to worry about listeners such as setToolbar not being initialized when a Fragment is being resumed.

See if AsyncUtil (part of support library) improves the dev process for showing local images on screen and see if it maintains or improves performance

In PicturesFragment, when the screen is rotated, LoaderCallbacks doesn't recall onLoadFinished

When switching back to the login screen from the register screen, it loads the login screen, then the register screen again, then reloads the login screen. It has something to do with the pop backstack animation.

See http://www.androiddesignpatterns.com/2013/08/fragment-transaction-commit-state-loss.html