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

Fix #2652: Splash Screen stretched #2655

Closed
wants to merge 5 commits into from
Closed

Conversation

anshbajpai
Copy link

@anshbajpai anshbajpai commented Feb 8, 2021

Explanation

This PR fixes #2652 the splash screen, and splash screen doesn't stretch anymore on large screen devices, by adding a new drawable file named splash_image.xml

Mobile Portrait Screenshot
MobilePortrait

Tablet Portrait Screenshot
TabletPortrait

Checklist

  • The PR title starts with "Fix #bugnum: ", followed by a short, clear summary of the changes. (If this PR fixes part of an issue, prefix the title with "Fix part of #bugnum: ...".)
  • The PR explanation includes the words "Fixes #bugnum: ..." (or "Fixes part of #bugnum" if the PR only partially fixes an issue).
  • The PR follows the style guide.
  • The PR does not contain any unnecessary auto-generated code from Android Studio.
  • The PR is made from a branch that's not called "develop".
  • The PR is made from a branch that is up-to-date with "develop".
  • The PR's branch is based on "develop" and not on any other branch.
  • The PR is assigned to an appropriate reviewer in both the Assignees and the Reviewers sections.

.idea/misc.xml Outdated Show resolved Hide resolved
.idea/misc.xml Outdated Show resolved Hide resolved
@rt4914
Copy link
Contributor

rt4914 commented Feb 8, 2021

@anshbajpai Please add images for splash screen in description for following cases:

  1. Mobile Portrait
  2. Mobile Landscape
  3. Tablet Portrait
  4. Tablet Landscape

In general any UI change related PR should have screenshots in description.

Copy link
Contributor

@rt4914 rt4914 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This approach is better but the issue is with the image in bigger devices. On bigger devices the image looks small.

app/src/main/res/drawable/splash_image.xml Show resolved Hide resolved
@rt4914
Copy link
Contributor

rt4914 commented Feb 8, 2021

@anshbajpai Update title to Fix #2652: Splash Screen stretched

@rt4914 rt4914 assigned anshbajpai and unassigned rt4914 and BenHenning Feb 8, 2021
@anshbajpai anshbajpai changed the title Fix #2652 - Splash Screen Stretched Fix #2652: Splash Screen stretched Feb 8, 2021
@anshbajpai
Copy link
Author

This approach is better but the issue is with the image in bigger devices. On bigger devices the image looks small.

@rt4914 so does the main logo should look the same like it does on mobile , on bigger devices like tablet . I have updated the current description with both the respective pictures.

@anshbajpai
Copy link
Author

@rt4914 I am not sure, why one of the checks has failed , could you help me in understanding it ?

@FareesHussain
Copy link
Contributor

@rt4914 I am not sure, why one of the checks has failed , could you help me in understanding it ?

@anshbajpai Don't worry about it. There are some flaky tests. We just need to re-run the workflow

@rt4914
Copy link
Contributor

rt4914 commented Feb 8, 2021

@mschanteltc Have a look at the PR description. Can we have mocks for mobile landscape, tablet landscape and tablet portrait to check what should be the exact dimens for the logo.

@rt4914 rt4914 assigned mschanteltc and unassigned anshbajpai Feb 8, 2021
@mschanteltc
Copy link

@mschanteltc Have a look at the PR description. Can we have mocks for mobile landscape, tablet landscape and tablet portrait to check what should be the exact dimens for the logo.

Is it possible to format the logo depending on the device width? I have mocks to show what percentage of the screen width the logo and margins should be.

@rt4914
Copy link
Contributor

rt4914 commented Feb 15, 2021

@mschanteltc Have a look at the PR description. Can we have mocks for mobile landscape, tablet landscape and tablet portrait to check what should be the exact dimens for the logo.

Is it possible to format the logo depending on the device width? I have mocks to show what percentage of the screen width the logo and margins should be.

@mschanteltc Yes it is possible to control margins based on device width but we cannot see the width can you provide the specs link. Also Image 1 and Image 2 have different sit above line UI. Suggest making it similar so that it becomes easier to implement.

@mschanteltc
Copy link

@mschanteltc Have a look at the PR description. Can we have mocks for mobile landscape, tablet landscape and tablet portrait to check what should be the exact dimens for the logo.

Is it possible to format the logo depending on the device width? I have mocks to show what percentage of the screen width the logo and margins should be.

@mschanteltc Yes it is possible to control margins based on device width but we cannot see the width can you provide the specs link. Also Image 1 and Image 2 have different sit above line UI. Suggest making it similar so that it becomes easier to implement.

Updated the links to include the specs. Also fixed the formatting of the logo so they should be the same now (mobile portrait and tablet portrait).

@rt4914
Copy link
Contributor

rt4914 commented Feb 17, 2021

@mschanteltc Have a look at the PR description. Can we have mocks for mobile landscape, tablet landscape and tablet portrait to check what should be the exact dimens for the logo.

Is it possible to format the logo depending on the device width? I have mocks to show what percentage of the screen width the logo and margins should be.

@mschanteltc Yes it is possible to control margins based on device width but we cannot see the width can you provide the specs link. Also Image 1 and Image 2 have different sit above line UI. Suggest making it similar so that it becomes easier to implement.

Updated the links to include the specs. Also fixed the formatting of the logo so they should be the same now (mobile portrait and tablet portrait).

@anshbajpai I think you can work on this now.

@BenHenning BenHenning added the PR don't merge - NEEDS UPDATE Corresponds to PRs that need to be updated with the latest develop changes before they can be merged label Feb 24, 2021
@BenHenning BenHenning removed the PR don't merge - NEEDS UPDATE Corresponds to PRs that need to be updated with the latest develop changes before they can be merged label Feb 24, 2021
@FareesHussain
Copy link
Contributor

@anshbajpai Any updates on this?

@FareesHussain
Copy link
Contributor

Closing this as there is no update from @anshbajpai

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Splash Screen Stretched
6 participants