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

New PoGo Loading Screen #1429

Open
wants to merge 16 commits into
base: master
Choose a base branch
from
Open

New PoGo Loading Screen #1429

wants to merge 16 commits into from

Conversation

Markusch85
Copy link

@Markusch85 Markusch85 commented Aug 29, 2016

Closes issues

  • Closes #1406

Changes

  • Added new loading Screen instead of the SplashScreen and Busy Dialog

Change details
I've added a new Control as loading Screen which is like the original one. You're able to set a custom loading message to the screen.

Other informations

It's the latest Version of the new loading Screen. There is only one Image and message (like the original one)
Here is a Screenshot
e68e1214-7089-11e6-88b3-4b712ec38754

@viniciusbrasil
Copy link

Niceeeee -

@robipzo
Copy link

robipzo commented Aug 29, 2016

What about putting a picture (loading screen) under your designed loadingbar

@Markusch85
Copy link
Author

@robipzo What Picture do you mean? The PoGo Icon or another Image?

@VictoremV
Copy link

Another image like the original loadscreen from android @Markusch85

@robipzo
Copy link

robipzo commented Aug 29, 2016

@Markusch85 I'm not sure if we can use the original ones but there were postet some nice picture with a text (like look surround you idk) on github. I cant find them maybe you can

@robipzo
Copy link

robipzo commented Aug 29, 2016

And not the PoGo icon as i said a pokemon picture with a text(look surround you etc).I think you know what i mean

@ArthurSDev
Copy link

I want a appx haah :D

@jakubsuchybio
Copy link
Collaborator

Hey @Markusch85 could you join forces with @swisszeni ? He is already working on something similar for 2.0 on new repo, but I think it could be even brought into 1.1
You can look on discussion about this here: PoGo-Devs/PoGo#26

@MedaiP90
Copy link

@Markusch85 nice work!

@Markusch85
Copy link
Author

@MedaiP90 Thank you. Later I'll try to implement a nice loading Screen a upload a screenshot here.

@Markusch85
Copy link
Author

Hey guys. Here is another Screenshot of a complete new Loading Screen. What do you mean?
new info message

@viniciusbrasil
Copy link

Its very nice :)

@RigleyC
Copy link

RigleyC commented Aug 31, 2016

Wow! Nice job @Markusch85 Markusch85 very good same , only a idea, I saw that the @viniciusbrasil put some images of loading screen also, you could put in form of "slides" while loads. Well, only a ideia, nothing of important :)

@Markusch85
Copy link
Author

@RigleyC Thank you. I've implemented a random logic, which is able to choose an image with the correct description. I haven't found the images from @viniciusbrasil without text. So, I didn't add them. If I'll find these images without text, I'll add them. The case is, that we have to translate the texts.

@RigleyC
Copy link

RigleyC commented Aug 31, 2016

@Markusch85 So, with " without text" you wanna say "without text of alerts"?
I didn't understand very well. I caught these images, found it the identical color and I removed the text. Was it?
teste1

@robipzo
Copy link

robipzo commented Aug 31, 2016

Nice work! Every picture is nice but I think its the best to use the original one.Just to come as close as possible to niantic version :)

@Markusch85
Copy link
Author

@RigleyC That's the right direction. ;)
I can not edit images very good. If you can remove the Box, where normally the text is displayed, I can add it to the dialog with the correct translated message

@RigleyC
Copy link

RigleyC commented Aug 31, 2016

Hey @Markusch85 What do you think? The resolution are good?
test2

@viniciusbrasil
Copy link

Have more one without text

@viniciusbrasil
Copy link

You cant just put a box dialog message overlaping the text

@viniciusbrasil
Copy link

?

@Markusch85
Copy link
Author

Thank you guys. Here is the next Screen.
new picture

@Markusch85
Copy link
Author

@viniciusbrasil I can do it, but it seems like a hack. If we have more clean Images without any text, we are more flexible.
Thx for the next image. Are your able to remove the box? Otherwise I have to implement the logic new.

@Markusch85
Copy link
Author

@skyne Thanks for finding the typo. ;)
I've get the image from Pokemon-GO-App-Assets-and-Images folder here in this repository. The other images are made by the community. I'm not a good designer. So I need help with the images

@RigleyC Hey, can you change the Image Warnings/1.png? It's the second posted Image from me. That wood be nice.


namespace PokemonGo_UWP.Views
{
public sealed partial class InfoMessage : UserControl
Copy link
Collaborator

Choose a reason for hiding this comment

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

Where is this class ever used?

@Markusch85
Copy link
Author

Markusch85 commented Sep 1, 2016

@swisszeni Sorry, this was the first loading message popup (first screenshot). I Forget to remove it.

@ALL Thanks for checking the code. The will be cleaner if somebody will review it.

@RigleyC
Copy link

RigleyC commented Sep 1, 2016

@Markusch85 The image of "Gyarados"? To remove box and the text?
Yes, I can, but I don't have this image, you can send for me?

@Markusch85
Copy link
Author

@RigleyC Thank you again. ;)
Is it able to stretch the following image?
1

