| Age | Commit message (Collapse) | Author |
|---|
|
|
|
Per PM/UX feedback.
Bug: 333108781
Test: Manual verification with ShareTest.
Flag: NONE
Change-Id: I53dea71242c4b689819c65608174a836dca102f5
|
|
Per UX feedback.
Bug: 333109123
Test: Manual testing with sharetest
Flag: NONE
Change-Id: I5e9abf758ab055e8ed491a1d0ccb0a5656000c4b
|
|
Adjust paddings for better symmetry.
Bug: 286284385
Test: atest IntentResolverUnitTests
Test: Manual testing in portrait and landscape.
Change-Id: Ie94a2f48ff7ce4c56afbef9006bfb2e8b457185b
|
|
- Remove divider above actions.
- Some color fixes.
- Actions are always horizontal, no attr needed.
- Lots of margin and padding tweaks per mocks.
- Fix horizontal recyclerview padding in RTL
Test: Lots of manual inspection with a variety of share payloads.
Test: atest IntentResolverUnitTests
Bug: 283245199
Change-Id: Ica2757a90387afd6be793b8c44fa47fd136056c7
|
|
Test: Install and launch Share Test
Test: Choose Five Actions and tap Share
Test: Observe the action chips match design
Test: Rotate the device to landscape
Test: Observe the action chips match design
BUG: 283245199
Change-Id: If7969168b9d63d74484aabf6c5dee07745276e8c
|
|
Reduce the severity of truncation and wrapping issues in both landscape
and portrait.
Bug: 281679930
Test: Manual test with chrome and ShareTest fake actions.
Change-Id: Ibcd98eb12a0225f5f1faff4c096b0af6656575d0
|
|
- Align colors with mocks.
- Increase custom action vertical padding
- Align text + image and file + text to the same 56x56 image size
- Reduce target vertical padding
- Don't disable text when share is "image only"
- Add an icon for single-file shares
- Update multi-file icon to align with mocks.
- Vertically align image in text+image
- Show image and title with text in landscape
- Reduce text preview padding to align with file+text padding.
Bug: 281541996
Test: build and test with ShareTest, Files and Photos.
Change-Id: Ia58315d765a824cb45ee1b84625c1262a8a3e76d
|
|
- Show preview images but reduce height.
- Reduce top margin above icons in target list.
Bug: 267522604
Test: Manual test with a variety of inputs from ShareTest in portrait
and landscape.
Change-Id: I4a373b5c9cab555ffb4c28447f76e057d4114902
|
|
On landscape phones:
- Hide all image previews except in the image+text case where it's now
much smaller.
- Increase max width for custom actions.
- Hide text title
- Remove minimum height for text preview container.
Bug: 267522604
Test: Manual tests on tablets and phones in both rotations.
Test: atest IntentResolverUnitTests
Change-Id: I7ad8d0b94f7133b36ce0d60640a31f7ef6d8cd7d
|
|
Previously had to define several values twice. Not use values/ for the
landscape phone values and values-h480dp/ for values for any device
config taller than 480dp (basically not a landscape phone).
Bug: 267522604
Test: Verifying no visual behavior change in either rotation on tablets
and phones.
Test: atest IntentResolverUnitTests
Change-Id: I8afdb13b7d4a7949ddbaf0f6eabc37d2b3e38a46
|
|
Make stuff smaller and less tall on landscape non-tablets.
Let the system restart sharesheet on rotation to allow for resources to
change.
Tablet displays in landscape generally should follow the un-compressed
UI of portrait phones, so the resources for that configuration have to
be listed twice.
This doesn't dismantle any of the code for handling orientation changes
ourselves, as taking that apart may be error-prone.
This is meant to a first pass, additional landscape miniturization will
follow, as there still isn't a ton of room to scroll at the bottom.
Bug: 267522604
Test: Testing in portrait and landscape with a variety of ShareTest
payloads
Change-Id: I4697ad9510526cc9ce5f9e0630080bef5fad377c
|
|
This copies Chooser classes and (as needed) their transitive
dependencies, while making minor mechanical changes to fix up
references for the new package. This is a large CL, especially
since we don't track the history of these files across projects
(framework vs. "unbundled"), and the meaningful changes could
easily get lost in the noise; reviewers should see accompanying
notes at go/chooser-fork-cl.
Test: locally re-enabled and ran UnbundledChooserActivityTest
(as updated in this CL to match the framework version), then
re-disabled before uploading. See notes for more info.
Change-Id: I6708e3563d18a9e31894dd36dcd65e9a5815abcf
|