eiginn / ndt

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

Flash client: Debug button should not be visible by default #160

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Run NDT Flash client from http://www.measurementlab.net/tools/ndt
2. Wait for test to complete

What is the expected output? What do you see instead?
The NDT Flash client shows every user a "Debug" button. It's not clear to a 
casual user what this information means or how they should interpret it. The 
Flash client should not show this by default. Instead, it should only show it 
in a "debug" mode (possibly initiated by a keyboard shortcut or URL variable).

What version of the product are you using? On what operating system?
Chrome 37 on Ubuntu x64

Please provide any additional information below.

Original issue reported on code.google.com by mtly...@google.com on 1 Oct 2014 at 6:41

GoogleCodeExporter commented 9 years ago

Original comment by jerem...@poczta.fm on 27 Oct 2014 at 10:46

GoogleCodeExporter commented 9 years ago

Original comment by skost...@soldevelo.com on 3 Nov 2014 at 8:52

GoogleCodeExporter commented 9 years ago
Currently there is debug option in config file which allows to set this 
variable and based on this setting debug button will or will not appear. It has 
been implemented earlier but I found some bug and fixed this in r1132. Is such 
solution correct? 

Original comment by skost...@soldevelo.com on 3 Nov 2014 at 3:16

GoogleCodeExporter commented 9 years ago
That sounds reasonable.

Original comment by mtly...@google.com on 3 Nov 2014 at 8:01

GoogleCodeExporter commented 9 years ago
Merged into trunk

Original comment by skost...@soldevelo.com on 13 Nov 2014 at 10:16