Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[FEATURE REQUEST] New background image for the login screen #2991

Closed
10 of 11 tasks
ChrisEdS opened this issue Oct 7, 2020 · 8 comments · Fixed by #3023
Closed
10 of 11 tasks

[FEATURE REQUEST] New background image for the login screen #2991

ChrisEdS opened this issue Oct 7, 2020 · 8 comments · Fixed by #3023

Comments

@ChrisEdS
Copy link

ChrisEdS commented Oct 7, 2020

Describe the solution you'd like

With the ownCloud Server 10.5 release we said goodbye to the satellite that was visible in the login screen. In the Android app it is still visible. To be in sync the picture should be changed there as well.

Steps taken

I have already prepared the new images in the following branch:
https://github.com/owncloud/android/tree/new-ci-images

Please note

What should be considered is the fact that the positioning of the new image should start from the lower right edge. The new CI image has design elements only in the left and right corner.

In portrait format you should see the following part of the design:

image

AC:

  • Login view will include the new image
  • Top drawer will include the new image

TASKS

  • Research
  • Create branch new_background_images
  • Development tasks
    • Create util for matrix scaling
    • apply matrix scaling util for login and drawer image
    • add compatibility for whitelabel version
  • Code review and apply changes requested
  • Test background design with ownbranded version of the app
  • Design test plan
  • QA
  • Merge branch new_background_images into master

PR

@jesmrec
Copy link
Collaborator

jesmrec commented Oct 8, 2020

Will include in 2.16.

Also, we have to change in the top side of the drawer where the satellite image is shown.

@ChrisEdS
Copy link
Author

ChrisEdS commented Oct 8, 2020

Also, we have to change in the top side of the drawer where the satellite image is shown.

I'm one step ahead. 👍

https://github.com/owncloud/android/blob/77e19f96c8bcd9d5dd65af4551bcf70666cb3888/owncloudApp/src/main/res/drawable-hdpi/drawer_header_background.png

But that image needs to be improved maybe.

@theScrabi
Copy link
Contributor

Currently the app crashes on this branch. The stacktrace depics that the login button is null. Which can not be. This behaviour is not related to anything that the code does at the moment, which is weird and might required deeper investigation.

The scale view from bottom right is not working appropriately as well: https://github.com/owncloud/android/pull/2996/files#diff-64c334a87f1aea1fb0d3bcbc70d21e40d71ab4f51551665eec612117962f17b3

This function needs to be called every time the layout needs to be updated. This is required since we want the anchor point of the image to be on the bottom right of the screen and the image. Then we want to scale towards the top left. Either cropping the top part of the image or the left part of it. This needs to be done in order to ensure that 1. The image covers the whole screen, 2. the shapes on the bottom right are always visible.

However apart from crashing the this function is not even scaling appropriately. I'd like to test it on a device that allows to freely size the app screen like resizing windows. This can be done with software like android x86, or anbox on ubuntu.

Additionally we need to distinguish between branded and the regular application. This needs to be done since branded apps do not use this image, and so the scaling must fall back to center crop.

I think the easiest way to go here is to keep center crop scaling but design a background image for portrait mode. This way we can ensure that the shapes are always visible without having to do complex matrix transformation.

@abelgardep
Copy link
Contributor

I think the easiest way to go here is to keep center crop scaling but design a background image for portrait mode. This way we can ensure that the shapes are always visible without having to do complex matrix transformation.

100% Agree. From my point of view we should not overcomplicate this issue. New images included in this commit are pretty wide and when we try to center them, those shapes are missing. An image for portrait mode will be centered automatically fixing the problem we have when trying to scale it starting from bottom right.

@theScrabi
Copy link
Contributor

theScrabi commented Nov 18, 2020

@ChrisEdS Maybe you can directly send us the SVG versions these can be shown more crisp on phones even with normal resolution ;)

@jesmrec jesmrec added the Sprint label Nov 20, 2020
@abelgardep
Copy link
Contributor

abelgardep commented Nov 24, 2020

Drawer Login Portrait
Screenshot_1606210005 Screenshot_1606210015
Login Landscape
Screenshot_1606320706

EDIT: Final background images.

@ChrisEdS @michaelstingl @theScrabi @jesmrec

@ChrisEdS
Copy link
Author

How important are the shapes in login landscape mode? At the moment we lose them. If we want to add a new background image exclusively for landscape, we probably will need to ask for an additional image in oB for landscape.

I can provide the image. But what about branding? In ownBrander we only upload one picture.

@michaelstingl
Copy link
Contributor

I can provide the image. But what about branding? In ownBrander we only upload one picture.

Yes, single-image-only for branding customers please. Vanilla-oC can have some extra eyecandy, similar as we do with the new-features-wizard.

@abelgardep abelgardep linked a pull request Nov 25, 2020 that will close this issue
@jesmrec jesmrec removed the Sprint label Dec 2, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment