diff options
| -rwxr-xr-x | org.fox.ttrss/build.gradle | 2 | ||||
| -rwxr-xr-x | org.fox.ttrss/src/main/java/org/fox/ttrss/ArticleFragment.java | 7 | ||||
| -rwxr-xr-x | org.fox.ttrss/src/main/java/org/fox/ttrss/ArticlePager.java | 2 | ||||
| -rwxr-xr-x | org.fox.ttrss/src/main/java/org/fox/ttrss/offline/OfflineArticleFragment.java | 9 | ||||
| -rwxr-xr-x | org.fox.ttrss/src/main/java/org/fox/ttrss/offline/OfflineArticlePager.java | 2 | ||||
| -rwxr-xr-x | org.fox.ttrss/src/main/res/layout/fragment_article.xml | 99 | ||||
| -rwxr-xr-x | org.fox.ttrss/src/main/res/layout/fragment_article_pager.xml (renamed from org.fox.ttrss/src/main/res/layout/article_pager.xml) | 5 | ||||
| -rw-r--r-- | org.fox.ttrss/src/main/res/layout/layout_detail_phone.xml | 57 |
8 files changed, 86 insertions, 97 deletions
diff --git a/org.fox.ttrss/build.gradle b/org.fox.ttrss/build.gradle index 34d31f9c..39fdb5ae 100755 --- a/org.fox.ttrss/build.gradle +++ b/org.fox.ttrss/build.gradle @@ -123,7 +123,7 @@ dependencies { implementation 'me.relex:circleindicator:1.2.2@aar' implementation 'com.nineoldandroids:library:2.4.0' implementation 'com.github.amulyakhare:TextDrawable:558677ea31' - + implementation 'com.telefonica:nestedscrollwebview:0.1.6' } java { toolchain { diff --git a/org.fox.ttrss/src/main/java/org/fox/ttrss/ArticleFragment.java b/org.fox.ttrss/src/main/java/org/fox/ttrss/ArticleFragment.java index 621238c1..959ef7bf 100755 --- a/org.fox.ttrss/src/main/java/org/fox/ttrss/ArticleFragment.java +++ b/org.fox.ttrss/src/main/java/org/fox/ttrss/ArticleFragment.java @@ -179,7 +179,6 @@ public class ArticleFragment extends androidx.fragment.app.Fragment { view.findViewById(R.id.article_scrollview).setVisibility(View.INVISIBLE); } */ - m_contentView = view.findViewById(R.id.article_scrollview); m_customViewContainer = view.findViewById(R.id.article_fullscreen_video); /* if (m_article.id == HeadlinesFragment.ARTICLE_SPECIAL_TOP_CHANGED) { @@ -346,8 +345,12 @@ public class ArticleFragment extends androidx.fragment.app.Fragment { host.setText(outhost.replaceFirst("^www\\.", "")); host.setTextSize(TypedValue.COMPLEX_UNIT_SP, m_articleSmallFontSize); host.setVisibility(View.VISIBLE); + } else { + host.setVisibility(View.GONE); } - } catch (MalformedURLException ignored) {} + } catch (MalformedURLException ignored) { + host.setVisibility(View.GONE); + } } TextView note = view.findViewById(R.id.note); diff --git a/org.fox.ttrss/src/main/java/org/fox/ttrss/ArticlePager.java b/org.fox.ttrss/src/main/java/org/fox/ttrss/ArticlePager.java index cdaca2dd..c94f0c3a 100755 --- a/org.fox.ttrss/src/main/java/org/fox/ttrss/ArticlePager.java +++ b/org.fox.ttrss/src/main/java/org/fox/ttrss/ArticlePager.java @@ -135,7 +135,7 @@ public class ArticlePager extends androidx.fragment.app.Fragment { @Override public View onCreateView(LayoutInflater inflater, ViewGroup container, Bundle savedInstanceState) { - View view = inflater.inflate(R.layout.article_pager, container, false); + View view = inflater.inflate(R.layout.fragment_article_pager, container, false); if (savedInstanceState != null) { if (m_activity instanceof DetailActivity) { diff --git a/org.fox.ttrss/src/main/java/org/fox/ttrss/offline/OfflineArticleFragment.java b/org.fox.ttrss/src/main/java/org/fox/ttrss/offline/OfflineArticleFragment.java index 9451f8f6..3b199d2a 100755 --- a/org.fox.ttrss/src/main/java/org/fox/ttrss/offline/OfflineArticleFragment.java +++ b/org.fox.ttrss/src/main/java/org/fox/ttrss/offline/OfflineArticleFragment.java @@ -199,7 +199,6 @@ public class OfflineArticleFragment extends Fragment { m_cursor.moveToFirst(); if (m_cursor.isFirst()) { - m_contentView = view.findViewById(R.id.article_scrollview); m_customViewContainer = view.findViewById(R.id.article_fullscreen_video); final String link = m_cursor.getString(m_cursor.getColumnIndex("link")); @@ -265,7 +264,13 @@ public class OfflineArticleFragment extends Fragment { if (comments != null) { comments.setVisibility(View.GONE); } - + + View host = view.findViewById(R.id.host); + + if (host != null) { + host.setVisibility(View.GONE); + } + TextView note = view.findViewById(R.id.note); if (note != null) { diff --git a/org.fox.ttrss/src/main/java/org/fox/ttrss/offline/OfflineArticlePager.java b/org.fox.ttrss/src/main/java/org/fox/ttrss/offline/OfflineArticlePager.java index 20c2fd0c..ce4f6f72 100755 --- a/org.fox.ttrss/src/main/java/org/fox/ttrss/offline/OfflineArticlePager.java +++ b/org.fox.ttrss/src/main/java/org/fox/ttrss/offline/OfflineArticlePager.java @@ -136,7 +136,7 @@ public class OfflineArticlePager extends Fragment { @Override public View onCreateView(LayoutInflater inflater, ViewGroup container, Bundle savedInstanceState) { - View view = inflater.inflate(R.layout.article_pager, container, false); + View view = inflater.inflate(R.layout.fragment_article_pager, container, false); if (savedInstanceState != null) { m_articleId = savedInstanceState.getInt("articleId", 0); diff --git a/org.fox.ttrss/src/main/res/layout/fragment_article.xml b/org.fox.ttrss/src/main/res/layout/fragment_article.xml index 0b1f1f0e..0f156245 100755 --- a/org.fox.ttrss/src/main/res/layout/fragment_article.xml +++ b/org.fox.ttrss/src/main/res/layout/fragment_article.xml @@ -22,17 +22,16 @@ <LinearLayout app:layout_scrollFlags="scroll|enterAlways" - android:paddingTop="0dp" - android:paddingStart="16dp" - android:paddingEnd="8dp" - android:paddingBottom="8dp" android:id="@+id/article_header" + android:background="?colorSurface" android:layout_width="match_parent" android:layout_height="wrap_content" android:orientation="vertical"> <LinearLayout android:orientation="horizontal" + android:layout_marginStart="16dp" + android:layout_marginEnd="16dp" android:layout_width="match_parent" android:layout_height="match_parent"> @@ -41,7 +40,6 @@ android:layout_width="match_parent" android:layout_height="wrap_content" android:layout_weight="1" - android:layout_marginEnd="8dp" android:textAlignment="viewStart" android:textColor="?colorPrimary" tools:text="Unreasonably long article title blah blah blah blah blah blah" @@ -78,34 +76,40 @@ <TextView android:id="@+id/comments" + android:layout_marginStart="16dp" + android:layout_marginEnd="16dp" tools:text="comments" android:layout_width="match_parent" android:layout_height="wrap_content" android:textAlignment="viewStart" android:layout_weight="1" android:fontFamily="sans-serif-light" - android:paddingTop="8dp" + android:layout_marginTop="8dp" android:textColor="?colorSecondary" android:textSize="12sp" /> <TextView android:id="@+id/host" tools:text="host" + android:layout_marginStart="16dp" + android:layout_marginEnd="16dp" android:layout_width="match_parent" android:layout_height="wrap_content" android:textAlignment="viewStart" android:layout_weight="1" android:fontFamily="sans-serif-light" - android:paddingTop="8dp" + android:layout_marginTop="8dp" android:textColor="?colorSecondary" - android:textSize="12sp" - android:visibility="gone"/> + android:textSize="12sp" /> + <LinearLayout android:layout_width="match_parent" android:layout_height="wrap_content" - android:layout_weight="1" - android:paddingTop="8dp"> + android:layout_marginStart="16dp" + android:layout_marginEnd="16dp" + android:layout_marginBottom="8dp" + android:layout_marginTop="8dp"> <TextView android:id="@+id/tags" @@ -134,58 +138,41 @@ </LinearLayout> - </LinearLayout> + <com.google.android.material.divider.MaterialDivider + android:layout_width="match_parent" + android:layout_height="wrap_content"/> - <com.google.android.material.divider.MaterialDivider - android:layout_width="match_parent" - android:layout_height="wrap_content"/> + <LinearLayout + android:layout_width="match_parent" + android:layout_height="wrap_content" + android:background="?colorSurfaceContainer"> + + <TextView + android:id="@+id/note" + tools:text="Article note (if any...)" + android:layout_width="match_parent" + android:layout_height="wrap_content" + android:background="?colorSecondaryContainer" + android:layout_marginTop="8dp" + android:layout_marginStart="8dp" + android:layout_marginEnd="8dp" + android:padding="8dp" + android:textColor="?colorOnSecondaryContainer" + android:textSize="12sp"/> + </LinearLayout> + </LinearLayout> </com.google.android.material.appbar.AppBarLayout> - <androidx.core.widget.NestedScrollView - android:id="@+id/article_scrollview" - android:scrollbars="vertical|horizontal" + <com.telefonica.nestedscrollwebview.NestedScrollWebView + android:id="@+id/article_content" + android:layout_marginTop="8dp" + android:layout_marginStart="16dp" + android:layout_marginEnd="16dp" app:layout_behavior="@string/appbar_scrolling_view_behavior" android:layout_width="match_parent" - android:layout_height="match_parent"> - - <LinearLayout - android:layout_width="match_parent" - android:layout_height="match_parent" - android:descendantFocusability="blocksDescendants" - android:orientation="vertical" > - - <TextView - android:id="@+id/note" - tools:text="Article note (if any...)" - android:layout_width="match_parent" - android:layout_height="wrap_content" - android:layout_weight="1" - android:background="?colorSecondaryContainer" - android:textColor="?colorOnSecondaryContainer" - android:textSize="12sp" - android:paddingLeft="16dp" - android:paddingTop="8dp" - android:paddingRight="16dp" - android:paddingBottom="8dp" /> - - <org.fox.ttrss.util.LessBrokenWebView - android:layout_margin="16dp" - android:id="@+id/article_content" - android:scrollbars="none" - android:layout_width="match_parent" - android:layout_height="wrap_content" /> - - </LinearLayout> - </androidx.core.widget.NestedScrollView> + android:layout_height="wrap_content" /> </com.otaliastudios.nestedscrollcoordinatorlayout.NestedScrollCoordinatorLayout> - <TextView - android:layout_width="match_parent" - android:layout_height="match_parent" - android:id="@+id/article_status_message" - android:gravity="center" - android:visibility="gone" /> - </FrameLayout>
\ No newline at end of file diff --git a/org.fox.ttrss/src/main/res/layout/article_pager.xml b/org.fox.ttrss/src/main/res/layout/fragment_article_pager.xml index 62b3fea4..1a687b1d 100755 --- a/org.fox.ttrss/src/main/res/layout/article_pager.xml +++ b/org.fox.ttrss/src/main/res/layout/fragment_article_pager.xml @@ -2,6 +2,5 @@ <androidx.viewpager.widget.ViewPager xmlns:android="http://schemas.android.com/apk/res/android" android:id="@+id/article_pager" - android:background="?android:colorBackground" - android:layout_width="fill_parent" - android:layout_height="fill_parent"/> + android:layout_width="match_parent" + android:layout_height="wrap_content"/> diff --git a/org.fox.ttrss/src/main/res/layout/layout_detail_phone.xml b/org.fox.ttrss/src/main/res/layout/layout_detail_phone.xml index 9421f008..429b27c0 100644 --- a/org.fox.ttrss/src/main/res/layout/layout_detail_phone.xml +++ b/org.fox.ttrss/src/main/res/layout/layout_detail_phone.xml @@ -1,38 +1,32 @@ -<FrameLayout xmlns:android="http://schemas.android.com/apk/res/android" - android:id="@+id/headlines" - android:layout_width="fill_parent" - android:layout_height="fill_parent"> +<androidx.coordinatorlayout.widget.CoordinatorLayout + xmlns:android="http://schemas.android.com/apk/res/android" + xmlns:app="http://schemas.android.com/apk/res-auto" + android:layout_width="match_parent" + android:layout_height="wrap_content"> + + <com.google.android.material.floatingactionbutton.FloatingActionButton + android:id="@+id/detail_fab" + android:layout_width="wrap_content" + android:layout_height="wrap_content" + android:layout_gravity="end|bottom" + android:visibility="gone" + android:src="@drawable/baseline_open_in_browser_24" + android:layout_margin="16dp" /> - <androidx.coordinatorlayout.widget.CoordinatorLayout - xmlns:app="http://schemas.android.com/apk/res-auto" + <com.google.android.material.appbar.AppBarLayout android:layout_width="match_parent" - android:layout_height="match_parent"> - - <com.google.android.material.floatingactionbutton.FloatingActionButton - android:id="@+id/detail_fab" - android:layout_width="wrap_content" - android:layout_height="wrap_content" - android:layout_gravity="end|bottom" - android:visibility="gone" - android:src="@drawable/baseline_open_in_browser_24" - android:layout_margin="16dp" /> - - <com.google.android.material.appbar.AppBarLayout - android:layout_width="match_parent" - android:layout_height="wrap_content"> - - <include layout="@layout/toolbar" android:id="@+id/toolbar" /> + android:layout_height="wrap_content"> - </com.google.android.material.appbar.AppBarLayout> + <include layout="@layout/toolbar" android:id="@+id/toolbar" /> - <FrameLayout - android:id="@+id/article_fragment" - app:layout_behavior=".util.DetailActivityScrollingViewBehavior" - android:layout_width="match_parent" - android:layout_height="wrap_content"> - </FrameLayout> + </com.google.android.material.appbar.AppBarLayout> - </androidx.coordinatorlayout.widget.CoordinatorLayout> + <FrameLayout + android:id="@+id/article_fragment" + app:layout_behavior=".util.DetailActivityScrollingViewBehavior" + android:layout_width="match_parent" + android:layout_height="wrap_content"> + </FrameLayout> <FrameLayout android:id="@+id/headlines_fragment" @@ -41,4 +35,5 @@ android:layout_width="0dp"> </FrameLayout> -</FrameLayout>
\ No newline at end of file +</androidx.coordinatorlayout.widget.CoordinatorLayout> + |