testselenium / fighting-layout-bugs

Automatically exported from code.google.com/p/fighting-layout-bugs
Apache License 2.0
1 stars 1 forks source link

possible bug in too low contrast detector #13

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
screenshot attached
link:
https://www.odesk.com/o/profiles/users/WordPress-SEO-SEM-SMM-Expert-AdWords-PPC-
Campaign-Magento-Admin_~~517fafc57ae2d3a3/

Original issue reported on code.google.com by nihuhoid on 25 Jul 2012 at 2:41

Attachments:

GoogleCodeExporter commented 8 years ago
Hmm, I failed to reproduce this behaviour.

Does this always happen, when you run your tests?

Could you please add the following code to your test case:

        final File tempDir = ...;
        final AtomicInteger i = new AtomicInteger(0);
        final NumberFormat nf = new DecimalFormat("00");
        Visualization.registerListener(new Visualization.Listener() {
            @Override
            public void algorithmStepFinished(String algorithm, String stepDescription, int[][] tempResult) {
                ImageHelper.pixelsToPngFile(tempResult, new File(tempDir, nf.format(i.incrementAndGet()) + "_" + algorithm + ".png"));
            }
            @Override
            public void algorithmFinished(String algorithm, String stepDescription, int[][] result) {
                ImageHelper.pixelsToPngFile(result, new File(tempDir, nf.format(i.incrementAndGet()) + "_" + algorithm + ".png"));
            }
        });

and than attach a zip file with all the temporary images.

Thank you in advance.

Original comment by michael.tamm2 on 29 Jul 2012 at 5:06

GoogleCodeExporter commented 8 years ago
Already resolved. 
Problem was caused by low screen color depth.
I'm running it in Xvfb and it was launched with 8 bit color instead 24.

Thanks.

Original comment by nihuhoid on 7 Aug 2012 at 6:34

GoogleCodeExporter commented 8 years ago
Already resolved. 
Problem was caused by low screen color depth.
I'm running it in Xvfb and it was launched with 8 bit color instead 24.

Thanks.

Original comment by nihuhoid on 7 Aug 2012 at 6:40

GoogleCodeExporter commented 8 years ago

Original comment by michael.tamm2 on 21 Aug 2012 at 6:36