Github user sagar15795 commented on a diff in the pull request:

    
https://github.com/apache/incubator-taverna-mobile/pull/78#discussion_r191054525
  
    --- Diff: 
app/src/main/java/org/apache/taverna/mobile/ui/UserProfileActivity.java ---
    @@ -0,0 +1,171 @@
    +package org.apache.taverna.mobile.ui;
    +
    +import android.support.v7.app.ActionBar;
    +import android.support.v7.app.AppCompatActivity;
    +import android.os.Bundle;
    +
    +import android.support.v7.widget.Toolbar;
    +import android.view.MenuItem;
    +import android.view.View;
    +import android.widget.LinearLayout;
    +import android.widget.TextView;
    +
    +import com.bumptech.glide.Glide;
    +import com.bumptech.glide.load.engine.DiskCacheStrategy;
    +import com.bumptech.glide.load.resource.drawable.GlideDrawable;
    +import com.bumptech.glide.request.animation.GlideAnimation;
    +import com.bumptech.glide.request.target.SimpleTarget;
    +
    +import org.apache.taverna.mobile.R;
    +import org.apache.taverna.mobile.data.DataManager;
    +import org.apache.taverna.mobile.data.local.PreferencesHelper;
    +
    +import butterknife.BindView;
    +import butterknife.ButterKnife;
    +import de.hdodenhof.circleimageview.CircleImageView;
    +
    +import static org.apache.taverna.mobile.TavernaApplication.getContext;
    +
    +
    +public class UserProfileActivity extends AppCompatActivity{
    +
    +    @BindView(R.id.toolbar)
    +    Toolbar mToolbar;
    +
    +    @BindView(R.id.user_name)
    +    TextView user_name;
    --- End diff --
    
    Use camel notation for the variable name. Also, use `m` prefix for all 
class variables.


---

Reply via email to