Reworked "add new" menu to better support multiple office suites - #17473
Conversation
19420db to
79fcfa1
Compare
|
APK file: https://github.com/nextcloud/android/actions/runs/31710340741/artifacts/9186734530 |
6161d7b to
5771390
Compare
|
APK file: https://github.com/nextcloud/android/actions/runs/31775512074/artifacts/9209879722 |
|
@kra-mo @daniele-verducci @tobiasKaminsky Regarding designThese are just details if it requires more effort I think we can skip it.
The back arrow currently spans the full width of the bottom sheet, like the other actions. Would it be possible to make it 48.dp size instead?
|
4ad1a41 to
4b07c69
Compare
|
Hey @alperozturk96 , thanks for the feedback! |
|
APK file: https://github.com/nextcloud/android/actions/runs/31788727775/artifacts/9214729118 |
|
@daniele-verducci You can fix : with
|
|
Yes, sorry, I still don't have enabled the pre-commit checks... fixed! |
|
APK file: https://github.com/nextcloud/android/actions/runs/31790537711/artifacts/9215432268 |
|
@daniele-verducci, Alper is right about the height and the contrast :)
For this, I'd also add that it should have rounded corners. |
5361711 to
52051a1
Compare
|
APK file: https://github.com/nextcloud/android/actions/runs/31815200041/artifacts/9224951094 |
|
Hey @alperozturk96 and @kra-mo , I've made some of the changes, but I'm finding challenging some others. Made: Margins and rounded edges on the ripple effect Challenging: The first one is technical: all the buttons are colored with colorMaterialButtonContent() (from MaterialViewThemeUtils), that expects a MaterialButton. Unfortunately i had to add the arrov as an ImageView over the MaterialButton, as it doesn't allow to specify drawableStart/drawableEnd, but only a single icon (yes, I feel it's ugly, but didn't come out with a better idea...). How can I theme that one? I'm probably missing something obvious here... The second one: I made the back button square instead than full width, but I'm not sure how to position it:
Any hint? |
3c135f6 to
f2325d4
Compare
|
Changes in behaviour Changed the order of the entried to comply with nextcloud/files-clients#126 Cosmetic changes Styled the "More" and "Back" button using the instance's theme color At this point I should have applied all the changes discussed above. The current back button position is the one that avoids the ripple touching the border (see: |
|
master-IT test failed: https://www.kaminsky.me/nc-dev/android-integrationTests/17473-IT-master-11-53 |
df7d7b9 to
372a2e9
Compare
Signed-off-by: daniele-verducci <daniele.verducci@nextcloud.com>
Signed-off-by: daniele-verducci <daniele.verducci@nextcloud.com>
Signed-off-by: daniele-verducci <daniele.verducci@nextcloud.com>
Signed-off-by: daniele-verducci <daniele.verducci@nextcloud.com>
Signed-off-by: daniele-verducci <daniele.verducci@nextcloud.com>
Signed-off-by: daniele-verducci <daniele.verducci@nextcloud.com>
Signed-off-by: daniele-verducci <daniele.verducci@nextcloud.com>
Signed-off-by: daniele-verducci <daniele.verducci@nextcloud.com>
Signed-off-by: daniele-verducci <daniele.verducci@nextcloud.com>
Signed-off-by: daniele-verducci <daniele.verducci@nextcloud.com>
Signed-off-by: daniele-verducci <daniele.verducci@nextcloud.com>
Signed-off-by: daniele-verducci <daniele.verducci@nextcloud.com>
Signed-off-by: daniele-verducci <daniele.verducci@nextcloud.com>
Signed-off-by: daniele-verducci <daniele.verducci@nextcloud.com>
Signed-off-by: daniele-verducci <daniele.verducci@nextcloud.com>
Signed-off-by: daniele-verducci <daniele.verducci@nextcloud.com>
…from their source Signed-off-by: daniele-verducci <daniele.verducci@nextcloud.com>
Signed-off-by: daniele-verducci <daniele.verducci@nextcloud.com>
Signed-off-by: daniele-verducci <daniele.verducci@nextcloud.com>
Signed-off-by: daniele-verducci <daniele.verducci@nextcloud.com>
Signed-off-by: alperozturk96 <alper_ozturk@proton.me>
73b8f84 to
b044cf2
Compare
|
blue-Light-Screenshot test failed, but no output was generated. Maybe a preliminary stage failed. |
|
APK file: https://github.com/nextcloud/android/actions/runs/32109517145/artifacts/9314672261 |
|
Hello there, We hope that the review process is going smooth and is helpful for you. We want to ensure your pull request is reviewed to your satisfaction. If you have a moment, our community management team would very much appreciate your feedback on your experience with this PR review process. Your feedback is valuable to us as we continuously strive to improve our community developer experience. Please take a moment to complete our short survey by clicking on the following link: https://cloud.nextcloud.com/apps/forms/s/i9Ago4EQRZ7TWxjfmeEpPkf6 Thank you for contributing to Nextcloud and we hope to hear from you soon! (If you believe you should not receive this message, you can add yourself to the blocklist.) |











Fix nextcloud/files-clients#114
Fix nextcloud/files-clients#128
🖼️ Screenshots
🏁 Checklist
/backport to stable-xx.x🤖 AI (if applicable)