| bc47398187c6ffd132435e51d8d61e6ec79a79db |
|
25-Sep-2012 |
Paul Westbrook <pwestbro@google.com> |
Merge Email1 into MR1
Change-Id: I45289d46b65faffc7a3a3dd46382899162f3aaab
/packages/apps/Email/src/com/android/email/activity/MailboxListFragment.java
|
| cc0185f07c9198008d8dc685ae9979f3e35e8539 |
|
28-Jun-2012 |
Marc Blank <mblank@google.com> |
Manual merge of Email from jb-ub-mail
Change-Id: I564ddf433d371f45ba78a10f73bb182d484c56f2
/packages/apps/Email/src/com/android/email/activity/MailboxListFragment.java
|
| 513486cfb5d446f098b1c97d16932e51e316d1a7 |
|
13-Oct-2011 |
Ben Komalo <benkomalo@google.com> |
Fix per account starred mailboxes.
We always switched to combined view when we tapped "Starred" in a
mailbox list. This was confusing since tapping to show the mailbox list
will take you to the mailbox list for the combined view, even if there's
only one account on the device! Other confusing things happened like
showing coloured account chips, even if there's only one account.
This fixes it so that we pass the account ID when building the
selection. Lots of wiring to make it happen, but very little change
overall.
Bug: 5388326
Change-Id: I1fe52f211cceca0c1b26581e200f3c7adcd0734a
/packages/apps/Email/src/com/android/email/activity/MailboxListFragment.java
|
| ce4ebc52a6fd7e2084eb6b54f9147269329aa87b |
|
07-Oct-2011 |
Ben Komalo <benkomalo@google.com> |
Ensure proper mailbox is highlighted.
Bug: 5424565
Change-Id: I1436568aa1c7d7d2becce27ee068d88deb217496
/packages/apps/Email/src/com/android/email/activity/MailboxListFragment.java
|
| 2845d72af03b26804fb305202646a9ede234e38e |
|
16-Jul-2011 |
Ben Komalo <benkomalo@google.com> |
Remove temp code that was causing an NPE
- since listContext can be null, this code was not safe.
we also filter out the search mailbox too, so it's no longer needed.
- don't ask to highlight a mailbox if doing a search
- remove a call in MailboxListFragment that was unconditionally telling
callbacks that something was selected when we started loading - this was
technically lying and if the item isn't in the list that was selected,
nothing should be called (as in the case of search). This was just an
optimization anyways and that callback is invoked later when the mailbox
list load completes.
Bug: 5037629
Change-Id: Id31c6795af9e64fa8682b67de9ab90540ee660df
/packages/apps/Email/src/com/android/email/activity/MailboxListFragment.java
|
| 8b72c7eb8defbf32fde6f650dc855bdf06e801a2 |
|
02-Jul-2011 |
Makoto Onuki <omakoto@google.com> |
Work around for bug 4674712
Seems like we're getting onItemClick() on header rows, which are
supposed to be disabled.
Change-Id: I03b108c508f46368cc420049eed67438af7a7735
/packages/apps/Email/src/com/android/email/activity/MailboxListFragment.java
|
| 9630e31bac9232528ab9a25d7fcc96fda647f165 |
|
01-Jul-2011 |
Makoto Onuki <omakoto@google.com> |
Fix bug 4901592 Do not drill-in to inbox by default
If the initial mailbox has child mailboxes, we should show the drilled-in
view by default, EXCEPT if the initial mailbox is inbox, in which case
we want to show the root view.
(And we drill-in if the user taps Inbox.)
Bug 4901592
Change-Id: Idc1462030cf8d971dd4dcbba647ac3a64cab6819
/packages/apps/Email/src/com/android/email/activity/MailboxListFragment.java
|
| 50d934360d51392ac2aa6f11de4d6e1446cf78c9 |
|
29-Jun-2011 |
Makoto Onuki <omakoto@google.com> |
Fix the "onPostExecute executed even when cancelled" issue
Renamed onPostExecute to onSuccess and made sure it won't called
if a task is cancelled in time.
Also removed isCancelled(). To implement it right we should make sure
that onPostExecute() isn't finished when setting mCancelled, but it's a bit
of a pain to implement right, and we don't really have to use it.
Change-Id: I3a0baf504506ffc4952a5553f7098a8415842fa3
/packages/apps/Email/src/com/android/email/activity/MailboxListFragment.java
|
| 4689cb7100a8b1cfa188499a9910912d15b3dad3 |
|
27-Jun-2011 |
Makoto Onuki <omakoto@google.com> |
New account spinner
- Don't use the action bar spinner.
Instead use a custom view to show the current account.
(It's not a spinner; now we show the dropdown list by ourselves,
which gives us more detailed control.)
- Also show the current mailbox name/unread count with the account name.
- Removed the mailbox info loader in ABC.
Instead, now the AccountSelectorAdapter loader loads the current
account/mailbox name, and the unread count as extras.
- Now ABC.Callback.onMailboxSelected passed an account ID as well
as a mailbox ID.
- There's new code paths that can cause the "fragment transaction in
onLoadFinished" exception. We need to fix this properly, but
for now we just use commitAllowingStateLoss().
Bug 4948352
Change-Id: I73bb8b7530f8328ca1c86382ac0a54086ad061d7
/packages/apps/Email/src/com/android/email/activity/MailboxListFragment.java
|
| 80d3875d306c60da83e547c573427627911f8a99 |
|
23-Jun-2011 |
Makoto Onuki <omakoto@google.com> |
Action bar: Use laoder to get current mailbox info
Before this CL, we had this crazy plumbing from MailboxListFragment
to ActionBarController to update the current mailbox name/message count.
This wouldn't work on 1-pane, so now ABC just gets the current mailbox id
from UIC and loads the name/count with its own loader.
Also...
- Fixed bug 4904450 and bug 4460470: Now we consistently use FolderProperties
to get proler display names and message counts.
- Renamed some confusing names in AccountSelectorAdapter
Change-Id: Ic7bea6da6d2859006fb8f9263024c7d5e62b1e7f
/packages/apps/Email/src/com/android/email/activity/MailboxListFragment.java
|
| e10215eaff7c06b44b95de87ea46030065ecbee5 |
|
23-Jun-2011 |
Ben Komalo <benkomalo@google.com> |
Fix some things with shortcuts/widgets.
- All mailboxes will be shown when selecting a shortcut for a normal
account.
- Combined unread won't show up in shortcuts for combined accounts
- only inbox is allowed for IMAP/POP widgets
- all mailboxes allowed for EAS widgets
Bug: 4725896
Change-Id: I5c77c2f5443e330ed451179761b1d9c8c0dd7619
/packages/apps/Email/src/com/android/email/activity/MailboxListFragment.java
|
| d5dfd76cb90a3c73a4d4f82ddfc6b2c7cfcc95e1 |
|
23-Jun-2011 |
Makoto Onuki <omakoto@google.com> |
Fix drag & drop
There was an assumption that the list view would only contain MailboxListItem's.
Now it's not always the case with the headers like "recent folders".
Also killed the timer thing for the nested navigation during DnD.
Bug 4904006
Change-Id: I4ee756775d0115f3a39086758be69100c19163c9
/packages/apps/Email/src/com/android/email/activity/MailboxListFragment.java
|
| f5418f1f93b02e7fab9f15eb201800b65510998e |
|
14-Jun-2011 |
Marc Blank <mblank@google.com> |
Move Account into its own top-level class
Change-Id: Ide7c991b7d4d418dbe17164421425bf898ba64ee
/packages/apps/Email/src/com/android/email/activity/MailboxListFragment.java
|
| dc6928d798d61049a47f4babeaa5aab90c091cc1 |
|
08-Jun-2011 |
Todd Kennedy <toddke@google.com> |
Add recent mailboxes to the mailbox list
Change-Id: I84ee9d0da43ccac4d10e5122f57583c4b39cf65d
/packages/apps/Email/src/com/android/email/activity/MailboxListFragment.java
|
| 1b404f4fff39977e4f8edc9258c1bd1075da11c6 |
|
07-Jun-2011 |
Todd Kennedy <toddke@google.com> |
Collapse MailboxAdapter & MailboxFragmentAdapter
MFA was the only subclass of MA and the abstraction no longer made sense.
Collapse the two into a single adapter class.
Change-Id: I32c6f027bc37f3da08626f743c3f494e5f48c3f0
/packages/apps/Email/src/com/android/email/activity/MailboxListFragment.java
|
| ab40c988216b32ed145c0cad45c25e9cf2509c85 |
|
06-Jun-2011 |
Makoto Onuki <omakoto@google.com> |
More work on fragment install/uninstall.
- Now we "uninstall" a fragment in Fragment.onDestroyView.
i.e. a fragment transaction is actually executed.
- Maintain our own "about to be removed" fragment list to avoid
double removal of a fragment.
Change-Id: I61328e0a09a7af00cbb0e6ba10a2d39c11b5c3dc
/packages/apps/Email/src/com/android/email/activity/MailboxListFragment.java
|
| 345a61d0b97411e9c26d9ec7a79bc6cd1afebd1d |
|
03-Jun-2011 |
Makoto Onuki <omakoto@google.com> |
Manually restore list view state.
Bug 4532222
Change-Id: I6f3394b7ffd55b32e1932e3e425cc0d0ae720e27
/packages/apps/Email/src/com/android/email/activity/MailboxListFragment.java
|
| 3d9b8e76f0a396370a5c0be99a34bf7c24bd20dd |
|
03-Jun-2011 |
Makoto Onuki <omakoto@google.com> |
Implement 1-pane navigation.
- Now that fragment useage is simplified (e.g. no new fragment
creation for nested mailbox navigation), most of the fragment
operation code for 2-pane is reuseable for 1-pane as well,
so moeved it to the base class.
- Temporarily added "Show all folders" as a menu option on 1-pane.
- Added "opener account id/mailbox id" to the message view fragment.
They are not used by the fragment itself, but they're used
by the UI controller for the back navigation. (And now the UI
controller doesn't maintain the current IDs by itself; rather
it gets them from the currently-active fragment.)
- Use async fragment transaction on 1-pane too, now that it always
gets the current state from the active fragment.
- Changed the timing when we install fragments from onAttachFragment
to fragments' onActivityCreated. So now all installed fragments are
created.
TODO Now that all installed fragments are guaranteed to be created,
remove all special trealment for the fragment argument accessor.
(They were meant be safe to call before onCreate, but it's not
necessary any more.)
Change-Id: I0ed100c3f0b460835b164c0dc908ea483a4e46ee
/packages/apps/Email/src/com/android/email/activity/MailboxListFragment.java
|
| 47d6f782de58625ae60dde165125ea04748d11b5 |
|
02-Jun-2011 |
Makoto Onuki <omakoto@google.com> |
Merge "Switch to async fragment transaction for two-pane"
|
| 3a8a1b451dc21eb0e1b05e04b1d4df8971187c84 |
|
02-Jun-2011 |
Makoto Onuki <omakoto@google.com> |
Change on the mailbox list item background.
- The drawable for the active drop target background was static, and
shared between items. But StateListDrawable has inernal state and
shouldn't be shared.
- Also make sure not to set the same background more than once.
For some reason, they will cause visal glitches when we switch to synchronous
fragment transaction.
Change-Id: I10fede2ad4e595f74d61768907b5b70fd5d4da21
/packages/apps/Email/src/com/android/email/activity/MailboxListFragment.java
|
| c7024f73ec95d610ac88fd3d884c131c0499b490 |
|
02-Jun-2011 |
Makoto Onuki <omakoto@google.com> |
Switch to async fragment transaction for two-pane
One-pane will follow too, but not soon.
Change-Id: Ie018f728273e9fa4a7e6cf3116a2a2afebcaecc5
/packages/apps/Email/src/com/android/email/activity/MailboxListFragment.java
|
| 844bf745044b4564f42a68f8b7d40105c4def294 |
|
01-Jun-2011 |
Makoto Onuki <omakoto@google.com> |
MailboxListFragment: In-place nested mailbox navigation
Now we reuse the same instance of the fragment for nested mailbox
navigation. (Don't use fragment transaction)
"CursorWithExtras" now only has the child count, so I removed the
bundle version and added a concrete class to MailboxFragmentAdapter.
With this CL the nested mailbox navigation on 1-pane should work, but
not back navigation. (Back press event isn't connected to the
fragment yet.)
Change-Id: I2c23651d9c8edb5fe062c68bbb9b462c8949ded4
/packages/apps/Email/src/com/android/email/activity/MailboxListFragment.java
|
| 0f84ff2c082dc4ea774f53fe6877752a769ca258 |
|
27-May-2011 |
Todd Kennedy <toddke@google.com> |
Add meta options in the picker
With this CL, we have "All accounts" in the account list and "Inbox" & "Unread"
in the mailbox list. "All" + "Unread" doesn't play very well, so, we should
seriously consider whether it's worth the effort to support it right now.
Change-Id: Ia1c07c89a535a68ffe6f3ae5009b5b6591a9acdb
/packages/apps/Email/src/com/android/email/activity/MailboxListFragment.java
|
| 90e08781ca8ce7f0911924d7c85619ca6b1634d1 |
|
26-May-2011 |
Todd Kennedy <toddke@google.com> |
Make it clear that args are immutable
Change-Id: I54657648fc7fcd145689a51acd8f3c27d3611a97
/packages/apps/Email/src/com/android/email/activity/MailboxListFragment.java
|
| ca6eca0466f9dc58317dd4334da2b951e23f06d7 |
|
25-May-2011 |
Todd Kennedy <toddke@google.com> |
Don't navigate to a shortcut mailbox
All shortcuts now have a mailbox associated with them. When launching
the app via shortcut, we want to show the messages within that mailbox, but,
we do not want to navigate into the mailbox; unless the mailbox has children.
Since we don't want to put too much informatin into the shortcut, we must
perform these checks at runtime. So, if ever we try to load a mailbox that
doesn't contain children, we load it's parent instead.
Change-Id: Idb5dbc7cd740b270a0068811abe685f963ca2c0b
/packages/apps/Email/src/com/android/email/activity/MailboxListFragment.java
|
| d1b7d43df66b073207e0988b2066aa6183afaffe |
|
26-May-2011 |
Makoto Onuki <omakoto@google.com> |
MailboxListFragment: Preparing to use back stack
- Added more lifecycle logs to understand fragment lifecycle better.
- Made sure all code work fine even in the state where a MailboxListFragment
is create but doesn't have views.
- Cleaned up list state resoration code.
Removed the setListAdapter call from onLoadFinished, which lets the
ListView restore the state by itself, in the case where we're re-loading
list for DB changes. We still need to do the restoration manually for
the orientation change case.
Change-Id: I96bff4ace6d48cc02113c2f5bd6a995260122334
/packages/apps/Email/src/com/android/email/activity/MailboxListFragment.java
|
| 3096b4ae18f55c9ebf04d83c534da4d5e3370932 |
|
25-May-2011 |
Makoto Onuki <omakoto@google.com> |
Add Controller.deleteMessages(long[]) for batch delete
- Also removed the accoundId parameter, which wasn't used.
- Also cleaned up MailboxListFragment.onDrop.
(the restored Message was only used to get the account id, but
it's no longer needed.)
Bug 4384642
Change-Id: I8f6635011dae0529a82972617101e1c130090b76
/packages/apps/Email/src/com/android/email/activity/MailboxListFragment.java
|
| d531dc3058c52b1e3ee2ee53eb8d63d6177d9f91 |
|
19-May-2011 |
Makoto Onuki <omakoto@google.com> |
Fix opening Starred mailbox.
We need to pass around the account ID with onMailboxSelected too.
(It's kinda sad it wouldn't have happened if we had par-account starred
mailbox.)
Also made sure MailboxListItem.mMailboxId now really contains only a mailbox
ID. Before this chage, we stored an account ID to this for an account row
on the combined mailbox.
Bug 4452811
Change-Id: I732fd8eb18f787f4a700a45a40768f96e3bb8751
/packages/apps/Email/src/com/android/email/activity/MailboxListFragment.java
|
| 01bd33f318f03d5496b4d252e3a536856405f73c |
|
19-May-2011 |
Makoto Onuki <omakoto@google.com> |
Change to Mailbox/MessageListFragment
- MailboxListFragment.Callback.onMailboxSelected is now split into
two different callbacks, one for regular selection and the
other for DnD for clarity.
- Added onDragStarted/Ended to MessageListFragment.Callback
- Changed Log to use this
Change-Id: I8861d3b43aa1b5b17c68c53605270e99733599f4
/packages/apps/Email/src/com/android/email/activity/MailboxListFragment.java
|
| 49de71e3d272b3070716f36ba574a620cdf62674 |
|
18-May-2011 |
Makoto Onuki <omakoto@google.com> |
Clean up Two pane
- Remove bunch of outdated TODOs
- Removed the stack for the back navigation for nested folders.
Instead, Back/Up will always just navigate to the parent mailbox.
- Two-pane UI controller no longer stores IDs by itself.
Now it always gets these from installed fragments.
- Added UIController.switchAccount(), which is the same as openAccount
except it's no-op if the account is already selected.
- Other small clean-ups.
Change-Id: Ie14bc4f4b6b2cf6afb69bc97967c25682d4c7c78
/packages/apps/Email/src/com/android/email/activity/MailboxListFragment.java
|
| b36ac017926f2557f4a476d0cefe49002d11233b |
|
17-May-2011 |
Makoto Onuki <omakoto@google.com> |
Changes for NO_XXX
- Added Message.NO_MESSAGE
- Renamed PSEUDO_ACCOUNT_ID_NONE to NO_ACCOUNT
- Removed PARENT_KEY_NONE and use NO_MAILBOX instead
- For starters, cleaned up the UI controllers to use them.
Change-Id: I6cfd87ece2fced8e9f7c76d034c4d1dbf9e4db10
/packages/apps/Email/src/com/android/email/activity/MailboxListFragment.java
|
| 54c91f00d7f967690a80b992062e75c40182d088 |
|
17-May-2011 |
Makoto Onuki <omakoto@google.com> |
Use IllegalArgumentException, not InvalidParameterException
Because IllegalArgumentException is the standard one.
Change-Id: Ib4d0bfa88a3d4218148a7cb248d9baa2124a72ba
/packages/apps/Email/src/com/android/email/activity/MailboxListFragment.java
|
| 954f037d8f1e64ae88e02fe18ecf6b72b00f3daa |
|
12-May-2011 |
Makoto Onuki <omakoto@google.com> |
Clean up fragments
Now that we don't reuse fragments and always use newInstance() to pass
arguments, there's a bunch of unnecessary things in them, such as
clearContent().
Also now MessageListFragment takes an account ID as an argument.
Bug 4346486
Change-Id: I7e05628c481ed56512c2281257239105d40ee1bc
/packages/apps/Email/src/com/android/email/activity/MailboxListFragment.java
|
| 53ea83ebf91f820692e8fa8e781f5cc982dd94db |
|
14-May-2011 |
Ben Komalo <benkomalo@google.com> |
Move Mailbox to top level class.
No other changes made.
Change-Id: I3d8f3c521dc0d902be313b25252b4b6a4a96e7ee
/packages/apps/Email/src/com/android/email/activity/MailboxListFragment.java
|
| bfac9f2e8a13f6c719608a6948203bbef921c99f |
|
13-May-2011 |
Makoto Onuki <omakoto@google.com> |
Move some of the log constants from Email to Logging
Change-Id: If9f4e4e3adcdef897a0d6a4e153bb446a8b24fdd
/packages/apps/Email/src/com/android/email/activity/MailboxListFragment.java
|
| 76061eba142586ade8678036b179ffab2ee7a8c0 |
|
13-May-2011 |
Todd Kennedy <toddke@google.com> |
Remove SuppressWarning("unused") wherever possible
Change-Id: Ie799f02ab39a7d020af1fb98b6bac45fc0fd1298
/packages/apps/Email/src/com/android/email/activity/MailboxListFragment.java
|
| 5701e0a555a5c263862156c1291aa13b06850425 |
|
12-May-2011 |
Todd Kennedy <toddke@google.com> |
Suspend notifications for currently shown account
We will suspend notifications whenever we display the message list for an
account (including "combined inbox"). As soon as the message list is paused,
notifications will be resumed.
Change-Id: I481a0f59ce68f89c32210d862d0267f3f334063b
/packages/apps/Email/src/com/android/email/activity/MailboxListFragment.java
|
| 8a894f1ca052b6ca6c5d8cac73fc0f3332f7ec48 |
|
10-May-2011 |
Makoto Onuki <omakoto@google.com> |
Remove MailboxListFragment.DEFAULT_MAILBOX_ID
Now that we supports nested folders on IMAP too, it's not needed.
Change-Id: I80a6d6b59444132762d56ae6e03a5de69863ab8d
/packages/apps/Email/src/com/android/email/activity/MailboxListFragment.java
|
| a9ac20b96f8d2e2e97fb2878afb9df4795024450 |
|
06-May-2011 |
Todd Kennedy <toddke@google.com> |
Add lastSeenMessageKey column to the mailbox table
The last seen message key is the id of the last message that we've "seen" for
that mailbox
Change-Id: I197f8a191654225ada1ce553cc959db775809c88
/packages/apps/Email/src/com/android/email/activity/MailboxListFragment.java
|
| 38847b78fa67b041d3cc82791fcfbc1191b74b11 |
|
04-May-2011 |
Makoto Onuki <omakoto@google.com> |
Remove clearContent and other stuff for reusing fragments
from Mailbox/MessageListFragment.
This fixes some issues with screen rotation.
(We still reuse MessageViewFragement for the phone, but this should eventually
be cleaned up too.)
Also removed obsolete comments/TODOs.
Bug 4341517
Change-Id: Ib3e713ccab023151c1c47d7d7e4cd1c0de74039d
/packages/apps/Email/src/com/android/email/activity/MailboxListFragment.java
|
| 8b6e443cdd231a615e01931970e25bd5ecb79493 |
|
27-Apr-2011 |
Makoto Onuki <omakoto@google.com> |
Merge "Merged activity: move UI from activity to UIController"
|
| aee368d205679a9b29f9955a48a8f413e42833fc |
|
26-Apr-2011 |
Makoto Onuki <omakoto@google.com> |
Merged activity: move UI from activity to UIController
- Renamed XLFragmentManager to UIControllerTwoPane
- Moved UI code from the activity to UIControllerTwoPane
- Bunch of clean-ups (Mostly renames to make things self-descriptive)
- Removed unused class MessageListFragment.State
- Fix bug 4341563
Change-Id: Ia2230bd5ec501fbc5c92b07b2ba874153b577a39
/packages/apps/Email/src/com/android/email/activity/MailboxListFragment.java
|
| e87ff6c3cbbfc5e3636f9827b58820652e3ea1c5 |
|
27-Apr-2011 |
Todd Kennedy <toddke@google.com> |
Opportunistic cleanup
not making any real code changes:
* removed deprecated, unused methods
* remove 'throws' clauses when that exception is never thrown
* renamed method Controller#moveMessage()-->moveMessages()
Change-Id: Ifd006f760f0c19283e94a11a45c71295c8da35f7
/packages/apps/Email/src/com/android/email/activity/MailboxListFragment.java
|
| 3de5fd026dc702d2cafc55b0a61039d4f582d932 |
|
26-Apr-2011 |
Makoto Onuki <omakoto@google.com> |
Fix phone UI.
Bug 4339904
Change-Id: I343e4a9a77ee7baeff7c9dc1222dad2701efb0df
/packages/apps/Email/src/com/android/email/activity/MailboxListFragment.java
|
| e62770d979d6fcd553e8447d45ec2024cdcdcde9 |
|
21-Apr-2011 |
Makoto Onuki <omakoto@google.com> |
Create new mailbox fragment for nested folder navigation
Create a new instance of the mailbox list fragment when navigating through
nested folders. (drill-in / going back to the root mailboxes.)
Also the fragment manager now has two public methdos for navigation that
are called by the activity. They will be the common interface for
the tablet UI manager (i.e. MessageListXLFragmentManager) and the
phone UI manager.
- openAccount(accountId)
Open the default view for the account. Used when switching accounts.
- open(long accountId, long mailboxId, long messageId)
Opne a particular view. Used when, for example, opening a message from the
widget.
Known issue:
- This breaks drag&drop through nested folder navigation.
This is because a new mailbox list fragment created during D&D doesn't
get the "drag started" event.
Change-Id: I69c14b71b4f681f8ab57f3ddd2cff9744a832076
/packages/apps/Email/src/com/android/email/activity/MailboxListFragment.java
|
| 38a33bd28fae9eb3226c90d3225b812ca23856f0 |
|
19-Apr-2011 |
Makoto Onuki <omakoto@google.com> |
Merged activity step1: Use FragmentTransaction (tablet)
Change-Id: Ia67e7f3e0658cb962d328c10affe11ccb5a165ea
/packages/apps/Email/src/com/android/email/activity/MailboxListFragment.java
|
| cfbf1eb66d39e5560e42b36537e1970abc16296d |
|
15-Apr-2011 |
Todd Kennedy <toddke@google.com> |
Clean up drag-and-drop
Basic refactoring of the drag-and-drop logic to make it a bit cleaner
Change-Id: I0e59074710be00bd285f637e92f554c8ee47fabf
/packages/apps/Email/src/com/android/email/activity/MailboxListFragment.java
|
| d1b1404d765fa36b281de3a2df1589bf11c28c04 |
|
15-Apr-2011 |
Todd Kennedy <toddke@google.com> |
Merge "Implement drag-n-drop with nested folders"
|
| 6c3cd4e2477eb1c7d1e107b3ea352639f1102933 |
|
15-Apr-2011 |
Todd Kennedy <toddke@google.com> |
Implement drag-n-drop with nested folders
We can now drill down into nested folders during drag-n-drop. Simply hover over
a collapsed folder for 750ms and it will auto-expand.
Change-Id: Ia9b42dd87e19c2ebdf8675c761cb1453bb26d158
/packages/apps/Email/src/com/android/email/activity/MailboxListFragment.java
|
| 8a05da7c8c5d5c499c1307cdfe8dee821960ef81 |
|
15-Apr-2011 |
Ben Komalo <benkomalo@google.com> |
Remove uneeded layer of parcelling.
ListStateSaver did nothing more than hold a Parcelable, and was a
Parcelable itself. The abstraction may have been useful at some point,
but that time seems to have passed.
Change-Id: I2f1f45828782ab44aa8935aa50b825e07845a2a5
/packages/apps/Email/src/com/android/email/activity/MailboxListFragment.java
|
| f7036b737907f06df1a06507754b28596cf8225e |
|
12-Apr-2011 |
Todd Kennedy <toddke@google.com> |
only allow navigating to mailboxes w/ children
If a mailbox doesn't have any sub-mailboxes, only allow selecting them to view
any contained messages. Do not update the mailbox list.
Change-Id: I6f469bf20a57dc440885402084c21ff184f13dff
/packages/apps/Email/src/com/android/email/activity/MailboxListFragment.java
|
| 5965b23909d498920c9830c3eb8341d0e0e5edce |
|
07-Apr-2011 |
Todd Kennedy <toddke@google.com> |
Always use the same loader id
There's no need to use a different loader ID for each mailbox. Just use
one loader ID to keep things simple.
Change-Id: I46cf0a9ad48c8e0badf72ee73e2d5edadda66912
/packages/apps/Email/src/com/android/email/activity/MailboxListFragment.java
|
| 007d0be4eddad21776ba45db5186bb8a9157504c |
|
07-Apr-2011 |
Todd Kennedy <toddke@google.com> |
First pass at nested folder support
currently only really works for exchange accounts. legacy accounts (imap
or pop) do not build a folder heirarchy.
Change-Id: Idb7c622b64a599a0776233a076359869f4ab4a4f
/packages/apps/Email/src/com/android/email/activity/MailboxListFragment.java
|
| ad7dc464db6421d3cda62204124dedaefb0f035d |
|
01-Apr-2011 |
Todd Kennedy <toddke@google.com> |
update mailbox adapter callback
the callback method name was not appropriate in all situations. This has been
renamed to just "onBind()" and will still be invoked whenever data is bound to
the adapter. also, create an empty callback that can be used if the adapter
doesn't need to be informed when a bind occurs.
Change-Id: Ifaa4aea3a59734557651270330ca9439e3ea4ab4
/packages/apps/Email/src/com/android/email/activity/MailboxListFragment.java
|
| 74bf57cfafde4095ee67985255cfaa8b1453115e |
|
31-Mar-2011 |
Todd Kennedy <toddke@google.com> |
Create separate adapters for mailbox lists
There are currently two places where we use a mailbox list -- in the main
view and in the "move to" dialog. We've replaced the single, modally
configured adapter with two specialized subclasses.
Change-Id: I5dfd5ccd435fefbab4b7647c2bd7a14818f73bf3
/packages/apps/Email/src/com/android/email/activity/MailboxListFragment.java
|
| dd123f927a69fd81ea0f4a08981e2f2bbc3ecd7a |
|
23-Feb-2011 |
Makoto Onuki <omakoto@google.com> |
Reset fragment state when opening a new account/mailbox
Without this, there's a small window where a fragment is in a valid
state after openXxx() before the list is loaded.
Bug 3420361
Change-Id: I7f84a94dec1419762aa6b24188c023abe974d2bf
/packages/apps/Email/src/com/android/email/activity/MailboxListFragment.java
|
| 31d9acbf0623872f9d4a2b3210b5970854b654c7 |
|
12-Feb-2011 |
Marc Blank <mblank@google.com> |
Email split, part huit: Refactor constants, clean emailcommon
* There are three pieces to this CL (sorry):
1) Move and/or rename some constants into emailcommon
2) Move Utility to emailcommon, moving the few UI
related utilities back into Email (FolderProperties
and UiUtilities)
3) Remove all references to resources from emailcommon
* The three pieces relate in that, between them, they allow
the emailcommon static library to compile cleanly
Bug: 3442973
Change-Id: Ic5e3abaa2a1b36999e0b6653c6c2134ea1bd544f
/packages/apps/Email/src/com/android/email/activity/MailboxListFragment.java
|
| a7bc0319a75184ad706bb35c049af107ac3688e6 |
|
10-Feb-2011 |
Marc Blank <mblank@google.com> |
Email split, part six: EmailContent
* Moved EmailContent to emailcommon
Change-Id: Ib3db1bfcfe74554c0e5afd3cfce6d72f26e9aeb9
/packages/apps/Email/src/com/android/email/activity/MailboxListFragment.java
|
| c540b57263250b961a29b26f367b10295ab2f4aa |
|
27-Jan-2011 |
Andy Stadler <stadler@google.com> |
Remove STOPSHIPs and a bit of debug logging
Bug: 3396875
Change-Id: I3e874e28c3412aadd9851000a83d69fc8c06aeb9
/packages/apps/Email/src/com/android/email/activity/MailboxListFragment.java
|
| 6bbc689efcb33c74129493e7097aa4a485a7d7c1 |
|
19-Jan-2011 |
Todd Kennedy <toddke@google.com> |
Implement final drag & drop spec
Current specification calls for the following:
* Droppable targets: do nothing
* Not droppable targets: grey text (hex #999999)
* Hover over droppable: use label/folder list pressed state
* Hover over not droppable: do nothing
* Destructive targets: background (hex #f10000)
We need to copy the resource from the framework as there is no supported way to
fetch the pressed state drawable during runtime. Adam filed bug 3370043 so we
can specify a drag target state directly in the selector.
bug 3154986
Change-Id: Ifd5c24a3dc46b5a1c64a149904657dda297ed047
/packages/apps/Email/src/com/android/email/activity/MailboxListFragment.java
|
| 31e25767918193217c884db72aaa98e50a4957d3 |
|
17-Jan-2011 |
Dianne Hackborn <hackbod@google.com> |
Follow API changes.
Change-Id: I1ca0fe365b953fa98a462912309a1c9647277ba7
/packages/apps/Email/src/com/android/email/activity/MailboxListFragment.java
|
| 8a63a72104ff86cbfa1d99767bf4be1c6dae899f |
|
11-Jan-2011 |
Todd Kennedy <toddke@google.com> |
Simplify drag-n-drop scrolling logic
Remove the concept of scroll speeds. According to Mark, multiple scroll speeds
is a "nice to have" feature and not a "must have". Besides, this did not work
(there was only ever one scroll speed), and, when fixed, the top scroll speed
was way too fast to be usable.
bug 3335062
Change-Id: I160bec2ca5de0b26da81ed31dfb96fc86e543988
/packages/apps/Email/src/com/android/email/activity/MailboxListFragment.java
|
| 07f5f60b8f84da92305d4f02ffedae9436f87787 |
|
09-Jan-2011 |
Todd Kennedy <toddke@google.com> |
Implement proper drag-n-drop
Still some tweaks to be done, but, we're real close to what it should
look like.
Bug 3322756
Change-Id: I8549ab59677a344e8f259c88bed950169d7381a8
/packages/apps/Email/src/com/android/email/activity/MailboxListFragment.java
|
| 0698f4027f099a017ad7815f16572ba422d3413d |
|
20-Dec-2010 |
Dianne Hackborn <hackbod@google.com> |
Update to new Loader APIs.
Change-Id: Ic228f7b785f0d902a0188cfe995006b73b8a3c8b
/packages/apps/Email/src/com/android/email/activity/MailboxListFragment.java
|
| 28d6e09575cc5b9091c5d29990792db3465cccf0 |
|
17-Dec-2010 |
Makoto Onuki <omakoto@google.com> |
Follow-up to the loader API change
This should fix the "attempt to re-open an already-closed object" exception
from SQLite.
(destroy()s don't have @Override becuase the base method is now depricated
and will be removed someday.)
Bug 3288666
Change-Id: I4780f6c8d89c7204b266608462c0833ad5af4e5f
/packages/apps/Email/src/com/android/email/activity/MailboxListFragment.java
|
| 80769cefb3f34dfeccb38fb5b67b3ced172b830e |
|
16-Dec-2010 |
Dianne Hackborn <hackbod@google.com> |
Prepare for some changes to LoaderManager/Loader.
Sorry it is a bit ugly, it is to allow this change to be
checked in prior to the first stage of the framework change
without breaking the build.
Change-Id: I1828579019ac0325d19c070a4c62cd79549e7d51
/packages/apps/Email/src/com/android/email/activity/MailboxListFragment.java
|
| 9d6d5b30917dcb8b4d90677f337cf4a230163239 |
|
24-Nov-2010 |
Makoto Onuki <omakoto@google.com> |
pixel-almost-perfect: mailbox list
Bug 3137965
Change-Id: Ic5aa3327ddbd456024e1938b74c7da0452b682f5
/packages/apps/Email/src/com/android/email/activity/MailboxListFragment.java
|
| f108837c1df2f20acbf5cd33d97a45f6747cff0e |
|
19-Nov-2010 |
Makoto Onuki <omakoto@google.com> |
Show current mailbox name and unread count on action bar.
- Show them when the mailbox list is hidden
- Data flow:
MailboxListFragment ->(Callback)-> MessageListXL ->
MessageListXLFragmentManager, which controls the action bar.
(I'm not too happy with this flow, but we can at least avoid
issuing yet another query to get these data.)
Also related:
- On the combined view, show the unread count for account rows
Bug 3137891
Change-Id: I37fa1258d946329905bbbc0da905928bcf2f4cb1
/packages/apps/Email/src/com/android/email/activity/MailboxListFragment.java
|
| c81bef672089654e6da3babbeb0172bd636564b2 |
|
14-Oct-2010 |
Marc Blank <mblank@google.com> |
Drag & Drop "move to folder"
* Drag & Drop "move to folder" preliminary implementation
* Use "drag handle" button to initiate drag
* Use the message list item itself as the drag thumbnail for
one item; "Move N messages" for multiple (not implemented)
* Disable MailboxList update during drag
* Use ListView itself as drag/drop target
Bug: 3138004
Change-Id: I9243e4f583cf91caa1b9dd04fd64d52e8b28fa30
/packages/apps/Email/src/com/android/email/activity/MailboxListFragment.java
|
| 730cc6724ae0fde63d6f11615338d2e01aef7e4b |
|
26-Oct-2010 |
Makoto Onuki <omakoto@google.com> |
Add "Combined view".
- Don't show combined mailboxes with regular mailboxes in the mailbox list.
- Add "Combined view" to the account selector instead.
- "Combined view" has all the combined mailboxes and accounts.
- Renambed these combined boxes. (e.g. "Combined inbox" -> "Inbox")
- Regular account view still has "Starred" mailbox, but it's actually
"combined" and not per-account.
- Re-order special mailboxes per latest wireframe.
Bug 3138004
Change-Id: I4c5860c6774b10c55ba0ca599373e51105432cf8
/packages/apps/Email/src/com/android/email/activity/MailboxListFragment.java
|
| 9b5001a34c82ea59b57a790fc221eec3af89e254 |
|
13-Oct-2010 |
Makoto Onuki <omakoto@google.com> |
Don't use negative IDs with ListView
ListView uses the _id column for some operations, including
onSave/RestoreInstanceState, and if the column contains negative values
they don't work as expected. The same assumption seems to be in other places
as well, so let's just avoid using negative IDs.
With this CL we now use two different IDs, one for ListView, which will
never be negative, and the other for us, the actual mailbox ID.
Bug 3049315
Change-Id: I263b4895212b5f8bb80c98acaf5c4eccd0bfef55
/packages/apps/Email/src/com/android/email/activity/MailboxListFragment.java
|
| 6993130a7cb089da642ee689603a2f8487b3fd39 |
|
09-Oct-2010 |
Makoto Onuki <omakoto@google.com> |
Auto-refresh mailbox list
Refresh mailbox list when changing the account, if it's been more than
5 minutes since the last refresh.
Change-Id: I5b1400bb881197e117b8863f850c368c2d1ccbc6
/packages/apps/Email/src/com/android/email/activity/MailboxListFragment.java
|
| 56034cab2022fe78ccbf4635617b1a54b4cc16b8 |
|
01-Oct-2010 |
Makoto Onuki <omakoto@google.com> |
Fix "move to" dialog style
- Pass the correct context to the adapter to make the text visible.
- MailboxesAdapter itself now has two modes (normal mode for MailboxList,
and "move to" mode), and use different views for them.
Change-Id: I2bd8a7d3a99d7fa3a047e653a77ec958e37e563b
/packages/apps/Email/src/com/android/email/activity/MailboxListFragment.java
|
| 62c9df806e6c165c9ffdc6b5c5372670e24e00df |
|
29-Sep-2010 |
Makoto Onuki <omakoto@google.com> |
Two bug fixes related to preserving list scroll position
* When changing account/mailbox, reset scroll position.
Applied same change as I4b368f51 to MailboxListFragment.
Also fixed a bug introduced in the same CL.
When CursorLoader detects content changed events and reloads itself,
only onLoadFinished() gets called, and in this case we should keep the scroll
position. This means mMailboxChanging should be cleared once data is loaded
for the next content changed reload.
Bug 3015916
* Don't try to make the selection visible when reloading the list for content
changed events.
We should make the selection visible only when account/mailbox is explicitly
selected. We can use the same mXxxChanging flags for this.
Bug 3036317
Change-Id: I93e3653da6816f491c1e99c9ef6d6b947be4de37
/packages/apps/Email/src/com/android/email/activity/MailboxListFragment.java
|
| 082443978595bee3b7907563dc7665f908872e20 |
|
24-Sep-2010 |
Makoto Onuki <omakoto@google.com> |
Workaround for ListView bug related to smoothScrollToPosition
Hopefully it'll soon be fixed in the framework, but it's blocking what
I want to do now.
Bug 3028455
Change-Id: I43ed059b7766a3bfc29166464bd449d905b207e4
/packages/apps/Email/src/com/android/email/activity/MailboxListFragment.java
|
| 58843f0dc336983bde54307d7387dcfcfc461017 |
|
22-Sep-2010 |
Makoto Onuki <omakoto@google.com> |
Highlight selected message on message list
Also make sure the highlighted mailbox is always visible on the mailbox list.
MessageListFragment now just uses the default background color, which means
it no longer changes background color for unread messages or checked messages.
This should be re-implemented if necessary with the newer wireframe.
Change-Id: I9bfbe9f0711ea4ab56be27778b00298d3033123b
/packages/apps/Email/src/com/android/email/activity/MailboxListFragment.java
|
| b4dacf2acfb8a07216c285af87e34a37619af50a |
|
22-Sep-2010 |
Makoto Onuki <omakoto@google.com> |
Don't show only special mailboxes on mailbox list
When a new account has just been set up and the app is still loading
mailboxes, keep the mailbox list empty and let ListFragment show
the progress icon, rather than showing "Combined Inbox" "All Starred" etc
without acutal inboxes, which looks like it's broken.
Change-Id: Ia3de4377b701f0033c5dc92f8b6b14651d4d8e1e
/packages/apps/Email/src/com/android/email/activity/MailboxListFragment.java
|
| 4580c3136d6dc82e3816cca104046a4a7af2aaf7 |
|
22-Sep-2010 |
Makoto Onuki <omakoto@google.com> |
Rename misnomered constant
Change-Id: I8d1024a85ad02ba7ce7ecf22f719cab34bf70307
/packages/apps/Email/src/com/android/email/activity/MailboxListFragment.java
|
| 93db1852bee2b1a23f051308e3634f58d33cb2c0 |
|
03-Sep-2010 |
Makoto Onuki <omakoto@google.com> |
MailboxList: Show the rotating icon when there's no mailbox
If the cursor is empty, hide the list and let the framework show
the rotating icon. (Don't set an empty cursor, which makes the list just
blank.)
We do this for only MailboxList because we know we'll get mailboxes
soon enough. We can't do this on MessageList because there may be really
no messages even if we wait.
Bug 2927973
Change-Id: Ic0346d2d90d03489072eb95a123281f60cda44dd
/packages/apps/Email/src/com/android/email/activity/MailboxListFragment.java
|
| 62f9c4d2803382f89cf8a19ed12b53b639d547fe |
|
25-Aug-2010 |
Marc Blank <mblank@google.com> |
Temporary UI for "move to folder"
* Many TODOs left in code.
* Only supports moving 1 message. (from message view)
Change-Id: Ibdec3163382345a7096c2cba51f448d69a9720af
/packages/apps/Email/src/com/android/email/activity/MailboxListFragment.java
|
| cadd46e3fba926e59eb4c8f2d2e3e84283536a09 |
|
31-Aug-2010 |
Makoto Onuki <omakoto@google.com> |
Preserve scroll position of message/mailbox list
* Preserve the scroll position in the following transitions
MessageList -> MessageView -> [back] -> MessageList
MessageList -> MessageView -> [screen rotation] -> [back] -> MessageList
two-pane -> compose -> [discard, etc] -> two-pane
two-pane -> switch to other app, and switch back -> two-pane
* Don't always refresh on onResume. Do it only when requested.
This supresses unnecessary refresh when coming back from other full-screen
activities, e.g. MessageCompose.
* Also updated the comments on the back handling.
Bug 2769052
Change-Id: Id83dc3e778c35860d634c68ecac0c7a07cc4057e
/packages/apps/Email/src/com/android/email/activity/MailboxListFragment.java
|
| c8502dd1946545cfa112c55aee1444c4544700af |
|
25-Aug-2010 |
Makoto Onuki <omakoto@google.com> |
Highlight selected mailbox on mailbox list
* Use ListView's CHOICE_MODE_SINGLE mode to select list item.
* Added a view that implements Checkable to change background
for selected item.
Change-Id: Id55999b3d024ad1852e2b8c6436cd22ef255cf95
/packages/apps/Email/src/com/android/email/activity/MailboxListFragment.java
|
| 21efedb67fdfff208cef3a18804771fd1d1fff30 |
|
16-Aug-2010 |
Makoto Onuki <omakoto@google.com> |
Rework/cleanup of "refresh".
Added RefreshManager, which is responsible for getting refresh requests
from UI and keeping track of what is being refreshed.
Conceptually it's a part of Controller, but extracted for easier testing.
- Now sendPendingMessagesForAllAccounts() is owned by RefreshManager
rather than Controller.
- Also updateMailboxRefreshTime/mailboxRequiresRefresh have been moved
in from the Email class.
- Now MessagingException implements a method to return an error message
for the UI.
The refresh button on 2-pane doesn't work as intended yet, because the
spec is a bit too complicated (as described in the TODO in
MessageListXLFragmentManager.onRefhres()).
This change touches many file mostly because it cleans up a lot
of code duplication.
Change-Id: I058ab745ccff10f6e574f6ec4569c84ac4a3e10e
/packages/apps/Email/src/com/android/email/activity/MailboxListFragment.java
|
| f513fbd8cb47a89d39ccf33874f1dea1676e4f98 |
|
18-Aug-2010 |
Makoto Onuki <omakoto@google.com> |
Implement UI event handlers.
Implemented
- The bottom buttons for MessageViwe
Delete, Mark unread, Reply, Reply all, Forward.
- Buttons for exchange invitation
View in Calender, "Yes", "No", "Maybe"
- Other MessageView events
onUrlInMessageClicked()
(Most other methods will probably be deprecated)
- Removed obsolete MailboxListFragment.Callback.onRefresh()
Fixes bug 2926517 minus "the background color of respond buttons
being black" part.
Change-Id: Ie58cbc9fde95a3e67d96846450f77ab58b175a55
/packages/apps/Email/src/com/android/email/activity/MailboxListFragment.java
|
| 79d4a2d9e1537727eb1b66c0df91b2b43ab1a9a5 |
|
11-Aug-2010 |
Makoto Onuki <omakoto@google.com> |
Make use of the retaining-loader feature.
Don't use restartLoader(). Use initLoader() with stopLoader() when necessary.
This allows us to make use of LoaderManager's "retaining" feature.
i.e. We won't have to requery when the screen orientation changes.
Also, don't start loading in onStart(). Fragment manager seems to kick
exisiting loaders after onStart(), so you'll end up getting onLoadFinished()
twice.
(Looks like I got unluck with this--there was no strong reason to start loading
in onStart rather than onResume, but I think we can leave other fragments as-is
as long as they don't use loaders.)
Seems to be working.
Change-Id: Ib4f72098bd0fcbfce284ae6e16055d20ee410cf3
/packages/apps/Email/src/com/android/email/activity/MailboxListFragment.java
|
| 94506fdaf651363adaf60650d9a2301fdb76724d |
|
04-Aug-2010 |
Makoto Onuki <omakoto@google.com> |
Small fixes/clean-ups.
- Fixed account selector default account look-up
- Renamed MessageListFragment.onRestoreInstanceState
to loadState.
(Activity.onRestoreInstanceState is called after onStart,
but this one is called in onCreate, so it wasn't a good
idea to use the same name.)
- Changed mailbox cursur requery timeout to 3 seconds
- Added some TODOs.
Change-Id: Ia2242cd9f74936d24756b800eacd126958eed330
/packages/apps/Email/src/com/android/email/activity/MailboxListFragment.java
|
| 833fe73b99e62ad9cf6e80c782717c7de1ff12e4 |
|
03-Aug-2010 |
Makoto Onuki <omakoto@google.com> |
Aadd special boxes to mailbox list.
Change-Id: I36616f53555346ca7fbb6f3426a0c3196d7bacbc
/packages/apps/Email/src/com/android/email/activity/MailboxListFragment.java
|
| 7ef278202e26f82612f94f18a4e971994f29c0ec |
|
03-Aug-2010 |
Makoto Onuki <omakoto@google.com> |
Clean up MailboxListFragment.
- Remove AsyncTasks, and use CursorLoader to load data.
- Get message counts for the drafts/trash mailboxes directly from
the db column.
- Remove obsolete code from MailboxesAdapter.
Change-Id: I93c72977c19b60581e1169ba9bd429912ba3e68f
/packages/apps/Email/src/com/android/email/activity/MailboxListFragment.java
|
| ee216da74aad5a378985c6a59f7b60391a8a63d9 |
|
28-Jul-2010 |
Makoto Onuki <omakoto@google.com> |
Adding account selector.
Also
- Renamed MessageListXLFragmentManager.setStart to onStart. (was typo)
- Fixed flag handling bug in MailboxListFragment
- Fixed the R key handling code. (The first R key was always ignored,
because the default requested orientation was neither landscape or
portrait.)
Change-Id: I0e14ce9f4fc5be973f7c0091f88fd4551a4329fa
/packages/apps/Email/src/com/android/email/activity/MailboxListFragment.java
|
| 02fa70aa185d8073966105e9986bd8e12200a526 |
|
23-Jul-2010 |
Makoto Onuki <omakoto@google.com> |
Refactoring MessageListXL
I always thought our Activities are way too fat, meaning we've put too many
things into activities without any structure.
The major problems with this are:
- They have too many fields, which are not final and not even orthogonal.
This makes them very hard to understand/maintain. Changing one tiny bit
can always cause unanticipated side-effects.
- Very hard, or almost impossible to test.
I really think we should break them into independent and self-contained
subcomponents which can be tested separately.
Introducing MessageListXLStateManager, which manages the current account,
mailbox and message, and show/hide/update fragments accordingly
for MessageListXL.
With this class, MessageListXL will be able to switch accounts/mailboxes/
messages by just calling the methods such as selectAccount(), without
worrying about when to show/hide what fragment and how to initialize them.
(In other words, MessageListXLStateManager encapsulates the two-pane screen
transition. It's not intended to be reused for the phone UI.)
I didn't make it a nested class in MessageListXL, because nested classes can't
have real private members (private member are accessible from outer classes and
even brother classes!!), and I wanted it to be really self-contained anyway.
Change-Id: I1c121e99e30f12cc118e1c35abc9b30f49939a4a
/packages/apps/Email/src/com/android/email/activity/MailboxListFragment.java
|
| 833eae6ac88cbd1e19404386c658d43b26cc3409 |
|
23-Jul-2010 |
Makoto Onuki <omakoto@google.com> |
Adding lifecycle log to fragments.
Change-Id: If83486b50c924fdb99a422814ede66da710bfc2d
/packages/apps/Email/src/com/android/email/activity/MailboxListFragment.java
|
| 9ddd1e7fa4b700326f00e406219b1ac584dee5a0 |
|
23-Jul-2010 |
Makoto Onuki <omakoto@google.com> |
Small changes to list fragments.
- Added parameter check to openXxx methods.
- Clear the list before (re-)load the data, to make ListFragment show the
"Loading..." animation.
- Moved a few lines around in MailboxListFragment.
Change-Id: I48d9759e118976bf9f8cce10069e2b9b5c253342
/packages/apps/Email/src/com/android/email/activity/MailboxListFragment.java
|
| 8167070158afcdb43081b3ff3d9b0379aeb2225c |
|
22-Jul-2010 |
Makoto Onuki <omakoto@google.com> |
Refactoring MailboxListFragment
- Broke down bindActivityInfo into setCallback and openMailboxes()
- openMailboxes() can be called multiple times against the same instance.
(We won't have to re-create the left pane on XL screen)
- Added EmptyCallback to avoid null checks
- Renamed a callback method
Change-Id: I3848a6f6da7f1782dcc9566967f8618ed79a878e
/packages/apps/Email/src/com/android/email/activity/MailboxListFragment.java
|
| d6a2978857e0866d7441f8e140338477545d59a5 |
|
21-Jul-2010 |
Makoto Onuki <omakoto@google.com> |
First cut of two pane activity.
How to launch:
- Kick "Email 2 Pane" in the app drawer.
Major known issues:
- When you launches "Email 2 Pane", it starts directly; the logic in Welcom
won't be executed.
- There's no UI to add accounts. Use the old UI.
- There's no way to select non-default accounts.
- The on-screen buttons except for older/newer don't work.
- No contex menus work.
Change-Id: I38374acafafbae62e46f84294a7677e54686cfc2
/packages/apps/Email/src/com/android/email/activity/MailboxListFragment.java
|
| 37e21ab968e3944b10304a620c3b0ecfe5175668 |
|
20-Jul-2010 |
Makoto Onuki <omakoto@google.com> |
Don't call setListAdapter until adapter is ready.
ListFragment shows the "Loading..." animation for us until setListAdapter is
first called. In order to make use of it, setListAdapter should be called
only when the underlying data is ready.
Change-Id: Iac903b1f10ad7ed4be04446ddb2d2172e84bfe16
/packages/apps/Email/src/com/android/email/activity/MailboxListFragment.java
|
| 7daacf83a4e789aaab11ef715f42470bcee51bfc |
|
08-Jul-2010 |
Makoto Onuki <omakoto@google.com> |
Switch to ListFragment, and use its default layout.
Change-Id: If8dead170a3662cbde87e6ead244363b11db01f6
/packages/apps/Email/src/com/android/email/activity/MailboxListFragment.java
|
| 7e071d6956076e80a1044989ba2df1235fc7e9f1 |
|
09-Jul-2010 |
Dianne Hackborn <hackbod@google.com> |
Fix the build -- follow API change.
Change-Id: I7a177a3bbd5b374cfdc2dd708cc8b1acd8c19a92
/packages/apps/Email/src/com/android/email/activity/MailboxListFragment.java
|
| 2aa8ea4399dd70c26d6c0679ff002fb16fdc5bff |
|
14-Jun-2010 |
Makoto Onuki <omakoto@google.com> |
Use Fragment's context menu API.
Change-Id: I1be6b17ff5bc7fd5258366bc064251c6f79c0cc8
/packages/apps/Email/src/com/android/email/activity/MailboxListFragment.java
|
| 75939a7dc8aabfb13bc376a6c9ae7ad4adc24881 |
|
12-Jun-2010 |
Dianne Hackborn <hackbod@google.com> |
Update to match new fragment context menu API.
Change-Id: I18950e55abd1e72d97f7a62393abe422001ee2f5
/packages/apps/Email/src/com/android/email/activity/MailboxListFragment.java
|
| 80091c8ef7cc1a9508951910574d7c41772bb0b2 |
|
08-Jun-2010 |
Andrew Stadler <stadler@android.com> |
Refactor MailboxList to use Fragment API
* Moves all list-related implementation to new MailboxListFragment
* One item that remains to be done is to remove the dependency on the
activity for handling context menu (longpress) in the list.
Change-Id: I7b5769d9d81fb685cf480de3d3e18b4e1c078f2d
/packages/apps/Email/src/com/android/email/activity/MailboxListFragment.java
|