Skip to content
This repository has been archived by the owner on May 5, 2020. It is now read-only.

Proper support for multiple screen aspect ratios #264

Open
7 tasks
justkd opened this issue May 18, 2018 · 22 comments
Open
7 tasks

Proper support for multiple screen aspect ratios #264

justkd opened this issue May 18, 2018 · 22 comments

Comments

@justkd
Copy link
Contributor

justkd commented May 18, 2018

Description

As a developer,
I need to update media assets to be aspect ratio agnostic. This especially affects iPhoneX.

This seems to be a hefty issue...

Some of what may be expected to be several small visual assets are actually single, full screen images.

Eg. The start screen is a single full screen image. The buttons are a part of the background, not individual assets used as the image for an actual button. The map view has the same issue: the images for the unlocked buttons are actually full screen with a transparent background.

Because of the forced aspect ratio of these assets, they can not be properly placed for the iPhoneX (and probably would have caused some headache on iPad).

The assets need to be broken up into individual components, including the stripes that are regularly used as an accent in the bottom corners. Then all of these assets will need to be lain out again, with proper constraints to account for the three different aspect ratios on iOS.

The continue button needs to be moved or changed in order to account for the extra space at the top of the iPhone X. It should probably be moved lower rather than elongated, since the iPhoneX will have different margins for a slide-out view depending on which landscape mode is currently being used (the top could be on the left or right).

This is an issue that also affects layouts in Android, since the full screen assets have similar problems with the multitude of Android screen sizes.

Mocks

image.png

screen shot 2018-05-17 at 10 01 08 pm

screen shot 2018-05-17 at 10 01 22 pm

screen shot 2018-05-17 at 10 01 32 pm

screen shot 2018-05-17 at 10 08 10 pm

Acceptance Criteria

Update [Required]

  • check and update programmatic layouts for all mini-games
  • update visual assets to be separate components where appropriate
  • update full screen asses (such as backgrounds) to be aspect ratio agnostic (maybe extra neutral space around the edges)
  • update storyboard layouts to use the new visual assets
  • check and update IBOutlets and IBActions to ensure code is still relevant and functional following storyboard updates

Definition of Done

  • All of the required items are completed.
  • Approval by 1 mentor.
@justkd justkd changed the title Xcode 9.3 Storyboards, iPhone X, serious break Xcode 9.3 Storyboards, iPhone X, serious but non-fatal break May 18, 2018
@justkd justkd changed the title Xcode 9.3 Storyboards, iPhone X, serious but non-fatal break Proper support for multiple screen aspect ratios Jul 27, 2018
@anubhavpulkit
Copy link
Contributor

Can I work on this issue?

@sunjunkie
Copy link
Contributor

@AnubhavSingh16 What do you propose to do exactly? This issue involves some UI/graphics work in addition to coding.

@anubhavpulkit
Copy link
Contributor

@sunjunkie I want to check and update IBOutlets and IBAction to ensure code is still relevant and functional following storyboard updates.

@sunjunkie
Copy link
Contributor

This issue requires UI and graphic updates before IBOutlets and IBActions can be checked or updated. So it cannot be worked on until new graphic elements are created.

@anubhavpulkit
Copy link
Contributor

This issue requires UI and graphic updates before IBOutlets and IBActions can be checked or updated. So it cannot be worked on until new graphic elements are created.

Ok so Can I Work on the update full-screen asses (such as backgrounds) to be aspect ratio agnostic (maybe extra neutral space around the edges)

@sunjunkie
Copy link
Contributor

No, that would only be a temporary fix. The full screen assets have to be recreated and the screen layout redone based on the new images. This will require a lot of discussion and UI/image development. This issue is not ready to be worked on until we get the graphics recreated.

@MANI14011998
Copy link

MANI14011998 commented Feb 22, 2020

This issue requires UI and graphic updates before IBOutlets and IBActions can be checked or updated. So it cannot be worked on until new graphic elements are created.

Hey @sunjunkie I have created new graphic images assets for all this issue.Is that ok to reopen this issue again

@justkd
Copy link
Contributor Author

justkd commented Feb 24, 2020

This issue is still open. Those closed tags are on other issues that were directed to this particular one. If you have some assets you've made, maybe you could post some examples here, or a link where we can download them, so we can see how they address the issue and discuss what might need to happen to implement a fix?

@vatsalkul
Copy link
Member

Hey! @sunjunkie @justkd I have made some graphics of buttons used in the app. Please have a look and let me know if I can replace the older Start View Scene Components with new ones. I will add more graphical stuffs in the folder after approval.
Drive Link

@MANI14011998
Copy link

School-3 with chimney
WhatsApp Image 2020-02-25 at 1 05 38 AM
old one
Screenshot 2020-02-21 at 9 20 52 PM

@justkd have look if this is assign to me i will make whole assets

@sunjunkie
Copy link
Contributor

@vatsalkul The images should not have extra pixels. Crop the image as close as possible to the button - as in this screenshot:
image

@sunjunkie
Copy link
Contributor

Let's break up this issue into smaller pieces. One issue for each screen we want to update.
@vatsalkul you are welcome to work on the Start View scene.
@MANI14011998 you are welcome to take the Mine Sweeper Game.
@AnubhavSingh16 you are welcome to take the map screen.
Each of you please create an issue for just one screen. Once I approve it, you can work on the image assets for that screen. Please follow the issue template.

@MANI14011998
Copy link

Ohk ma'am i will do that

@vatsalkul
Copy link
Member

@sunjunkie Please review my Issue

@anubhavpulkit
Copy link
Contributor

anubhavpulkit commented Feb 29, 2020 via email

@anubhavpulkit
Copy link
Contributor

@sunjunkie I can't find any issue in map screen as mention above I think it is solved so Can I take the Mine Sweeper Game issue?

@anubhavpulkit
Copy link
Contributor

There is only one issue on the map which is Smoke of the chimney.

@anubhavpulkit
Copy link
Contributor

@sunjunkie So can I work on Minesweeper game issue, please.

@sunjunkie
Copy link
Contributor

@AnubhavSingh16 There is a big issue with the map - The map should have a blackground image and separate images for the buildings. So there is a work to be done breaking that up.

Anyway, yes, create an issue for the mine sweeper game and I'll review it before you work on it.

@anubhavpulkit
Copy link
Contributor

Thank You so much @sunjunkie

@anubhavpulkit
Copy link
Contributor

@sunjunkie now only first issue is not solved in this list of issues so can you close this issue and make(or I open) a new one only for the first issue.

@sunjunkie
Copy link
Contributor

Ok. I'll take a look.

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

No branches or pull requests

5 participants