Otherwise, I've only found the next Image:
pokemon-go-loading-screen

@Seelengrab
Copy link
Collaborator

Hm.. Personally, I don't think we should add custom loading screens.

There's no "official" translation and localisation (or rather, reliably translating the meaning) might be a problem. For example, "Try to not smoke in crowds. Nobody likes to breathe Koffing fumes" can hardly be accurate since smokers obviously want to smoke (otherwise they wouldn't be smokers). Now I'm sure you meant something along the lines of "Not everyone wants to inhale Koffing fumes", but we'd have to check every PR and every translation of custom lines for those small mistakes and errors that can be made in translation. I just don't think we have the manpower (or fluency in all those languages) necessary for that, even if it's just a relatively small feature now.

About the dialog box/information at the top of the screen.. @swisszeni is currently working on a refactor of the message box system for a future release here, which has been mentioned before in this thread. It's going to look like the one in the official app from what I've seen of it, so remaking it now seems redundant (especially since he'd have to incorporate/refactor your changes as well).

In the end, it's @ST-Apps' decision, those are just my two cents about this.

@Markusch85
Copy link
Author

Yeah, I think you're right.We should just use the official Screens. I have no Pokémon Go for Android or iOS, so I don't know what loading Screens there are. I thougt That are all the Images I've got from the community. I'll revert the Images and add only the original one.

About the Dialog: I've removed the Control, because I don't use it. That was only the first new Loading message. Now it's the full Screen loading message.

@Seelengrab Thanks for your opinion.

@RigleyC Sorry for your work, which won't be in the PoGo App. I know that you've spent much time to remove the Texts and boxes. But I think we should use only the original one. Than the App will be like the Android or iOS App.

@Seelengrab
Copy link
Collaborator

No problem. Having custom loading screens and all that stuff isn't out of the question though, it's just not a priority right now and would add more work than it's worth at the moment :) So maybe (possibly, not certainly!) it'll be in at a later date.

Alright, once you've removed the images and changed the necessary parts of your code, please update the original post of the PR to be representative of the current status of the PR and what it looks like. After that, I think it's good to go.

@Markusch85
Copy link
Author

Markusch85 commented Sep 1, 2016

I've reverted the custom images an Messages. Here is the latest screenshot. The original post is updated.

@Seelengrab
Copy link
Collaborator

Alright, apart from the still distorted image, LGTM.

@Markusch85
Copy link
Author

Markusch85 commented Sep 1, 2016

I know the image has not the best resolution, but that's the only image I've found. It's from the Pokemon-GO-App-Assets-and-Images Folder in this repository.

@RigleyC
Copy link

RigleyC commented Sep 1, 2016

@Markusch85 Oh, all right at least we have the images, I hope can be implemented one day. :)
Well, I'm going try improve the image oficial, soon I going post something. ;)
You saying (and I see) that is little stretched, I'm gonna see the that I can do.

@Rover656
Copy link

Rover656 commented Sep 2, 2016

Great work!

@Markusch85 Markusch85 changed the title New gps message New PoGo Loading Screen Sep 2, 2016
@Rover656
Copy link

Rover656 commented Sep 3, 2016

@FloMoTionGo If you want to install latest official release goto http://github.com/ST-Apps/PoGo-UWP/releases and if you want to compile a bleeding-edge version download the repo using the green button on home page and follow instructions of dev wiki to get up and running.
Please, if you have any more questions, ask on Reddit.
Also, please edit your posts rather than double post, thanks.

@ArthurSDev
Copy link

wp_ss_20160905_0006
We could implement a GPS message (GPS signal not found) like this

@Markusch85
Copy link
Author

There is a re-implementation of the message infos (e.g. your GPS message) in progress. So, another developer will add these feature in the future

@Somebody2804
Copy link

It's not really that easy tho tbh is it @Rover656 I think people shouldn't try if they aren't a dev I tried but I'm not gonna pretend I understand because if I try a bleeding-edge install it may do something really bad and I'll be a total noob and "turn it off and on again"

@Seelengrab
Copy link
Collaborator

Please keep discussion relevant to the issue. Off-topic discussion or discussion related to another issue/feature request is best done in the respective issue, discord or the subreddit.

@Sinbad1999
Copy link

i love the new loading screens

@PristoDavi
Copy link

Nice PR!

@ghost
Copy link

ghost commented Oct 19, 2016

Start screen customizable, add all images and add settings for custom start screen

@certiodaniel
Copy link

Add the pokemon loading screen music to finish it off:-

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

Successfully merging this pull request may close these issues.