summaryrefslogtreecommitdiff
path: root/src/com/android/mail/ui/AbstractActivityController.java
Commit message (Collapse)AuthorAgeFilesLines
* Pass selected folder to message search.Danny Baumann2017-05-241-1/+4
| | | | | | Allows searching in selected folder instead of inbox. Change-Id: I251a34b85aa35997a9d7c5705d4e206c8c88f564
* Revert "UnifiedEmail: Supported Email search function"Danny Baumann2016-12-301-187/+48
| | | | | | This reverts commit ad4f76791d5306f6046150cefd7de07d9d15aa33. Change-Id: I396973b7f27a5df483a7845b16b1db804bb8916b
* Revert "UnifiedEmail: exit search will crash NPE."Michael Bestas2016-12-301-3/+1
| | | | This reverts commit 31e09729acae86fb54ab0195ccc44ee5fc075e65.
* UnifiedEmail: exit search will crash NPE.shiwan2016-11-201-1/+3
| | | | | | | | | when send a email in the automatic test. email exit search function,the mfolder is null. should be add judge of null about the mfolder paramter Change-Id: Ic68f6b60a1160b7e879b94022fadf7c1601e07a6 CRs-Fixed: 1090019
* UnifiedEmail: Supported Email search functionweitan2016-08-241-48/+187
| | | | | | | | | | | | | | | | | | | 1. Use the search function to search the local mail. 2. Search results interface: Filters All, Subject, Sender, Receiver click on the different filter, and search the corresponding results. All for the default selection filter. 3. Search results interface: slide switch filter, remove "sliding removing a single mail" effect. 4. Search results interface: alternative condition, slide switch filter, cancel the multiselect state. 5. Search results interface: UI-filters layout, sliding effect, search, according to the number of search results highlighted. Change-Id: I2af40a386de55e221878bd346cda3ea1cff8f2fa CRs-Fixed: 1023498
* UnifiedEmail: Support configure the sync size for each mail contentweitan2016-08-241-0/+22
| | | | | | | | Email Application will fetch/sync the mail content as the sync size to save user's data. Change-Id: Idfd1b671495544cb44b488355f08731afb2f3794 CRs-Fixed: 1016744
* Prevent rare NPETony Mantler2014-10-161-1/+3
| | | | | | b/18017065 Change-Id: I3f6c8bfcd0650adc32a601120f3dfade23eff0cc
* Peek mode for 2-pane landscapeAndy Huang2014-10-151-33/+97
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Peek mode allows CV fragments to be visible but not marked viewed and read. TPC keeps a global flag for whether the current conversation should be peeked vs marked read. I have tried to manually reset the flag in the 6 cases that cause the current conversation to change: 1. tap another conversation (don't peek) 2. swipe to another conversation (don't peek) 3. keyboard-advance to another conversation (peek) 4. auto-advance=newer|older (don't peek) 5. auto-advance=list (go older & peek) 6. empty CV and a list cursor has loaded (peek at i=0) Swipe, in particular, required a new onConversationViewSwitched() call in CPA because onPageSelected() is triggered both when swiping around in the view pager AND when simply setting the initial page as part of constructing a view pager. Thankfully, there is only ever one copy of this peek flag in the app, in TPC, since AAC/TPC is where all the work is done both to mark conversations seen and to switch to new conversations. Rotation to portrait on tablets now requires tearing down a ViewPager; this code and thought process is documented here: http://go/xqaxk TODO: selected indicator for transition from peek->read state in-place TODO: touches in CVF should mark it read Bug: 17291366 Change-Id: I24f71a2b7985773814d8caad9f3ab3fe5c3609c8
* Add Vector Drawables (UnifiedEmail)Rohan Shah2014-10-091-3/+3
| | | | | | | | Had to rename a couple resources to be consistent with what UX has in drive. Bug: 17936539 Change-Id: I9ca2c0ec870afcdea52c1fbf1e55468b930301ae
* Merge "Remove analytics for power experiment" into ub-gmail-ur14-devJin Cao2014-10-061-12/+0
|\
| * Remove analytics for power experimentJin Cao2014-09-301-12/+0
| | | | | | | | | | | | b/17508768 Change-Id: I5140e43f744711252c0daf6742e9e77ee0465b9f
* | Fix TL flicker before CV shows upon opening notificationRohan Shah2014-10-031-3/+13
| | | | | | | | | | | | | | | | | | Pager no longer animates in unless we've selected a conversation from the TL (In all other cases, it will now directly appear). Bug: 17808571 Change-Id: Iadabdfa6ec913b1f5f9979dfe50c74eaeb51d326
* | Fix message starring via overflowTony Mantler2014-10-021-1/+1
|/ | | | | | | | | Fixes a case where we were unstarring a message and then asking a different copy of it if it's starred or not. b/17734406 Change-Id: I1f032f11241a55743355cfac5f4b7f3bb044b872
* Merge "New empty states for tablet landscape" into ub-gmail-ur14-devJin Cao2014-10-011-0/+5
|\
| * New empty states for tablet landscapeJin Cao2014-09-301-0/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | For tablet landscape, the empty icon should be shown in the CV pane while the empty text (if the folder is empty) should be shown in the list pane. Added a new empty icon for not having any selected conversation, this state does not have text associated with it since the list pane still showed the available items. b/16876067 Change-Id: I27b7b9ae8ce739f23829afc292df083e8399a118
* | Display sync errors using snackbar and not as a TL footerJames Lemieux2014-09-301-34/+5
|/ | | | | | | | | | | | | | | | | | | | | | b/16463253 The FAB compose button overlaps the action button found in the TL footer when network errors occur during sync. To avoid this overlap, the snackbar is used to display these errors and they no longer appear as a TL footer. In order to signal the sync error to AAC for display in the snackbar, the Folder.lastSyncResult needed to be encoded in the manner that AAC reads. This was not happening for POP/IMAP/Exchange accounts, so a large portion of this change is encoding that value properly every place it is written. To ensure the value is read/written properly everywhere, common methods were introduced in UIProvider that do this work. UIProviderTest was also added to ensure the read/write methods agree with each other. Finally, the display of the "Load More" TL footer was updated to match the latest spec. Change-Id: Ia44f4ca1caa77c5d76f58d75fa4ab308442d2a72
* Merge "Set drawer icons (burger and back) instead of using default" into ↵Rohan Shah2014-09-291-17/+19
|\ | | | | | | ub-gmail-ur14-dev
| * Set drawer icons (burger and back) instead of using defaultRohan Shah2014-09-291-17/+19
| | | | | | | | | | | | | | | | | | | | | | | | | | We don't want the animation to show, since it seems broken when we're partially dragging in/out the drawer. So, we set the icons as we do in TPL. Also, fixed the case where dragging out the drawer resulted in the icon changing for CV. Bug: 17629517 Bug: 17410604 Change-Id: I5fbb8d7af28af23a9b8dddb7801f7c687c11d557
* | Only allow for targetted menu options if the cursor is readyJin Cao2014-09-291-2/+4
|/ | | | | | b/17645730 Change-Id: Ifb864cf73217d0f1e80878a88c208277cad8cb26
* Merge "New keyboard shortcuts" into ub-gmail-ur14-devJin Cao2014-09-251-84/+109
|\
| * New keyboard shortcutsJin Cao2014-09-251-84/+109
| | | | | | | | | | | | | | | | | | Does not include ctrl+ENTER for SEND & we need a new key for REPLY since we are not supporting shift. b/17070243 Change-Id: Ia5f179336dfc2b6631e1a5f26ee888f82f4873c0
* | Merge "Bail out of loader callbacks if activity is destroyed" into ↵Tony Mantler2014-09-241-0/+9
|\ \ | | | | | | | | | ub-gmail-ur14-dev
| * | Bail out of loader callbacks if activity is destroyedTony Mantler2014-09-241-0/+9
| |/ | | | | | | | | | | b/17613892 Change-Id: I6bd046bb047939fcd79f4da7954b647dbd46fdc7
* | Merge "More keyboard polish" into ub-gmail-ur14-devJin Cao2014-09-241-2/+2
|\ \ | |/ |/|
| * More keyboard polishJin Cao2014-09-181-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Weird condition where user presses right to go into a conversation, we load the conversation then set mConversationToShow = null. But then our posted runnable for "peeking" the conversation completes, and we show a null conversation which defaults to the first conv. - Don't focus if the pager can't find anything to focus. This prevents the focus from being put to default location (top left) if onRequestFocusInDescendants fail. - When the user taps a conversation, the list loses selectedItem. Use checked item as fallback for navigation. However, sometimes the listview loses focus when the user taps a conversation, thus it wont receive any onKey events. I'll try to figure that out later. - Check target size before commiting the undo action for toast bar. b/17570560 b/17567978 Change-Id: I3d471dc1c1d956957463198086112d5a352fb775
* | Remove previous code for moving FAB into TLJin Cao2014-09-181-1/+2
|/ | | | | | | | | Use ray's approach of moving the view into TL so we won't have to keep these states for animating the FAB. b/17514925 Change-Id: Ib1c6328158a12defba9fc0c13640e1814481f89f
* [Quantum search] new two-pane land search result listJin Cao2014-09-171-0/+1
| | | | | | | | | | In two-pane landscape, change the search bar accordingly when we are in conversation mode (so the menu items are visible). b/17361017 Change-Id: Icb07fff91f0533681478d866430fbaa82bc5850c
* fix slow 2-pane transitions; fix 2-pane animations in portraitAndy Huang2014-09-161-11/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | The instant visibility changes on layout after mode changes were preventing the animations from running. Defer any cases when moving to invisible state until after the transition is complete. Move FAB reveal earlier when going CV->TL on 2-pane portrait by triggering list-visibility-changed manually when entering TL mode. TL- >CV hides the FAB after transition complete as usual. Position and animate 2-pane elements after normal layout, since some size calculations depend on having performed a layout already. Fixes some FAB-related race conditions I started seeing with the animations working again. b/17514925 tracks the fact that FAB shouldn't need to move about in this way in the first place (since it's pretty much always tethered to TL). Also short-circuit an infinite loop that triggered continuous relayouts when an Ad is present. This change reverts to the older logic of only positioning/animating panels on known layout change events, which also has the side effect of smoothing out the animation. Bug: 17477114 Bug: 16818991 Change-Id: Ice1f6b896b9d7dc434debb3c2cf15aec87551ebc
* Log when users manually refreshJin Cao2014-09-151-0/+12
| | | | | | b/17508768 Change-Id: I61cd3a5a655c125a6cf762ccabbfe80243699658
* Welcome tour smooth launchGreg Bullock2014-09-121-36/+5
| | | | | | | | | | | Remove the loader mechanism and instead have the activity determine if the tour is required. Also a minor refactoring of moving the VOICE_SEARCH_REQUEST_CODE from MaterialSearchViewController into AbstractActivityController. Bug: 16124942 "WarmWelcome screen is shown after Inbox view is presented..." Change-Id: Icd0ed2a976d8e8acc387336d1a81c690ab961efb
* Use v7 ActionBarDrawerToggle. v4 is deprecated.Andrew Sapperstein2014-09-101-3/+3
| | | | | | b/17425762 Change-Id: Ieeaf52eec9a7f01467334d7d2f544da6a561b16a
* Update "New message" toast bar in CVJames Lemieux2014-09-031-0/+1
| | | | | | | | | | | | | | b/17373672 The bar in conversation view prompting the user to update the thread with newly arrived messages has been changed to use the ActionableToastBar to look consistent with conventional snackbar messages. In two pane layout it is now possible to have a conventional snackbar message showing over the TL and "new message" snackbar showing over the CV simultaneously. All snackbar messages have had their trailing punctuation (period) removed. Change-Id: Iaf7d5d9afc38360d07a7b6602d6bc5626ecc3de2
* Add a delay to loading peeked conversationsJin Cao2014-09-021-2/+2
| | | | | | | | | | | | | Don't instantly start loading a peeked conversation when the user is navigating with the keyboard. This prevents the app from freaking out when the user is trying to quickly scroll down the TL. Current delay is 500ms. b/3304236 Change-Id: I5bbdafc5187a7f2b96e28aed1b60b694d4565db1
* Merge "Keyboard support for focused state" into ub-gmail-ur14-devJin Cao2014-09-031-23/+23
|\
| * Keyboard support for focused stateJin Cao2014-09-021-23/+23
| | | | | | | | | | | | | | | | | | | | | | | | | | | | - remove focusable from FAB so keyboard won't navigate to it. - rename selectionSet to checkedSet - selected is a state for listview to focus one of its elements - what we really want is checked state for checked items in CAB mode. - show the blue focus bar when isSelected returns true - this is auto-set by the framework as we navigate using the keyboard. - remove blue background color for selected state b/17258708 Change-Id: Ic9739c9b349403942f64fafa26500c08d1f1cc4c
* | Use custom toolbar only for tablets and fix line sizeJin Cao2014-09-021-0/+3
| | | | | | | | | | | | | | | | | | | | | | - only use custom toolbar for tablets so phones won't get any bugs associated with it. - for tablet toolbar title, set it to single line and ellipsize properly in case of long title. - also align title text accordingly. b/17332114 b/17331396 b/17360830 Change-Id: Ib7eecf684303a877a228d7453e2df8825842b6ba
* | Pressing back in CAB mode should close CAB.Andrew Sapperstein2014-08-291-0/+4
|/ | | | | | Fixes b/17112203. Change-Id: Ia5aa0ff22620d429cfedff5139f6737766693b08
* remove drag/drop codeAndy Huang2014-08-281-178/+1
| | | | | Bug: 17322937 Change-Id: Idc21156e6d5e9ea4e83e3c17e92a66388ffc7a6a
* Merge "Fix last email archive (No more lost archives!)" into ub-gmail-ur14-devRohan Shah2014-08-271-4/+2
|\
| * Fix last email archive (No more lost archives!)Rohan Shah2014-08-261-4/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Took away the quick return after showNextConversation. It seems to be causing the last action to not archive/delete as there is no conversation to show, which leads to the if registering as true and returning without finishing the deletion. There is still something weird with calling delete twice, but at the least the bug is fixed (I'll continue looking into the unnecessary delete call later at b/17290364) Bug: 9904716 Change-Id: If0af5502dc8363a2e4267dbdcf7c79c06f15dff1
* | Merge "Compose button is only visible in regular convlist" into ↵Jin Cao2014-08-271-1/+8
|\ \ | |/ |/| | | ub-gmail-ur14-dev
| * Compose button is only visible in regular convlistJin Cao2014-08-261-1/+8
| | | | | | | | | | | | | | | | Or regular conv mode if its two-pane landscape. b/17258369 Change-Id: Icf9c82b8ddc7a1d4c42f29761758cda31b48c071
* | Merge "Analytics: new custom dimension to track the type of account." into ↵Régis Décamps2014-08-261-2/+5
|\ \ | |/ |/| | | ub-gmail-ur14-dev
| * Analytics: new custom dimension to track the type of account.Régis Décamps2014-08-251-2/+5
| | | | | | | | | | | | | | | | Add a custom dimension to track the account type. The account domain is already track in dimension #1. Bug: 16316134 analytics for major new features Change-Id: I44998cd327fb9a595f29a48c00ea715bd7d23663
* | Merge "Add custom view for toolbar to reposition search icon" into ↵Jin Cao2014-08-251-3/+0
|\ \ | | | | | | | | | ub-gmail-ur14-dev
| * | Add custom view for toolbar to reposition search iconJin Cao2014-08-251-3/+0
| |/ | | | | | | | | | | | | | | | | | | | | | | Remove search from the default menu items and add the search button to a custom view so we can position it ourselves. This involved extending the toolbar class because custom views cover up the toolbar title by default. b/17070560 Change-Id: Iba7d127a58b8061d84228732fcadbd82665d0835
* / Remove some unnecessary interfaceJin Cao2014-08-251-31/+19
|/ | | | | | | | | Remove unused interface for ActivityController and ControllableActivity. b/17258810 Change-Id: I8edbff09b1a5a3278efad8675e3fc3f71e82d2e4
* Merge "Refactoring: Move Analytics email-providers out of AOSP" into ↵Régis Décamps2014-08-221-7/+3
|\ | | | | | | ub-gmail-ur14-dev
| * Refactoring: Move Analytics email-providers out of AOSPRégis Décamps2014-08-211-6/+2
| | | | | | | | | | | | | | | | | | Google analytics is not used in the AOSP UnifiedEmail project. Therefore AnalyticsUtils should not handle the detection of email providers based on the email address. Bug: 17141023 Better aggregate email providers in dimension "Email Provider" Change-Id: Ib91310ec244906ba6b98fa75f22f5f0cb2f5926c
* | prevent list swipes on 2-pane, part 2Andy Huang2014-08-211-0/+5
| | | | | | | | | | | | | | | | | | Previous CL only worked for the lifetime of a single CLF, so the state reset on account or folder change. Have new CLFs init their state properly. Bug: 17070336 Change-Id: I5208295e6dd2e460fc8b41bec249130708fda713