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

Improve Rant loading (& fix rendering problems) #37

Open
Frioo opened this issue Nov 19, 2017 · 11 comments
Open

Improve Rant loading (& fix rendering problems) #37

Frioo opened this issue Nov 19, 2017 · 11 comments

Comments

@Frioo
Copy link

Frioo commented Nov 19, 2017

Sometimes when I click a rant nothing happens even though I wait for a few seconds.
The previous rant does not disappear from the detailed view. Other times, it will show up after a few seconds.
My suggestion: review the process of opening a new rant (sometimes previous ones do not disappear, which is weird) and show a progress bar to inform the user something is happening.
Cheers!

@JakubSteplowski
Copy link
Owner

You should already see the indeterminate progress ring at the bottom of the Rant Page:

ezgif com-optimize

Let me know if you don't see it.

@Frioo
Copy link
Author

Frioo commented Nov 19, 2017

Couldn't see it. I was using the dark theme (purpleish, just like in the mobile app).
I switched to the light theme, could see it just fine, now turned the dark one back again and it's working.
What a little bug! I will try to capture the issue with clicking on a rant not doing anything.

@JakubSteplowski
Copy link
Owner

Are you on PC or mobile?
And if you're on mobile, which device are you using?

@Frioo
Copy link
Author

Frioo commented Nov 19, 2017

PC, W10 Pro

@JakubSteplowski
Copy link
Owner

Really strange render problem.
It doesn't seem releated to the app, though.

I hope it won't occur anymore in future, the progress ring is just a XAML component which should be loaded correctly by Windows.

Anyway, if you see a blank screen and not an error message, the app is loading the Rant normally, nothing to worry about, just wait a few seconds.
If something is wrong (with your internet connection, devRant servers or problems with app) it shows an error message:

immagine

If you don't see it, just wait. ;)

I will close the issue for now, but if it will occur again or something similar, just write here. Thanks. :)

@Frioo
Copy link
Author

Frioo commented Nov 19, 2017

Maybe fetching the data is really fast, but displaying it and all that rendering stuff makes it so weird?
bug
Anyways, I will try to capture that main issue but it's really hard to reproduce.

@JakubSteplowski
Copy link
Owner

I will try to improve the rendering.
Maybe I will find a way to remove at all the cache (that maybe cause that it appears all at once sometimes).

@JakubSteplowski JakubSteplowski changed the title Show (indeterminate) progress bar when loading a new rant. Improve Rant loading (& fix render problems) Nov 19, 2017
@Frioo
Copy link
Author

Frioo commented Nov 19, 2017

I managed to capture that bug. Here's the video.
The first occurence is at the very beginning, and then skip to the moment in which I open my notifs (it happens over there too). Weird since it works just fine after the second click.

@JakubSteplowski
Copy link
Owner

Wait, now you're talking about a problem with the click event?
You need to click directly on the content to activate the event, it's ignored if you click on the border.

@Frioo
Copy link
Author

Frioo commented Nov 19, 2017

So it's a feature!

@JakubSteplowski
Copy link
Owner

It's more like the way the ListViewItems tap event works. It's not something I did. :P
I will try to do some tests with the PointerPress event, which maybe with the mouse will work better.

@JakubSteplowski JakubSteplowski changed the title Improve Rant loading (& fix render problems) Improve Rant loading (& fix rendering problems) Nov 30, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants