History log of /packages/apps/UnifiedEmail/src/com/android/mail/browse/MessageAttachmentTile.java
Revision Date Author Comments (<<< Hide modified files) (Show modified files >>>)
4f347e811052f446c3958c76db278bcd7b39a44f 26-Feb-2014 Andy Huang <ath@google.com> allow an extra option on attachments (in bar and tile UI)

Bug: 7659807

Change-Id: I4c1294d2ba772a70139e0b37429b17bc6096cafd
/packages/apps/UnifiedEmail/src/com/android/mail/browse/MessageAttachmentTile.java
e003b4cc4a23e6d394998bebe26430a361c5ac29 16-Aug-2013 Andy Huang <ath@google.com> metrics for attachments

Improve folder-switch logging with detailed data for system folders.
Also prevent exits from a blank compose from triggering a log event.

Bug: 9861033
Change-Id: Ie75323fe3704d3010acd864c6d7c646ebd7708b5
/packages/apps/UnifiedEmail/src/com/android/mail/browse/MessageAttachmentTile.java
aa27bc0e1c3bb6be4609b00007637a9d3e960f5e 02-Aug-2013 Scott Kennedy <skennedy@google.com> Fix more warnings

Change-Id: I78c6df845aae3475c1e2469b29b52cc995d8d3ae
/packages/apps/UnifiedEmail/src/com/android/mail/browse/MessageAttachmentTile.java
479505d71969e26b0785d8e0e1b81108731cf827 21-Mar-2013 Mark Wei <markwei@google.com> Attachments in Conversation List: Backend changes so that the appropriate
attachments are passed to the UI. Implement PhotoManager so we can start showing
images.

Upon sync, we extract two image attachments from the first unread message, and
add them to the conversations table.

UI changes as well.

Load BEST if that is available, else load SIMPLE.
No vertical dividers in attachment previews.
Attachment previews are above labels.
Round badge appears for unread conversations when there are more images than
shown. Only appears after the previews have loaded.
Clicking on the attachment previews launches photoviewer.
Attempt to load bigger images than necessary to prevent reloading during mark
read/unread and rotate.
Show animated loading icon while previews are loading.
Previews load from top to bottom, from left to right.

Big performance improvements:
No longer decoding bitmap on UI thread. This kills most of the jank.
Only load when we are not scrolling. This kills a lot of jank.
Remaining jank will be delt with later:
DividedImageCanvas.setDimensions() creates a huge empty bitmap on the UI
thread, which causes a GC. ath will fix.

Multiple images in one line now load whatever's ready first. Added tracing.
Bitmap cache given more memory.
Following ath's advice, decode Bitmap from InputStream rather than byte[]. Don't
compress into BitmapHolder. Only use the bigger Bitmap cache.

Bug: 7481049
Change-Id: Icf823f56abc9345e6d0b800a1e6d45444d8fa028
/packages/apps/UnifiedEmail/src/com/android/mail/browse/MessageAttachmentTile.java
3b965d78774a42358ce6bbdcc43b4c8df130a60e 25-Jun-2013 Scott Kennedy <skennedy@google.com> Clean up a bunch of warnings

Bug: 9565838
Change-Id: Ie3a380581e81c63aaa0764875c55bfdccd5f2b13
/packages/apps/UnifiedEmail/src/com/android/mail/browse/MessageAttachmentTile.java
7434e800d4313a227120ca36bd95683752a7879f 21-Jun-2013 Andrew Sapperstein <asapperstein@google.com> Attachments in eml files.

It's kinda slow for large eml files
(because we have to parse the entire
file) but it works. Hooray.

Fixes b/6393073.

Change-Id: I2d7a87a484cd282a000a0905fe069f3ab45e2061
/packages/apps/UnifiedEmail/src/com/android/mail/browse/MessageAttachmentTile.java
69c4ddb7a49f11377143c9ff1c7b85b2450449e9 25-Feb-2013 Mark Wei <markwei@google.com> Do not request thumbnail download if there is no network connection.

Remove duplicate get identifier for attachments.

Bug: 8218823
Change-Id: Ia4ca9e6b6798b9132242ff0d9d046cd3e75e4e9b
/packages/apps/UnifiedEmail/src/com/android/mail/browse/MessageAttachmentTile.java
8750066b10f80e2a8080016973b3296d76d18266 20-Feb-2013 Mark Wei <markwei@google.com> Set MIME type correctly on drafts when content provider does not provide it.

Bug: 8235172
Change-Id: I7bbccbecb7d038f3add4503be0dcf460a4864b2e
/packages/apps/UnifiedEmail/src/com/android/mail/browse/MessageAttachmentTile.java
1867a56f7b9b3146829752b942285ffdc9b3f3de 20-Feb-2013 Mark Wei <markwei@google.com> When normalizing MIME type, also infer it from filename in case it is generic.

Bug: 8223790
Change-Id: Ie093eea2fb1710bfe66472e92e4f5db2b4247f53
/packages/apps/UnifiedEmail/src/com/android/mail/browse/MessageAttachmentTile.java
267453587f13986acc7de3d52de95184c38fb4e1 18-Jan-2013 Mark Wei <markwei@google.com> When clicking on a MessageAttachmentTile, the images download with varying
priority.

The clicked image should have highest priority, and images further to the left
and right should have consecutively less priority.

Bug: 8023818
Change-Id: I5531971f00db68e4b99bb210c757db53fc2a936c
/packages/apps/UnifiedEmail/src/com/android/mail/browse/MessageAttachmentTile.java
0eb46cb4c18b91ee272caca7f62f4a15ec2668ef 17-Jan-2013 Mark Wei <markwei@google.com> Modify image attachments previews to conform to UX specs

Thumbnail and full image start loading at the same time.

Once thumbnail is shown, we display a determinate progress bar while downloading
the full image.

Remove progress bar from image tiles in ConversationView.

Bug: 8021747
Change-Id: I600a8bb8337dd7ee6e7cec2a1221824b04a9e2ef
/packages/apps/UnifiedEmail/src/com/android/mail/browse/MessageAttachmentTile.java
1aee17e324f204080baaabceb64f4edf73681542 14-Jan-2013 Mark Wei <markwei@google.com> Load image attachments from live conversations.

MessageAttachmentTile automatically downloads image attachment thumbnails when
missing.

PhotoViewFragment first loads thumbnail, then loads the full image. Shows and
hides the progressbar correctly.

Added ability for UiProvider to specify which rendition of an attachment to
download.

Bug: 7987412
Change-Id: Ief7f4c9a4a354d27045f6415a32211484a8527c6
/packages/apps/UnifiedEmail/src/com/android/mail/browse/MessageAttachmentTile.java
90528f425e09a72b1505c3a1066051ab5a3a67b4 13-Oct-2012 Paul Westbrook <pwestbro@google.com> Show progress on attachment tiles

Bug: 7298245
Change-Id: I8f7b4e449b652d27e22b2f553292f4f4b5de27a4
/packages/apps/UnifiedEmail/src/com/android/mail/browse/MessageAttachmentTile.java
a7e154530add87e05d1fcee980e1a1fc34a456e7 18-Sep-2012 mindyp <mindyp@google.com> Lots of fixes to move to folder dialog.

Fixes b/6919987 bring back divided "move to folders" sections

Change-Id: Ia7d709c5cd1f8787e2e1083442aa4eaf50d6babb
/packages/apps/UnifiedEmail/src/com/android/mail/browse/MessageAttachmentTile.java
8081df46ef5a7794374e41cd1836e778a2da9b31 11-Sep-2012 Paul Westbrook <pwestbro@google.com> Restore download progress dialog

Now the attachment download progress is using an DialogFragment
so the dialog correctly gets restored on orientation change

Bug: 7137689
Change-Id: I90a225f423bf75fd4dbeb464051374919a3cff62
/packages/apps/UnifiedEmail/src/com/android/mail/browse/MessageAttachmentTile.java
62066e47fe69c33b60ed970e46809ba81d044064 13-Sep-2012 Mark Wei <markwei@google.com> Prevent photos attached during Compose from showing generic thumbnail briefly.

Cache the loaded previews on save instance state, so when the configuration
changes, we don't have to re-download all the previews.

