Revert "use scrollview/webview instead of titlewebview hack"
This reverts commit 780629fff5
.
Conflicts:
res/layout/article_fragment.xml
src/org/fox/ttrss/ArticleFragment.java
src/org/fox/ttrss/offline/OfflineArticleFragment.java
This commit is contained in:
parent
ced4e4b089
commit
cb18030f6b
@ -1,105 +1,97 @@
|
||||
<RelativeLayout xmlns:android="http://schemas.android.com/apk/res/android"
|
||||
<LinearLayout xmlns:android="http://schemas.android.com/apk/res/android"
|
||||
android:id="@+id/article_fragment"
|
||||
android:layout_width="fill_parent"
|
||||
android:layout_height="fill_parent"
|
||||
android:background="?articleBackground"
|
||||
android:orientation="vertical"
|
||||
android:padding="5sp" >
|
||||
|
||||
<org.fox.ttrss.util.NoChildFocusScrollView
|
||||
android:id="@+id/scrollView1"
|
||||
<org.fox.ttrss.TitleWebView
|
||||
android:id="@+id/content"
|
||||
android:layout_width="match_parent"
|
||||
android:layout_height="match_parent"
|
||||
android:fillViewport="false" >
|
||||
android:layout_weight="1">
|
||||
|
||||
<LinearLayout
|
||||
android:id="@+id/article_header"
|
||||
android:layout_width="match_parent"
|
||||
android:layout_height="wrap_content"
|
||||
android:layout_weight="0"
|
||||
android:orientation="vertical"
|
||||
android:paddingBottom="2dp" >
|
||||
|
||||
<TextView
|
||||
android:id="@+id/title"
|
||||
android:layout_width="match_parent"
|
||||
android:layout_height="wrap_content"
|
||||
android:layout_weight="1"
|
||||
android:paddingBottom="4dp"
|
||||
android:text="My simple headline"
|
||||
android:textColor="?linkColor"
|
||||
android:textSize="18sp" />
|
||||
|
||||
<TextView
|
||||
android:id="@+id/author"
|
||||
android:layout_width="match_parent"
|
||||
android:layout_height="wrap_content"
|
||||
android:layout_weight="1"
|
||||
android:ellipsize="end"
|
||||
android:gravity="right"
|
||||
android:singleLine="true"
|
||||
android:text="by Author"
|
||||
android:textColor="?headlineSecondaryTextColor"
|
||||
android:textSize="12sp" />
|
||||
|
||||
<TextView
|
||||
android:id="@+id/comments"
|
||||
android:layout_width="match_parent"
|
||||
android:layout_height="wrap_content"
|
||||
android:layout_weight="1"
|
||||
android:gravity="right"
|
||||
android:text="24 comments"
|
||||
android:textColor="?linkColor"
|
||||
android:textSize="12sp" />
|
||||
|
||||
<ImageView
|
||||
android:layout_width="match_parent"
|
||||
android:layout_height="wrap_content"
|
||||
android:layout_marginBottom="4dp"
|
||||
android:layout_marginTop="4dp"
|
||||
android:layout_weight="0"
|
||||
android:background="?ttrssHorizontalDivider"
|
||||
android:paddingTop="2dip" />
|
||||
|
||||
<LinearLayout
|
||||
android:layout_width="match_parent"
|
||||
android:layout_height="match_parent"
|
||||
android:orientation="vertical" >
|
||||
android:layout_height="wrap_content"
|
||||
android:paddingBottom="4dp"
|
||||
android:layout_weight="1" >
|
||||
|
||||
<LinearLayout
|
||||
android:id="@+id/article_header"
|
||||
android:layout_width="match_parent"
|
||||
<TextView
|
||||
android:id="@+id/tags"
|
||||
android:layout_width="wrap_content"
|
||||
android:layout_height="wrap_content"
|
||||
android:orientation="vertical"
|
||||
android:paddingBottom="2dp" >
|
||||
android:layout_weight="0.5"
|
||||
android:ellipsize="end"
|
||||
android:singleLine="true"
|
||||
android:text="Example Feed"
|
||||
android:textColor="?headlineSecondaryTextColor"
|
||||
android:textSize="12sp" />
|
||||
|
||||
<TextView
|
||||
android:id="@+id/title"
|
||||
android:layout_width="match_parent"
|
||||
android:layout_height="wrap_content"
|
||||
android:layout_weight="1"
|
||||
android:paddingBottom="4dp"
|
||||
android:text="My simple headline"
|
||||
android:textColor="?linkColor"
|
||||
android:textSize="18sp" />
|
||||
|
||||
<TextView
|
||||
android:id="@+id/author"
|
||||
android:layout_width="match_parent"
|
||||
android:layout_height="wrap_content"
|
||||
android:layout_weight="1"
|
||||
android:ellipsize="end"
|
||||
android:gravity="right"
|
||||
android:singleLine="true"
|
||||
android:text="by Author"
|
||||
android:textColor="?headlineSecondaryTextColor"
|
||||
android:textSize="12sp" />
|
||||
|
||||
<TextView
|
||||
android:id="@+id/comments"
|
||||
android:layout_width="match_parent"
|
||||
android:layout_height="wrap_content"
|
||||
android:layout_weight="1"
|
||||
android:gravity="right"
|
||||
android:text="24 comments"
|
||||
android:textColor="?linkColor"
|
||||
android:textSize="12sp" />
|
||||
|
||||
<ImageView
|
||||
android:layout_width="match_parent"
|
||||
android:layout_height="wrap_content"
|
||||
android:layout_marginBottom="4dp"
|
||||
android:layout_marginTop="4dp"
|
||||
android:layout_weight="0"
|
||||
android:background="?ttrssHorizontalDivider"
|
||||
android:paddingTop="2dip" />
|
||||
|
||||
<LinearLayout
|
||||
android:layout_width="match_parent"
|
||||
android:layout_height="wrap_content"
|
||||
android:layout_weight="1"
|
||||
android:paddingBottom="4dp" >
|
||||
|
||||
<TextView
|
||||
android:id="@+id/tags"
|
||||
android:layout_width="wrap_content"
|
||||
android:layout_height="wrap_content"
|
||||
android:layout_weight="0.5"
|
||||
android:ellipsize="end"
|
||||
android:singleLine="true"
|
||||
android:text="Example Feed"
|
||||
android:textColor="?headlineSecondaryTextColor"
|
||||
android:textSize="12sp" />
|
||||
|
||||
<TextView
|
||||
android:id="@+id/date"
|
||||
android:layout_width="wrap_content"
|
||||
android:layout_height="wrap_content"
|
||||
android:layout_weight="0.5"
|
||||
android:gravity="right"
|
||||
android:text="Jan 01, 12:00"
|
||||
android:textAppearance="?android:attr/textAppearanceSmall"
|
||||
android:textColor="?headlineSecondaryTextColor"
|
||||
android:textSize="12sp" />
|
||||
</LinearLayout>
|
||||
</LinearLayout>
|
||||
|
||||
<WebView
|
||||
android:id="@+id/content"
|
||||
android:layout_width="match_parent"
|
||||
android:layout_height="wrap_content" />
|
||||
<TextView
|
||||
android:id="@+id/date"
|
||||
android:layout_width="wrap_content"
|
||||
android:layout_height="wrap_content"
|
||||
android:layout_weight="0.5"
|
||||
android:gravity="right"
|
||||
android:text="Jan 01, 12:00"
|
||||
android:textAppearance="?android:attr/textAppearanceSmall"
|
||||
android:textColor="?headlineSecondaryTextColor"
|
||||
android:textSize="12sp" />
|
||||
</LinearLayout>
|
||||
</org.fox.ttrss.util.NoChildFocusScrollView>
|
||||
</LinearLayout>
|
||||
|
||||
|
||||
</RelativeLayout>
|
||||
</org.fox.ttrss.TitleWebView>
|
||||
|
||||
</LinearLayout>
|
||||
|
@ -161,11 +161,11 @@
|
||||
android:title="@string/offline_oldest_first" />
|
||||
</PreferenceCategory>
|
||||
<PreferenceCategory android:title="@string/debugging" >
|
||||
<CheckBoxPreference
|
||||
<!-- <CheckBoxPreference
|
||||
android:defaultValue="true"
|
||||
android:key="webview_hardware_accel"
|
||||
android:summary="@string/accel_webview_summary"
|
||||
android:title="@string/accel_webview_title" />
|
||||
android:title="@string/accel_webview_title" /> -->
|
||||
<CheckBoxPreference
|
||||
android:defaultValue="false"
|
||||
android:key="transport_debugging"
|
||||
|
@ -23,7 +23,6 @@ import android.os.Bundle;
|
||||
import android.preference.PreferenceManager;
|
||||
import android.support.v4.app.Fragment;
|
||||
import android.text.Html;
|
||||
import android.util.Log;
|
||||
import android.util.TypedValue;
|
||||
import android.view.ContextMenu;
|
||||
import android.view.ContextMenu.ContextMenuInfo;
|
||||
@ -164,7 +163,7 @@ public class ArticleFragment extends Fragment implements GestureDetector.OnDoubl
|
||||
registerForContextMenu(web);
|
||||
|
||||
// prevent flicker in ics
|
||||
if (!m_prefs.getBoolean("webview_hardware_accel", true)) {
|
||||
if (android.os.Build.VERSION.SDK_INT >= 11) {
|
||||
web.setLayerType(View.LAYER_TYPE_SOFTWARE, null);
|
||||
}
|
||||
|
||||
@ -281,7 +280,7 @@ public class ArticleFragment extends Fragment implements GestureDetector.OnDoubl
|
||||
}
|
||||
}
|
||||
|
||||
content += "</body></html>";
|
||||
content += "<p> </p><p> </p><p> </p><p> </p></body></html>";
|
||||
|
||||
try {
|
||||
String baseUrl = null;
|
||||
|
91
src/org/fox/ttrss/TitleWebView.java
Normal file
91
src/org/fox/ttrss/TitleWebView.java
Normal file
@ -0,0 +1,91 @@
|
||||
package org.fox.ttrss;
|
||||
|
||||
// http://www.techques.com/question/1-9718245/Webview-in-Scrollview
|
||||
|
||||
import android.content.Context;
|
||||
import android.graphics.Canvas;
|
||||
import android.util.AttributeSet;
|
||||
import android.view.MotionEvent;
|
||||
import android.view.View;
|
||||
import android.webkit.WebView;
|
||||
|
||||
public class TitleWebView extends WebView{
|
||||
|
||||
public TitleWebView(Context context, AttributeSet attrs){
|
||||
super(context, attrs);
|
||||
}
|
||||
|
||||
private int titleHeight;
|
||||
|
||||
@Override
|
||||
protected void onMeasure(int widthMeasureSpec, int heightMeasureSpec){
|
||||
super.onMeasure(widthMeasureSpec, heightMeasureSpec);
|
||||
// determine height of title bar
|
||||
View title = getChildAt(0);
|
||||
titleHeight = title==null ? 0 : title.getMeasuredHeight();
|
||||
}
|
||||
|
||||
@Override
|
||||
public boolean onInterceptTouchEvent(MotionEvent ev){
|
||||
return true; // don't pass our touch events to children (title bar), we send these in dispatchTouchEvent
|
||||
}
|
||||
|
||||
private boolean touchInTitleBar;
|
||||
@Override
|
||||
public boolean dispatchTouchEvent(MotionEvent me){
|
||||
|
||||
boolean wasInTitle = false;
|
||||
switch(me.getActionMasked()){
|
||||
case MotionEvent.ACTION_DOWN:
|
||||
touchInTitleBar = (me.getY() <= visibleTitleHeight());
|
||||
break;
|
||||
|
||||
case MotionEvent.ACTION_UP:
|
||||
case MotionEvent.ACTION_CANCEL:
|
||||
wasInTitle = touchInTitleBar;
|
||||
touchInTitleBar = false;
|
||||
break;
|
||||
}
|
||||
if(touchInTitleBar || wasInTitle) {
|
||||
View title = getChildAt(0);
|
||||
if(title!=null) {
|
||||
// this touch belongs to title bar, dispatch it here
|
||||
me.offsetLocation(0, getScrollY());
|
||||
return title.dispatchTouchEvent(me);
|
||||
}
|
||||
}
|
||||
// this is our touch, offset and process
|
||||
me.offsetLocation(0, -titleHeight);
|
||||
return super.dispatchTouchEvent(me);
|
||||
}
|
||||
|
||||
/**
|
||||
* @return visible height of title (may return negative values)
|
||||
*/
|
||||
private int visibleTitleHeight(){
|
||||
return titleHeight-getScrollY();
|
||||
}
|
||||
|
||||
@Override
|
||||
protected void onScrollChanged(int l, int t, int oldl, int oldt){
|
||||
super.onScrollChanged(l, t, oldl, oldt);
|
||||
View title = getChildAt(0);
|
||||
if(title!=null) // undo horizontal scroll, so that title scrolls only vertically
|
||||
title.offsetLeftAndRight(l - title.getLeft());
|
||||
}
|
||||
|
||||
@Override
|
||||
protected void onDraw(Canvas c){
|
||||
|
||||
c.save();
|
||||
int tH = visibleTitleHeight();
|
||||
if(tH>0) {
|
||||
// clip so that it doesn't clear background under title bar
|
||||
int sx = getScrollX(), sy = getScrollY();
|
||||
c.clipRect(sx, sy+tH, sx+getWidth(), sy+getHeight());
|
||||
}
|
||||
c.translate(0, titleHeight);
|
||||
super.onDraw(c);
|
||||
c.restore();
|
||||
}
|
||||
}
|
@ -211,7 +211,7 @@ public class OfflineArticleFragment extends Fragment implements GestureDetector.
|
||||
getActivity().getTheme().resolveAttribute(R.attr.linkColor, tv, true);
|
||||
|
||||
// prevent flicker in ics
|
||||
if (!m_prefs.getBoolean("webview_hardware_accel", true)) {
|
||||
if (android.os.Build.VERSION.SDK_INT >= 11) {
|
||||
web.setLayerType(View.LAYER_TYPE_SOFTWARE, null);
|
||||
}
|
||||
|
||||
@ -280,7 +280,7 @@ public class OfflineArticleFragment extends Fragment implements GestureDetector.
|
||||
cssOverride +
|
||||
"</style>" +
|
||||
"</head>" +
|
||||
"<body>" + articleContent + "</body></html>";
|
||||
"<body>" + articleContent + "<p> </p><p> </p><p> </p><p> </p></body></html>";
|
||||
|
||||
try {
|
||||
String baseUrl = null;
|
||||
|
@ -1,34 +0,0 @@
|
||||
package org.fox.ttrss.util;
|
||||
|
||||
import android.content.Context;
|
||||
import android.util.AttributeSet;
|
||||
import android.view.View;
|
||||
import android.webkit.WebView;
|
||||
import android.widget.ScrollView;
|
||||
|
||||
public class NoChildFocusScrollView extends ScrollView {
|
||||
|
||||
public NoChildFocusScrollView(Context context) {
|
||||
super(context);
|
||||
// TODO Auto-generated constructor stub
|
||||
}
|
||||
|
||||
|
||||
public NoChildFocusScrollView(Context context, AttributeSet attrs) {
|
||||
super(context, attrs);
|
||||
// TODO Auto-generated constructor stub
|
||||
}
|
||||
|
||||
public NoChildFocusScrollView(Context context, AttributeSet attrs,
|
||||
int defStyle) {
|
||||
super(context, attrs, defStyle);
|
||||
// TODO Auto-generated constructor stub
|
||||
}
|
||||
|
||||
@Override
|
||||
public void requestChildFocus(View child, View focused) {
|
||||
if (focused instanceof WebView )
|
||||
return;
|
||||
super.requestChildFocus(child, focused);
|
||||
}
|
||||
}
|
Loading…
Reference in New Issue
Block a user