Skip to content
This repository has been archived by the owner on Jul 8, 2020. It is now read-only.

Fixed 107 #108

Merged
merged 5 commits into from
Jun 11, 2016
Merged

Fixed 107 #108

merged 5 commits into from
Jun 11, 2016

Conversation

shundroid
Copy link
Member

@shundroid shundroid commented May 28, 2016

概要

fixed #107
@nakamurataichi さんの報告です。ありがとうございます。

詳細

Issue(#107)にも書きましたが、
2つ空のフレームがあり、1つのフレームに描いた後、フレームを交換すると、Previewがバグります。
原因は、insertBeforeをミスっていて、実際は交換していなかったという・・ものです。

直した点

  • insertBeforeで正しく交換するように変更
  • previewは、Canvasでなく、Imgタグを使用。
    • デバッグがしやすい(src属性で見れる)
    • 変えたおかげでinsertBeforeのミスに気付いた

@hrl7 さん、レビューをしていただいてもよろしいでしょうか。


This change is Reviewable

@zigen
Copy link
Contributor

zigen commented Jun 11, 2016

Reviewed 3 of 3 files at r3.
Review status: all files reviewed at latest revision, all discussions resolved.


Comments from Reviewable

@zigen
Copy link
Contributor

zigen commented Jun 11, 2016

:lgtm:

Previously, shundroid wrote…

Fixed 107

概要

fixed #107

@nakamurataichi さんの報告です。ありがとうございます。

詳細

Issue(#107)にも書きましたが、

2つ空のフレームがあり、1つのフレームに描いた後、フレームを交換すると、Previewがバグります。

原因は、insertBeforeをミスっていて、実際は交換していなかったという・・ものです。

直した点

  • insertBeforeで正しく交換するように変更
  • previewは、Canvasでなく、Imgタグを使用。
    • デバッグがしやすい(src属性で見れる)
    • 変えたおかげでinsertBeforeのミスに気付いた

@hrl7 さん、レビューをしていただいてもよろしいでしょうか。


Review status: :shipit: all files reviewed at latest revision, all discussions resolved, all commit checks successful.


Comments from Reviewable

@zigen zigen merged commit ed0af69 into comozilla:develop Jun 11, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants