]> gitweb.fperrin.net Git - Dictionary.git/blobdiff - src/com/hughes/android/dictionary/DictionaryActivity.java
Fixed crashes, Slovenian fix, hide up/down buttons, proper keyboard
[Dictionary.git] / src / com / hughes / android / dictionary / DictionaryActivity.java
index afe2696e1afbf63524627b616644a69ded63645a..fe0e57fb2243c767e8d62b18cba48fabd301bc64 100644 (file)
@@ -19,6 +19,7 @@ import android.app.ListActivity;
 import android.content.Context;
 import android.content.Intent;
 import android.content.SharedPreferences;
+import android.graphics.Color;
 import android.graphics.Typeface;
 import android.net.Uri;
 import android.os.Bundle;
@@ -32,6 +33,7 @@ import android.text.Selection;
 import android.text.Spannable;
 import android.text.TextWatcher;
 import android.text.method.LinkMovementMethod;
+import android.text.style.ClickableSpan;
 import android.text.style.StyleSpan;
 import android.util.Log;
 import android.util.TypedValue;
@@ -44,11 +46,11 @@ import android.view.MenuItem.OnMenuItemClickListener;
 import android.view.MotionEvent;
 import android.view.View;
 import android.view.View.OnClickListener;
+import android.view.View.OnFocusChangeListener;
 import android.view.View.OnLongClickListener;
 import android.view.ViewGroup;
 import android.view.WindowManager;
 import android.view.inputmethod.InputMethodManager;
-import android.widget.AdapterView;
 import android.widget.AdapterView.AdapterContextMenuInfo;
 import android.widget.BaseAdapter;
 import android.widget.Button;
@@ -61,6 +63,7 @@ import android.widget.ListView;
 import android.widget.TableLayout;
 import android.widget.TableRow;
 import android.widget.TextView;
+import android.widget.TextView.BufferType;
 import android.widget.Toast;
 
 import com.hughes.android.dictionary.DictionaryInfo.IndexInfo;
@@ -76,6 +79,7 @@ import com.hughes.android.dictionary.engine.TokenRow;
 import com.hughes.android.dictionary.engine.TransliteratorManager;
 import com.hughes.android.util.IntentLauncher;
 import com.hughes.android.util.NonLinkClickableSpan;
+import com.hughes.util.StringUtil;
 
 import java.io.File;
 import java.io.FileWriter;
@@ -119,9 +123,12 @@ public class DictionaryActivity extends ListActivity {
 
     // package for test.
     final Handler uiHandler = new Handler();
-    
+
     TextToSpeech textToSpeech;
     volatile boolean ttsReady;
+    
+    int textColorFg = Color.BLACK;
+    
 
     private final Executor searchExecutor = Executors.newSingleThreadExecutor(new ThreadFactory() {
         @Override
@@ -203,10 +210,11 @@ public class DictionaryActivity extends ListActivity {
 
         application = (DictionaryApplication) getApplication();
         theme = application.getSelectedTheme();
+        textColorFg = getResources().getColor(theme.tokenRowFgColor);
 
         final Intent intent = getIntent();
         dictFile = new File(intent.getStringExtra(C.DICT_FILE));
-        
+
         ttsReady = false;
         textToSpeech = new TextToSpeech(getApplicationContext(), new OnInitListener() {
             @Override
@@ -215,7 +223,7 @@ public class DictionaryActivity extends ListActivity {
                 updateTTSLanuage();
             }
         });
-
+        
         try {
             final String name = application.getDictionaryName(dictFile.getName());
             this.setTitle("QuickDic: " + name);
@@ -310,6 +318,12 @@ public class DictionaryActivity extends ListActivity {
         searchText = (EditText) findViewById(R.id.SearchText);
         searchText.requestFocus();
         searchText.addTextChangedListener(searchTextWatcher);
+        searchText.setOnFocusChangeListener(new OnFocusChangeListener() {
+            @Override
+            public void onFocusChange(View v, boolean hasFocus) {
+                Log.d(LOG, "searchText onFocusChange hasFocus=" + hasFocus);
+            }
+        });
 
         // Set the search text from the intent, then the saved state.
         String text = getIntent().getStringExtra(C.SEARCH_TOKEN);
@@ -359,26 +373,32 @@ public class DictionaryActivity extends ListActivity {
                 onUpDownButton(false);
             }
         });
+        upButton.setVisibility(PreferenceManager.getDefaultSharedPreferences(this)
+                .getBoolean(getString(R.string.showPrevNextButtonsKey), true) ? View.VISIBLE
+                : View.GONE);
+        downButton.setVisibility(PreferenceManager.getDefaultSharedPreferences(this)
+                .getBoolean(getString(R.string.showPrevNextButtonsKey), true) ? View.VISIBLE
+                : View.GONE);
 
-        getListView().setOnItemSelectedListener(new ListView.OnItemSelectedListener() {
-            @Override
-            public void onItemSelected(AdapterView<?> adapterView, View arg1, final int position,
-                    long id) {
-                if (!searchText.isFocused()) {
-                    if (!isFiltered()) {
-                        final RowBase row = (RowBase) getListAdapter().getItem(position);
-                        Log.d(LOG, "onItemSelected: " + row.index());
-                        final TokenRow tokenRow = row.getTokenRow(true);
-                        searchText.setText(tokenRow.getToken());
-                    }
-                }
-            }
-
-            @Override
-            public void onNothingSelected(AdapterView<?> arg0) {
-            }
-        });
-
+//        getListView().setOnItemSelectedListener(new ListView.OnItemSelectedListener() {
+//            @Override
+//            public void onItemSelected(AdapterView<?> adapterView, View arg1, final int position,
+//                    long id) {
+//                if (!searchText.isFocused()) {
+//                    if (!isFiltered()) {
+//                        final RowBase row = (RowBase) getListAdapter().getItem(position);
+//                        Log.d(LOG, "onItemSelected: " + row.index());
+//                        final TokenRow tokenRow = row.getTokenRow(true);
+//                        searchText.setText(tokenRow.getToken());
+//                    }
+//                }
+//            }
+//
+//            @Override
+//            public void onNothingSelected(AdapterView<?> arg0) {
+//            }
+//        });
+//
         // ContextMenu.
         registerForContextMenu(getListView());
 
@@ -414,12 +434,12 @@ public class DictionaryActivity extends ListActivity {
     protected void onPause() {
         super.onPause();
     }
-    
+
     @Override
     protected void onActivityResult(int requestCode, int resultCode, Intent result) {
         super.onActivityResult(requestCode, resultCode, result);
         if (result != null && result.hasExtra(C.SEARCH_TOKEN)) {
-            Log.d(LOG, "onActivityResult: " + result.toString());
+            Log.d(LOG, "onActivityResult: " + result.getStringExtra(C.SEARCH_TOKEN));
             jumpToTextFromHyperLink(result.getStringExtra(C.SEARCH_TOKEN), indexIndex);
         }
     }
@@ -470,9 +490,18 @@ public class DictionaryActivity extends ListActivity {
     }
 
     private void showKeyboard() {
-        Log.d(LOG, "Trying to show soft keyboard.");
-        final InputMethodManager manager = (InputMethodManager) getSystemService(Context.INPUT_METHOD_SERVICE);
-        manager.showSoftInput(searchText, InputMethodManager.SHOW_IMPLICIT);
+        searchText.postDelayed(new Runnable() {
+            @Override
+            public void run() {
+                Log.d(LOG, "Trying to show soft keyboard.");
+                final boolean searchTextHadFocus = searchText.hasFocus();
+                searchText.requestFocus();
+                final InputMethodManager manager = (InputMethodManager) getSystemService(Context.INPUT_METHOD_SERVICE);
+                manager.showSoftInput(searchText, InputMethodManager.SHOW_IMPLICIT);
+                if (!searchTextHadFocus) {
+                    defocusSearchText();
+                }
+            }}, 100);
     }
 
     void updateLangButton() {
@@ -487,16 +516,18 @@ public class DictionaryActivity extends ListActivity {
         // }
         updateTTSLanuage();
     }
-    
+
     private void updateTTSLanuage() {
-        if (!ttsReady) {
+        if (!ttsReady || index == null || textToSpeech == null) {
+            Log.d(LOG, "Can't updateTTSLanguage.");
             return;
         }
         final Locale locale = new Locale(index.sortLanguage.getIsoCode());
         Log.d(LOG, "Setting TTS locale to: " + locale);
         final int ttsResult = textToSpeech.setLanguage(locale);
-        if (ttsResult != TextToSpeech.SUCCESS) {
-            Log.e(LOG, "TTS not available in this language.");
+        if (ttsResult != TextToSpeech.LANG_AVAILABLE || 
+            ttsResult != TextToSpeech.LANG_COUNTRY_AVAILABLE) {
+            Log.e(LOG, "TTS not available in this language: ttsResult=" + ttsResult);
         }
     }
 
@@ -505,7 +536,7 @@ public class DictionaryActivity extends ListActivity {
             currentSearchOperation.interrupted.set(true);
             currentSearchOperation = null;
         }
-        changeIndexGetFocusAndResearch((indexIndex + 1) % dictionary.indices.size());
+        changeIndexAndResearch((indexIndex + 1) % dictionary.indices.size());
     }
 
     void onLanguageButtonLongClick(final Context context) {
@@ -546,19 +577,20 @@ public class DictionaryActivity extends ListActivity {
                 final DictionaryInfo dictionaryInfo = getItem(position);
 
                 final LinearLayout result = new LinearLayout(parent.getContext());
-                
+
                 for (int i = 0; i < dictionaryInfo.indexInfos.size(); ++i) {
                     if (i > 0) {
                         final TextView dash = new TextView(parent.getContext());
                         dash.setText("-");
                         result.addView(dash);
                     }
-                    
+
                     final IndexInfo indexInfo = dictionaryInfo.indexInfos.get(i);
                     final Button button = new Button(parent.getContext());
                     button.setText(indexInfo.shortName);
                     final IntentLauncher intentLauncher = new IntentLauncher(parent.getContext(),
-                            getLaunchIntent(application.getPath(dictionaryInfo.uncompressedFilename),
+                            getLaunchIntent(
+                                    application.getPath(dictionaryInfo.uncompressedFilename),
                                     i, searchText.getText().toString())) {
                         @Override
                         protected void onGo() {
@@ -568,9 +600,9 @@ public class DictionaryActivity extends ListActivity {
                     };
                     button.setOnClickListener(intentLauncher);
                     result.addView(button);
-                    
+
                 }
-                
+
                 final TextView nameView = new TextView(parent.getContext());
                 final String name = application
                         .getDictionaryName(dictionaryInfo.uncompressedFilename);
@@ -604,7 +636,7 @@ public class DictionaryActivity extends ListActivity {
         dialog.show();
     }
 
-    private void changeIndexGetFocusAndResearch(int newIndex) {
+    private void changeIndexAndResearch(int newIndex) {
         Log.d(LOG, "Changing index to: " + newIndex);
         if (newIndex == -1) {
             Log.e(LOG, "Invalid index.");
@@ -614,11 +646,10 @@ public class DictionaryActivity extends ListActivity {
         index = dictionary.indices.get(indexIndex);
         indexAdapter = new IndexAdapter(index);
         Log.d(LOG, "changingIndex, newLang=" + index.longName);
+        setDictionaryPrefs(this, dictFile, indexIndex, searchText.getText().toString());
         setListAdapter(indexAdapter);
         updateLangButton();
-        searchText.requestFocus(); // Otherwise, nothing may happen.
-        onSearchTextChange(searchText.getText().toString());
-        setDictionaryPrefs(this, dictFile, indexIndex, searchText.getText().toString());
+        setSearchText(searchText.getText().toString(), true);
     }
 
     void onUpDownButton(final boolean up) {
@@ -641,14 +672,9 @@ public class DictionaryActivity extends ListActivity {
         }
         final Index.IndexEntry dest = index.sortedIndexEntries.get(destIndexEntry);
         Log.d(LOG, "onUpDownButton, destIndexEntry=" + dest.token);
-        searchText.removeTextChangedListener(searchTextWatcher);
-        searchText.setText(dest.token);
-        if (searchText.getLayout() != null) {
-            // Surprising, but this can otherwise crash sometimes...
-            Selection.moveToRightEdge(searchText.getText(), searchText.getLayout());
-        }
+        setSearchText(dest.token, false);
         jumpToRow(index.sortedIndexEntries.get(destIndexEntry).startRow);
-        searchText.addTextChangedListener(searchTextWatcher);
+        defocusSearchText();
     }
 
     // --------------------------------------------------------------------------
@@ -661,17 +687,17 @@ public class DictionaryActivity extends ListActivity {
     public boolean onCreateOptionsMenu(final Menu menu) {
         application.onCreateGlobalOptionsMenu(this, menu);
 
-        {
-            final MenuItem randomWord = menu.add(getString(R.string.randomWord));
-            randomWord.setOnMenuItemClickListener(new OnMenuItemClickListener() {
-                public boolean onMenuItemClick(final MenuItem menuItem) {
-                    final String word = index.sortedIndexEntries.get(random
-                            .nextInt(index.sortedIndexEntries.size())).token;
-                    setSearchText(word, true);
-                    return false;
-                }
-            });
-        }
+//        {
+//            final MenuItem randomWord = menu.add(getString(R.string.randomWord));
+//            randomWord.setOnMenuItemClickListener(new OnMenuItemClickListener() {
+//                public boolean onMenuItemClick(final MenuItem menuItem) {
+//                    final String word = index.sortedIndexEntries.get(random
+//                            .nextInt(index.sortedIndexEntries.size())).token;
+//                    setSearchText(word, true);
+//                    return false;
+//                }
+//            });
+//        }
 
         {
             final MenuItem dictionaryList = menu.add(getString(R.string.dictionaryManager));
@@ -780,19 +806,20 @@ public class DictionaryActivity extends ListActivity {
                 }
             });
         }
-        
-        if (row instanceof TokenRow) {
+
+        if (row instanceof TokenRow && ttsReady) {
             final MenuItem speak = menu.add(R.string.speak);
             speak.setOnMenuItemClickListener(new OnMenuItemClickListener() {
                 @Override
                 public boolean onMenuItemClick(MenuItem item) {
-                    textToSpeech.speak(((TokenRow) row).getToken(), TextToSpeech.QUEUE_FLUSH, new HashMap<String, String>());
+                    textToSpeech.speak(((TokenRow) row).getToken(), TextToSpeech.QUEUE_FLUSH,
+                            new HashMap<String, String>());
                     return false;
                 }
             });
         }
     }
-    
+
     private void jumpToTextFromHyperLink(final String selectedText, final int defaultIndexToUse) {
         int indexToUse = -1;
         for (int i = 0; i < dictionary.indices.size(); ++i) {
@@ -816,13 +843,12 @@ public class DictionaryActivity extends ListActivity {
             indexToUse = defaultIndexToUse;
         }
         final boolean changeIndex = indexIndex != indexToUse;
-        // If we're not changing index, we have to trigger search:
-        setSearchText(selectedText, !changeIndex); 
         if (changeIndex) {
-            changeIndexGetFocusAndResearch(indexToUse);
+            setSearchText(selectedText, false);
+            changeIndexAndResearch(indexToUse);
+        } else {
+            setSearchText(selectedText, true);
         }
-        // Give focus back to list view because typing is done.
-        getListView().requestFocus();
     }
 
     @Override
@@ -868,7 +894,8 @@ public class DictionaryActivity extends ListActivity {
         getListView().requestFocus();
 
         // Visual indication that a new keystroke will clear the search text.
-        searchText.selectAll();
+        // Doesn't seem to work unless earchText has focus.
+        // searchText.selectAll();
     }
 
     @SuppressWarnings("deprecation")
@@ -976,11 +1003,17 @@ public class DictionaryActivity extends ListActivity {
             }
         }, 20);
     }
-    
+
     private final void jumpToRow(final int row) {
-        Log.d(LOG, "jumpToRow: " + row);
-        setSelection(row);
+        final boolean refocusSearchText = searchText.hasFocus();
+        Log.d(LOG, "jumpToRow: " + row + ", refocusSearchText=" + refocusSearchText);
+        getListView().requestFocusFromTouch();
+        getListView().setSelectionFromTop(row, 0);
         getListView().setSelected(true);
+        if (refocusSearchText) {            
+            searchText.requestFocus();
+        }
+        //Log.d(LOG, "getSelectedItemPosition():" + getSelectedItemPosition());
     }
 
     static final Pattern WHITESPACE = Pattern.compile("\\s+");
@@ -1004,7 +1037,7 @@ public class DictionaryActivity extends ListActivity {
         boolean done = false;
 
         SearchOperation(final String searchText, final Index index) {
-            this.searchText = searchText.trim();
+            this.searchText = StringUtil.normalizeWhitespace(searchText);
             this.index = index;
         }
 
@@ -1021,7 +1054,7 @@ public class DictionaryActivity extends ListActivity {
                     searchResult = index.findInsertionPoint(searchText, interrupted);
                 } else {
                     searchTokens = Arrays.asList(searchTokenArray);
-                    multiWordSearchResult = index.multiWordSearch(searchTokens, interrupted);
+                    multiWordSearchResult = index.multiWordSearch(searchText, searchTokens, interrupted);
                 }
                 Log.d(LOG,
                         "searchText=" + searchText + ", searchDuration="
@@ -1225,9 +1258,11 @@ public class DictionaryActivity extends ListActivity {
             return result;
         }
 
-        private TableLayout getPossibleLinkToHtmlEntryView(final boolean isTokenRow, final String text, final boolean hasMainEntry, final List<HtmlEntry> htmlEntries, final String htmlTextToHighlight, ViewGroup parent, final TableLayout result) {
+        private TableLayout getPossibleLinkToHtmlEntryView(final boolean isTokenRow,
+                final String text, final boolean hasMainEntry, final List<HtmlEntry> htmlEntries,
+                final String htmlTextToHighlight, ViewGroup parent, final TableLayout result) {
             final Context context = parent.getContext();
-            
+
             final TableRow tableRow = new TableRow(result.getContext());
             tableRow.setBackgroundResource(hasMainEntry ? theme.tokenRowMainBg
                     : theme.tokenRowOtherBg);
@@ -1238,8 +1273,15 @@ public class DictionaryActivity extends ListActivity {
             }
             result.addView(tableRow);
 
+            // Make it so we can long-click on these token rows, too:
             final TextView textView = new TextView(context);
-            textView.setText(text);
+            textView.setText(text, BufferType.SPANNABLE);
+            createTokenLinkSpans(textView, (Spannable) textView.getText(), text);
+            final TextViewLongClickListener textViewLongClickListenerIndex0 = new TextViewLongClickListener(
+                    0);
+            textView.setOnLongClickListener(textViewLongClickListenerIndex0);
+            result.setLongClickable(true);
+            
             // Doesn't work:
             // textView.setTextColor(android.R.color.secondary_text_light);
             textView.setTypeface(typeface);
@@ -1251,53 +1293,55 @@ public class DictionaryActivity extends ListActivity {
                 textView.setTextSize(TypedValue.COMPLEX_UNIT_SP, fontSizeSp);
             }
             lp.weight = 1.0f;
-            
+
             textView.setLayoutParams(lp);
             tableRow.addView(textView);
 
-            
             if (!htmlEntries.isEmpty()) {
-                final ImageButton button = new ImageButton(context);
-                button.setImageResource(R.drawable.ic_menu_forward);
-                button.setOnClickListener(new OnClickListener() {
+                final ClickableSpan clickableSpan = new ClickableSpan() {
+                    @Override
+                    public void onClick(View widget) {
+                    }
+                };
+                ((Spannable) textView.getText()).setSpan(clickableSpan, 0, text.length(), Spannable.SPAN_INCLUSIVE_INCLUSIVE);
+                result.setClickable(true);
+                textView.setClickable(true);
+                textView.setMovementMethod(LinkMovementMethod.getInstance());
+                textView.setOnClickListener(new OnClickListener() {
                     @Override
                     public void onClick(View v) {
-                        final String html = HtmlEntry.htmlBody(htmlEntries, index.shortName);
+                        String html = HtmlEntry.htmlBody(htmlEntries, index.shortName);
+                        //Log.d(LOG, "html=" + html);
                         startActivityForResult(
-                                HtmlDisplayActivity.getHtmlIntent(String.format("<html><head></head><body>%s</body></html>", html), htmlTextToHighlight, false),
+                                HtmlDisplayActivity.getHtmlIntent(String.format(
+                                        "<html><head></head><body>%s</body></html>", html),
+                                        htmlTextToHighlight, false),
                                 0);
                     }
                 });
-                tableRow.addView(button);
-                lp = new TableRow.LayoutParams(1);
-                lp.weight = 0.0f;
-                button.setLayoutParams(lp);
-                //result.setColumnStretchable(0, true);
-                //result.setColumnStretchable(1, false);
             }
-            result.setLongClickable(true);
             return result;
         }
-        
+
         private TableLayout getView(TokenRow row, ViewGroup parent, final TableLayout result) {
             final IndexEntry indexEntry = row.getIndexEntry();
-            return getPossibleLinkToHtmlEntryView(true, indexEntry.token, row.hasMainEntry, indexEntry.htmlEntries, null, parent, result);
+            return getPossibleLinkToHtmlEntryView(true, indexEntry.token, row.hasMainEntry,
+                    indexEntry.htmlEntries, null, parent, result);
         }
-        
+
         private TableLayout getView(HtmlEntry.Row row, ViewGroup parent, final TableLayout result) {
             final HtmlEntry htmlEntry = row.getEntry();
             final TokenRow tokenRow = row.getTokenRow(true);
-            return getPossibleLinkToHtmlEntryView(false, getString(R.string.seeAlso, htmlEntry.title, htmlEntry.entrySource.getName()), 
-                    false, Collections.singletonList(htmlEntry), tokenRow.getToken(), parent, result);
+            return getPossibleLinkToHtmlEntryView(false,
+                    getString(R.string.seeAlso, htmlEntry.title, htmlEntry.entrySource.getName()),
+                    false, Collections.singletonList(htmlEntry), tokenRow.getToken(), parent,
+                    result);
         }
 
-
     }
-    
-
 
     static final Pattern CHAR_DASH = Pattern.compile("['\\p{L}\\p{M}\\p{N}]+");
-
+    
     private void createTokenLinkSpans(final TextView textView, final Spannable spannable,
             final String text) {
         // Saw from the source code that LinkMovementMethod sets the selection!
@@ -1305,7 +1349,7 @@ public class DictionaryActivity extends ListActivity {
         textView.setMovementMethod(LinkMovementMethod.getInstance());
         final Matcher matcher = CHAR_DASH.matcher(text);
         while (matcher.find()) {
-            spannable.setSpan(new NonLinkClickableSpan(), matcher.start(), matcher.end(),
+            spannable.setSpan(new NonLinkClickableSpan(textColorFg), matcher.start(), matcher.end(),
                     Spannable.SPAN_INCLUSIVE_EXCLUSIVE);
         }
     }
@@ -1387,7 +1431,7 @@ public class DictionaryActivity extends ListActivity {
     private class SearchTextWatcher implements TextWatcher {
         public void afterTextChanged(final Editable searchTextEditable) {
             if (searchText.hasFocus()) {
-                Log.d(LOG, "Search text changed with focus: " + searchText.getText());
+                Log.d(LOG, "SearchTextWatcher: Search text changed with focus: " + searchText.getText());
                 // If they were typing to cause the change, update the UI.
                 onSearchTextChange(searchText.getText().toString());
             }