Logan676 / anymemo

Automatically exported from code.google.com/p/anymemo
GNU General Public License v2.0
0 stars 0 forks source link

Rapid Orientation Changes creates thinking Messagebox #276

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. On phone have auto-rotate settings enabled
2. Open App in Portrait orientation.
3. Turn phone to Landscape
4. Turn phone back to Portait quickly
5. Repeat steps 2-3 twice or even 3 times.
6. Stop rotating
7. Look at dialog box with the spinning circle

A side issue is there is also a slight flicker on rotation now, that 
occasionally will reveal the answer during orientation change before the HTML 
properties take effect.

What is the AnyMemo version (E.g 8.999.1, 9.0, 9.1.1)?
Version 10. However I believe this problem also started happening in the 
version 9.0 series. 

What is your Android version?
Android 4.2

Original issue reported on code.google.com by Neofireb...@gmail.com on 9 Nov 2013 at 12:57

GoogleCodeExporter commented 9 years ago
I can confirm the problem. The progress dialog won't disappear after successive 
quick rotation. 

The key difference between that AnyMemo 10.2 and 10.3 is that AnyMemo will 
destroy and recreate the ui after screen rotation. This is done according to 
Android development guide and for compatibility with Android 4.3 and 4.4.

The flicker is caused by the destroy and recreate process.

Original comment by mrlhwlib...@gmail.com on 9 Nov 2013 at 6:30

GoogleCodeExporter commented 9 years ago
Good to know. Thanks for the info and excellent work.

Original comment by Neofireb...@gmail.com on 10 Nov 2013 at 6:42

GoogleCodeExporter commented 9 years ago
This bug is actually quite hard to fix. It will take some extra time to fix.

If the dialog didn't disappear, you can click outside the spinning dialog to 
cancel it. I don't find any negative effect on db if the spinning dialog didn't 
disappear. If you see any corruption / inconsistent in db, please let me know 
and I will make this issue higher priority if you see so.

Original comment by mrlhwlib...@gmail.com on 19 Nov 2013 at 12:27

GoogleCodeExporter commented 9 years ago
Yes, I am aware of canceling it through clicking outside of the box. I cant say 
I've seen a corruption from it or a negative effect. I will report if such 
thing does happen

Original comment by Neofireb...@gmail.com on 19 Nov 2013 at 2:42

GoogleCodeExporter commented 9 years ago
Finally fixed the problem. You can try out tomorrow's nightly build and see if 
it still happens.

http://code.google.com/p/anymemo/source/detail?r=2970dcc517bff6030be0edf25c9ec9c
d2d73f3cc

Original comment by mrlhwlib...@gmail.com on 27 Nov 2013 at 2:12

GoogleCodeExporter commented 9 years ago
BTW, nightly builds are here: 
http://anymemo.info/autobuild/job/AnyMemo/

tomorrow is Nov 28 UTC.

Original comment by mrlhwlib...@gmail.com on 27 Nov 2013 at 2:13

GoogleCodeExporter commented 9 years ago
This is indeed fixed for the messagebox not being destroyed problem. 

Though, now when there is an orientation change to landscape the grading 
buttons are now across the right side of the screen instead of against the 
bottom. Im not sure if this was intentionally done for tablets. Though this 
looks ugly and messes up the layout for display for small-screen phones 
(especially mine with a cracked screen). Is there a way to make that optionable 
or revert to the old layout?

Original comment by 1kblabbi...@gmail.com on 4 Dec 2013 at 8:01

GoogleCodeExporter commented 9 years ago
The layout was to address issue 231 that enhances UI for large screens. I found 
it is also useful for smaller screens, but it is open to discussion. You can 
add your concerns to issue 231.

Original comment by mrlhwlib...@gmail.com on 4 Dec 2013 at 11:24

GoogleCodeExporter commented 9 years ago
In tomorrow's nightly build, I limit the new landscape layout to large screen 
devices only. Since this issue is closed, further discussion are on issue 231.

Original comment by mrlhwlib...@gmail.com on 7 Dec 2013 at 1:33