Bug: 7140604
Change-Id: Ic3fd640cb0116729868548cdd620b67bc06f3967
/packages/apps/UnifiedEmail/src/com/android/mail/browse/MessageAttachmentTile.java
7018b9b61d8d95a382e9a3759deb08e8240355c7 07-Aug-2012 Andrew Sapperstein <asapperstein@google.com> Deleted setPhotoName call. API is being deleted.

Change-Id: I9a56dfe0c1ad183da48beca25f2d9b3a05517d0d
/packages/apps/UnifiedEmail/src/com/android/mail/browse/MessageAttachmentTile.java
4cb51dbce9635f4bf68a6de88f759e2f86d20325 21-Jul-2012 Andrew Sapperstein <asapperstein@google.com> Image attachments open photo viewer immediately.

Instead of downloading the first image and waiting for
that operation to complete before entering the photo viewer,
we now kick off the downloads for all photos that are not
currently on device. The photo viewer shows download progress
and also shows the photos once the download has completed.

Change-Id: Ib9561b45020bbab1ffbcec9e88aab9604036fd5a
/packages/apps/UnifiedEmail/src/com/android/mail/browse/MessageAttachmentTile.java
afff6d79b4ea2713ef75bfdad8abd7e21ee20080 26-Jun-2012 Andrew Sapperstein <asapperstein@google.com> Updated the UI for attachments per revised mocks.

Now showing name/subtitle when we don't have a
preview or if we are in the compose view.

Still needs revised assets.

Change-Id: Ie522b5dd28b605c8e16793a65a75696c00169fae
/packages/apps/UnifiedEmail/src/com/android/mail/browse/MessageAttachmentTile.java
b334c9035e9b7a38766bb66c29da2208525d1e11 25-Jun-2012 Paul Westbrook <pwestbro@google.com> Changes to support different Log tags

Change-Id: I7121fbf7132444fc0f5c0c130373e3f2d1a3cf7a
/packages/apps/UnifiedEmail/src/com/android/mail/browse/MessageAttachmentTile.java
5c721d9e6df069b6385929817bb2b61faeba25a3 23-Jun-2012 Andrew Sapperstein <asapperstein@google.com> Removing the photo viewer from unified mail.

Change-Id: Iea42c019fa31a7ead997505da295eb4aae0e2e4e
/packages/apps/UnifiedEmail/src/com/android/mail/browse/MessageAttachmentTile.java
237129ddf57a1e25469d8990322fb7913e18ae20 15-Jun-2012 Andrew Sapperstein <asapperstein@google.com> Compose now shows images as tiled attachments.

It's still a bit buggy (notably, once the previews
do not show up after the initial attaching). Those bugs
will be fixed in the next commit but this was a good stopping point.

In order to reuse as much code as possible, the AttachmentTileGrid
was moved into com.android.mail.ui. Additionally, MessageAttachmentTile
was split into AttachmentTile and MessageAttachmentTile. AttachmentTile
is a base class that handles all of the tile interaction that is common
between the browse and compose versions. At present, this is mostly
the fetching and displaying of the preview image.

As a result, MessageAttachmentTile now derives from AttachmentTile.
MessageAttachmentTile handles the interaction specific to browsing
messages (such as opening the photo viewer, etc).
There is also new class called ComposeAttachmentTile
which derives from AttachmentTile and handles the interaction
specific to composition of messages.

The AttachmentsView in composition now contains two children in its
view hierarchy, an AttachmentTileGrid and an inner LinearLayout
that now stores the non-tiled attachments.

Change-Id: Iad85529a7b6d573060645499271860cf07877754
/packages/apps/UnifiedEmail/src/com/android/mail/browse/MessageAttachmentTile.java
532f48253d23ffcb974ab61856df131021dde02a 14-Jun-2012 Andrew Sapperstein <asapperstein@google.com> Enabled actions in the photo viewer.

All four actions (save, save all, share, share all)
are working properly. However, other apps do not
implement receiving the ACTION_SEND and ACTION_SEND_MULTIPLE
so the share and share all actions are a bit janky.

Change-Id: Ifcd3ed36c6bfce4a023b54a61f77e8ae4fd8f3b8
/packages/apps/UnifiedEmail/src/com/android/mail/browse/MessageAttachmentTile.java
4c7d8b5f38bfa80192d457025b834c975d7d38a3 12-Jun-2012 Andrew Sapperstein <asapperstein@google.com> UI improvements to photo viewer.

