hifive-snu / hifive-pitalium-explorer

:hand: Hifive test explorer
Apache License 2.0
4 stars 1 forks source link

Code review required #20

Closed simnalamburt closed 9 years ago

simnalamburt commented 9 years ago
  1. Used InputStream instead of ImageIO.

    이거 해서 많이 빨라지긴 했는데, 에지디텍터나 getDiff 는 BufferedImage를 쓰기때문에 ImageIO를 피할수가 없어서 여전히 느림. 테스트가 통과한경우를 볼때엔 쌩 스크린샷 파일을 그대로 출력하기때문에 꽤 빠르게 작동하는데, 테스트가 실패한 경우를 볼때엔 별 차이 없음.

  2. Simplified ImageController.

    쓸데없이 여러개로 쪼개진 함수 하나로 합침.

  3. Show white background instead of black background in Edge overlapping diff view.

    회사에서 그때 흰바탕이 더 보기 좋다고 그랬음.

  4. Use multiply as a blend mode for Edge overlapping view instead of alpha.

    이러면 Expected 엣지랑 Actual 엣지 겹치는곳이 검정색으로 표시됨.

  5. Use HTML5 <canvas> to overlap two image.

    var canvas = document.getElementById('myCanvas');
    var context = canvas.getContext('2d');
    var img = new Image();
    
    img.onload = function() {
     context.drawImage(img, 69, 50);
    };
    img.src = 'http://www.html5canvastutorials.com/demos/assets/darth-vader.jpg';
  6. Implemented image magnifier only for Edge overlapping mode.

Closes #17, #19

sim0629 commented 9 years ago

LGTM

kcm1700 commented 9 years ago

IE에서 검은색이 아니라 빨간색으로 보여요. 여기서도 예쁘진 않아도 보이게는 수정을 해야 할 거 같아요.

kcm1700 commented 9 years ago

IE workaround의 proof of concept: 93e23c75408b2f622d63167bb908b8e2db415c8d

kcm1700 commented 9 years ago

Reviewed. Looks good to me. Thanks for the great work.