SpaceDash icon indicating copy to clipboard operation
SpaceDash copied to clipboard

Create a Rocket Details Page

Open swiftlysingh opened this issue 4 years ago • 22 comments

I am attaching some UI Mockups, which should be opened when someone taps on the rocket on the home page. There should be a little animation that will extend the white part and everything should dissolve in.

96335627-accaa580-1097-11eb-91be-214dc2c0305e 96335716-21054900-1098-11eb-8127-0877d12a6fca

The second screen should open when someone taps on the read more button.

The first order of business is to create the UI, animation can be configured later. Use dummy data for now, we will integrate it with API afterward.

PS: I am not the best designer, if you have some ideas to better the UI or animation, let us discuss in the comments.

swiftlysingh avatar Sep 25 '20 20:09 swiftlysingh

Hi. I'll work on this

devwaseem avatar Oct 02 '20 06:10 devwaseem

Sure thing. I am assigning it to you.

swiftlysingh avatar Oct 02 '20 10:10 swiftlysingh

@devwaseem Can you confirm if you are still working on this issue?

swiftlysingh avatar Oct 06 '20 12:10 swiftlysingh

Can we use programmatic UI to achieve this?

dheerajghub avatar Oct 08 '20 15:10 dheerajghub

I generally prefer but you definitely can use programmatic UI. Just make sure you have properly documented the code. I would recommend this video for documenting your code.

swiftlysingh avatar Oct 08 '20 15:10 swiftlysingh

Yeah sure! I'll do this. Can you assign it to me? And can I add my own design ideas in it?

dheerajghub avatar Oct 09 '20 01:10 dheerajghub

if the design ideas are minor it will be fine. But if there is anything major, provide mockups of the changes in the comments of this issue before implementing code. @dheerajghub

swiftlysingh avatar Oct 09 '20 03:10 swiftlysingh

Sure thing. I am assigning this to you. While you are here, do 🌟 the repo!

swiftlysingh avatar Oct 09 '20 03:10 swiftlysingh

Look at these designs do you find these relevant, I've already done with the first part 😅 but the second one is just mock up! Any modifications you want me to do! Once you acknowledge me that this is relevant and good I'll proceed further.

little modification!

dheerajghub avatar Oct 09 '20 04:10 dheerajghub

@dheerajghub I don't think that would work. I have some metrics that this design is not. Also, I have plans to integrate AR on tap action of Rockets.

On second thoughts, if you can somehow integrate all the metrics in the little pop-up, while not overcrowding it and having an option to add a button for AR in the future that would be great.

swiftlysingh avatar Oct 09 '20 05:10 swiftlysingh

AR Button can be placed on the rocket detail page, If you want that! BTW if we implement two popups in one view controller that sure will become messy!

dheerajghub avatar Oct 09 '20 06:10 dheerajghub

I am not saying two popups. I am saying add more metrics to the one you have made for Description. After a line or two of description, add some metrics like present in my design. Which would increase the pop-up's height.

swiftlysingh avatar Oct 09 '20 06:10 swiftlysingh

With metrics you mean data? And what about AR Button?

dheerajghub avatar Oct 09 '20 06:10 dheerajghub

Let's discuss this over slack!

swiftlysingh avatar Oct 09 '20 06:10 swiftlysingh

This is my final work what do you think?

dheerajghub avatar Oct 09 '20 12:10 dheerajghub

Personally, I don't like it. The initial mockup was according to me more inclusive of the current design. Can you let me know whats wrong with my design?

swiftlysingh avatar Oct 09 '20 15:10 swiftlysingh

In your second mockup, there is a popUp view which contains a description of the rocket what if the description is too long this is not a great idea to show it as a popup! What's wrong about my design?? Give feedback to both views. Please! And What exactly you don't like about this!

dheerajghub avatar Oct 09 '20 17:10 dheerajghub

It's ok to create a pop-up as you designed for the read more. If the description is too long, it should adjust accordingly. It's not a good option to leave a full view empty "if" description is long.

As for your implementation, I don't like the design. Plus, if you have actually looked at my design it's not a pop-up, its a separate view. In your design, we would be having the pop-up inside of pop-ups I don't think that's a good idea.

swiftlysingh avatar Oct 10 '20 06:10 swiftlysingh

@dheerajghub you will need to open a new pr as pr #62 was not building successfully and is reverted back.

swiftlysingh avatar Oct 22 '20 20:10 swiftlysingh

@dheerajghub I am going to unassign this issue because of no reply and inactivity.

swiftlysingh avatar Oct 27 '20 06:10 swiftlysingh

@pushpinderpalsingh im working on this issue can you please assign to me?

anubhavpulkit avatar Oct 27 '20 06:10 anubhavpulkit

Sure thing. I am assigning this to you.

swiftlysingh avatar Oct 27 '20 06:10 swiftlysingh