weijiang8410 / mathdoku

Automatically exported from code.google.com/p/mathdoku
0 stars 0 forks source link

Notification bar missing in landscape mode on some devices #143

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
I can see why it is doing this:

// If too little height then request full screen usage
if (mUtil.getDisplayHeight() < 750) {
    this.getWindow().setFlags(
        WindowManager.LayoutParams.FLAG_FULLSCREEN,
        WindowManager.LayoutParams.FLAG_FULLSCREEN);

But not sure I agree with it. Personally I find it annoying when apps remove my 
notification bar. This even happens on my Nexus 7, which should have enough 
space. I'm not saying this has to be changed, I'd just like it reviewed (and if 
the decision is to keep it, review the minimum display height, since it 
shouldn't really be necessary on a Nexus 7).

Original issue reported on code.google.com by em...@srlee.com on 17 Apr 2013 at 6:33

GoogleCodeExporter commented 9 years ago
This behavior is not introduced in this version. Therefore I see no reason to 
change this at this moment. 

Also note issue27 in which it is requested to add an option to allways play in 
full screen mode. Let discuss this in the context of v2.0. If okay, please move 
to v2.0.

Original comment by paul.din...@gmail.com on 17 Apr 2013 at 6:46

GoogleCodeExporter commented 9 years ago
Sorry - I didn't check against the "production" version - you're right - it's 
there as well. Changing milestone to 2.0 as requested.

Original comment by em...@srlee.com on 17 Apr 2013 at 7:26

GoogleCodeExporter commented 9 years ago
Removed the automatisch switch to full screen mode as new option to play in 
full screen mode was added in revision 357. 

Original comment by paul.din...@gmail.com on 21 Jun 2013 at 1:39

GoogleCodeExporter commented 9 years ago
Unfortunately the issue has not been verified explicitly by an independent 
tester.

Original comment by paul.din...@gmail.com on 8 Sep 2013 at 10:48