[MediaWiki-commits] [Gerrit] apps...wikipedia[master]: Create embryo description editing activity.

2016-10-26 Thread jenkins-bot (Code Review)
jenkins-bot has submitted this change and it was merged.

Change subject: Create embryo description editing activity.
..


Create embryo description editing activity.

This makes article descriptions clickable (by adding a ClickableSpan to
the relevant text portion).
This also creates a blank activity that will be the container for the
description editing workflow. The activity is not entirely blank: it
accepts a PageTitle parameter, which feeds it with the requested page
title, which will contain its current description.

Bug: T146639
Bug: T146640
Change-Id: Iaa62a901d93376cf743bfd933db0b6ea47be658c
---
M app/src/main/AndroidManifest.xml
M 
app/src/main/java/org/wikipedia/activity/SingleFragmentActivityWithToolbar.java
A app/src/main/java/org/wikipedia/descriptions/DescriptionEditActivity.java
A app/src/main/java/org/wikipedia/descriptions/DescriptionEditFragment.java
M app/src/main/java/org/wikipedia/page/leadimages/ArticleHeaderView.java
M app/src/main/java/org/wikipedia/page/leadimages/LeadImagesHandler.java
A app/src/main/res/layout/fragment_description_edit.xml
M app/src/main/res/layout/view_article_header.xml
8 files changed, 174 insertions(+), 4 deletions(-)

Approvals:
  jenkins-bot: Verified
  Niedzielski: Looks good to me, approved



diff --git a/app/src/main/AndroidManifest.xml b/app/src/main/AndroidManifest.xml
index 8c2830c..bf97417 100644
--- a/app/src/main/AndroidManifest.xml
+++ b/app/src/main/AndroidManifest.xml
@@ -156,6 +156,10 @@
   android:theme="@style/PageTheme"
   android:exported="false"/>
 
+
+
  {
+private static final String EXTRA_TITLE = "title";
+
+public static Intent newIntent(@NonNull Context context, @NonNull 
PageTitle title) {
+return new Intent(context, DescriptionEditActivity.class)
+.putExtra(EXTRA_TITLE, GsonMarshaller.marshal(title));
+}
+
+@Override
+public void onCreate(Bundle savedInstanceState) {
+super.onCreate(savedInstanceState);
+setWordmarkVisible(false);
+if (getSupportActionBar() != null) {
+getSupportActionBar().setDisplayHomeAsUpEnabled(true);
+}
+}
+
+@Override
+public DescriptionEditFragment createFragment() {
+return 
DescriptionEditFragment.newInstance(GsonUnmarshaller.unmarshal(PageTitle.class, 
getIntent().getStringExtra(EXTRA_TITLE)));
+}
+}
\ No newline at end of file
diff --git 
a/app/src/main/java/org/wikipedia/descriptions/DescriptionEditFragment.java 
b/app/src/main/java/org/wikipedia/descriptions/DescriptionEditFragment.java
new file mode 100644
index 000..711e7f0
--- /dev/null
+++ b/app/src/main/java/org/wikipedia/descriptions/DescriptionEditFragment.java
@@ -0,0 +1,56 @@
+package org.wikipedia.descriptions;
+
+import android.os.Bundle;
+import android.support.annotation.NonNull;
+import android.support.annotation.Nullable;
+import android.support.v4.app.Fragment;
+import android.view.LayoutInflater;
+import android.view.View;
+import android.view.ViewGroup;
+import android.widget.TextView;
+
+import org.wikipedia.R;
+import org.wikipedia.json.GsonMarshaller;
+import org.wikipedia.json.GsonUnmarshaller;
+import org.wikipedia.page.PageTitle;
+
+import butterknife.BindView;
+import butterknife.ButterKnife;
+import butterknife.Unbinder;
+
+public class DescriptionEditFragment extends Fragment {
+private static final String EXTRA_TITLE = "title";
+
+@BindView(R.id.description_edit_page_title) TextView pageTitleText;
+private Unbinder unbinder;
+
+private PageTitle pageTitle;
+
+@NonNull
+public static DescriptionEditFragment newInstance(@NonNull PageTitle 
title) {
+DescriptionEditFragment instance = new DescriptionEditFragment();
+Bundle args = new Bundle();
+args.putString(EXTRA_TITLE, GsonMarshaller.marshal(title));
+instance.setArguments(args);
+return instance;
+}
+
+@Nullable
+@Override
+public View onCreateView(LayoutInflater inflater, ViewGroup container, 
Bundle savedInstanceState) {
+super.onCreateView(inflater, container, savedInstanceState);
+View view = inflater.inflate(R.layout.fragment_description_edit, 
container, false);
+unbinder = ButterKnife.bind(this, view);
+
+pageTitle = GsonUnmarshaller.unmarshal(PageTitle.class, 
getActivity().getIntent().getStringExtra(EXTRA_TITLE));
+
+pageTitleText.setText(pageTitle.getDisplayText());
+return view;
+}
+
+@Override public void onDestroyView() {
+unbinder.unbind();
+unbinder = null;
+super.onDestroyView();
+}
+}
diff --git 
a/app/src/main/java/org/wikipedia/page/leadimages/ArticleHeaderView.java 
b/app/src/main/java/org/wikipedia/page/leadimages/ArticleHeaderView.java
index c5b9019..29b15b9 100644
--- a/app/src/main/java/org/wikipedia/page/leadimages/ArticleHeaderView.java
+++ 