Photo viewer now shows the display name of the
image. Additionally, it now has a subtitle that
contains the current number of that attachment.

Additionally, the implementation in Unified utilizes
a derived class of PhotoViewActivity that tweaks
the subtitle to display image size and adds action
items. Currently the items are visible but do not do
anything. Next commit will wire these up.

As part of refactoring to make the derived classing work
properly, a bunch of legacy code was deleted including
all of BaseFragmentActivity as its sole reason remaining
was for spoofing the ActionBar.

Finally, a small change to the cancel attachment button
was made so that it was appropriately selectable and
48dip by 48dip.

Change-Id: I5a50bd420fa23a3e7d8734b5a46a6f28198ccef4
/packages/apps/UnifiedEmail/src/com/android/mail/browse/MessageAttachmentTile.java
4aa1c1340556c646175c3d0b193be05876ad95a0 09-Jun-2012 Andrew Sapperstein <asapperstein@google.com> Non-image attachments now display as a list.

There are now two separate attachment areas in
the MessageFooterView. The AttachmentTileGrid
contains all images in the gridded tile look.
Additionally, there is a LinearLayout that contains
0 or more MessageAttachmentBars. These bars contain
a small icon, info about each attachment, and
an overflow button that contains additional options
other than the default for that attachment type.

In addition, there is now AttachmentActionHandler
that performs a lot of the interaction that both
attachment views share.

Change-Id: I60d297b7514e6ee35d23a9ec250833e886328d4d
/packages/apps/UnifiedEmail/src/com/android/mail/browse/MessageAttachmentTile.java
e4798f5d8bd07eb39bf9dac57fd2a304b53916a0 06-Jun-2012 Andrew Sapperstein <asapperstein@google.com> Reworked attachment grid to use a dp-based layout.

Instead of having a resource selection for the
number of columns (which requires
a different value for portrait and landscape as well
as every screen size), utilize a minimum dip tile size
that figures out, while measuring, the appropriate number
of columns to use and expands the tile size to fit the
entire width with that number of columns.
As a result we have only two different values
for the minimum tile size. 120dip is the default and
180dip for sw600dp. Verified that the appropriate
number of columns occurs on all Google experience
devices.

Additionally, added a header and bar for the attachments
so that the user can clearly see where attachments
begin.

Change-Id: I48b1645e1004596747c1585e44a387b1231bd661
/packages/apps/UnifiedEmail/src/com/android/mail/browse/MessageAttachmentTile.java
b40bfd3a7a44a6b9d1ce1ed43d98ab228890d199 01-Jun-2012 Andrew Sapperstein <asapperstein@google.com> Refactoring a bunch of code.

Removed some unnecessary files. Drastically shrunk
others. Moved one. The moved file is currently unused
but is something I'd like to add at some point.

Change-Id: I22dc75e841d2e6685081fa1d428152887055fc93
/packages/apps/UnifiedEmail/src/com/android/mail/browse/MessageAttachmentTile.java
15b43f21f67e425bdfc2c401b3ac4367075c2dd1 31-May-2012 Andrew Sapperstein <asapperstein@google.com> Fixed a bug where thumbnails wouldn't show.

Also improved the scaling of thumbnail images and
added a black background. Need new icon for non-images.

Change-Id: I8f5a5df85dce3a3e331cd343171869fcb99f311f
/packages/apps/UnifiedEmail/src/com/android/mail/browse/MessageAttachmentTile.java
f986850eb04ff614feb915e178227c4f53ca695b 31-May-2012 Andrew Sapperstein <asapperstein@google.com> Attachments now show in a grid-like view.

Currently, two per row are displayed in portrait
and three per row in landscape. Need UX decision
on how many to show for larger screen sizes.

Much of this code is taken from Contacts which
used a similar layout for favorite contacts.
That layout was taken and tweaked to suit the
attachment handling purposes.

Still need to add the appropriate interactions
for the various attachment types per UX's guidelines.
Will add that in the next CL.

Change-Id: Iad6ea716a680b1fd48f16d34de2a5bc2e87b7db5
/packages/apps/UnifiedEmail/src/com/android/mail/browse/MessageAttachmentTile.java