[MediaWiki-commits] [Gerrit] apps...wikipedia[master]: Create embryo description editing activity.

2016-10-26 Thread Dbrant (Code Review)
Dbrant has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/318114

Change subject: Create embryo description editing activity.
..

Create embryo description editing activity.

This makes article descriptions clickable (by adding a ClickableSpan to
the relevant text portion).
This also creates a blank activity that will be the container for the
description editing workflow. The activity is not entirely blank: it
accepts a PageTitle parameter, which feeds it with the requested page
title, which will contain its current description.

Bug: T146639
Bug: T146640
Change-Id: Iaa62a901d93376cf743bfd933db0b6ea47be658c
---
M app/src/main/AndroidManifest.xml
M 
app/src/main/java/org/wikipedia/activity/SingleFragmentActivityWithToolbar.java
A app/src/main/java/org/wikipedia/descriptions/DescriptionEditActivity.java
A app/src/main/java/org/wikipedia/descriptions/DescriptionEditFragment.java
M app/src/main/java/org/wikipedia/page/leadimages/ArticleHeaderView.java
M app/src/main/java/org/wikipedia/page/leadimages/LeadImagesHandler.java
A app/src/main/res/layout/fragment_description_edit.xml
M app/src/main/res/layout/view_article_header.xml
8 files changed, 179 insertions(+), 4 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/apps/android/wikipedia 
refs/changes/14/318114/1

diff --git a/app/src/main/AndroidManifest.xml b/app/src/main/AndroidManifest.xml
index 8c2830c..bf97417 100644
--- a/app/src/main/AndroidManifest.xml
+++ b/app/src/main/AndroidManifest.xml
@@ -156,6 +156,10 @@
   android:theme="@style/PageTheme"
   android:exported="false"/>
 
+
+
  {
+protected static final String EXTRA_TITLE = "title";
+
+@Override
+public void onCreate(Bundle savedInstanceState) {
+super.onCreate(savedInstanceState);
+setWordmarkVisible(false);
+}
+
+public static Intent newIntent(@NonNull Context context, @NonNull 
PageTitle title) {
+return new Intent(context, DescriptionEditActivity.class)
+.putExtra(EXTRA_TITLE, GsonMarshaller.marshal(title));
+}
+
+@Override
+public DescriptionEditFragment createFragment() {
+return 
DescriptionEditFragment.newInstance(GsonUnmarshaller.unmarshal(PageTitle.class, 
getIntent().getStringExtra(EXTRA_TITLE)));
+}
+}
\ No newline at end of file
diff --git 
a/app/src/main/java/org/wikipedia/descriptions/DescriptionEditFragment.java 
b/app/src/main/java/org/wikipedia/descriptions/DescriptionEditFragment.java
new file mode 100644
index 000..2df472e
--- /dev/null
+++ b/app/src/main/java/org/wikipedia/descriptions/DescriptionEditFragment.java
@@ -0,0 +1,64 @@
+package org.wikipedia.descriptions;
+
+import android.os.Bundle;
+import android.support.annotation.NonNull;
+import android.support.annotation.Nullable;
+import android.support.v4.app.Fragment;
+import android.support.v7.app.AppCompatActivity;
+import android.view.LayoutInflater;
+import android.view.View;
+import android.view.ViewGroup;
+import android.widget.TextView;
+
+import org.wikipedia.R;
+import org.wikipedia.json.GsonMarshaller;
+import org.wikipedia.json.GsonUnmarshaller;
+import org.wikipedia.page.PageTitle;
+
+import butterknife.BindView;
+import butterknife.ButterKnife;
+import butterknife.Unbinder;
+
+import static org.wikipedia.descriptions.DescriptionEditActivity.EXTRA_TITLE;
+
+public class DescriptionEditFragment extends Fragment {
+@BindView(R.id.description_edit_page_title) TextView pageTitleText;
+private Unbinder unbinder;
+
+private PageTitle pageTitle;
+
+@NonNull
+public static DescriptionEditFragment newInstance(@NonNull PageTitle 
title) {
+DescriptionEditFragment instance = new DescriptionEditFragment();
+Bundle args = new Bundle();
+args.putString(EXTRA_TITLE, GsonMarshaller.marshal(title));
+instance.setArguments(args);
+return instance;
+}
+
+@Nullable
+@Override
+public View onCreateView(LayoutInflater inflater, ViewGroup container, 
Bundle savedInstanceState) {
+super.onCreateView(inflater, container, savedInstanceState);
+View view = inflater.inflate(R.layout.fragment_description_edit, 
container, false);
+unbinder = ButterKnife.bind(this, view);
+
+
getAppCompatActivity().getSupportActionBar().setDisplayHomeAsUpEnabled(true);
+getAppCompatActivity().getSupportActionBar().setTitle("");
+
+pageTitle = GsonUnmarshaller.unmarshal(PageTitle.class, 
getActivity().getIntent().getStringExtra(EXTRA_TITLE));
+
+pageTitleText.setText(pageTitle.getDisplayText());
+return view;
+}
+
+@Override public void onDestroyView() {
+unbinder.unbind();
+unbinder = null;
+super.onDestroyView();
+}
+
+private AppCompatActivity getAppCompatActivity() {
+return (AppCompatActivity) getActivity();
+}
+}
diff --git