Skip to content

Fixed the problem that background is not drawn - #14

Closed
mikkame wants to merge 1 commit into
thinreports-php:masterfrom
mikkame:fixed-background-not-drawn
Closed

Fixed the problem that background is not drawn#14
mikkame wants to merge 1 commit into
thinreports-php:masterfrom
mikkame:fixed-background-not-drawn

Conversation

@mikkame

@mikkame mikkame commented Nov 17, 2015

Copy link
Copy Markdown
Contributor

バッックグラウンドが描画されない問題を解消

@hidakatsuya

Copy link
Copy Markdown
Contributor

@mikkame

ずいぶん時間が経ってしまいましたが、 DF が指定されいないとどう表示されるのでしょう?色が描画されない?問題のスクショなどを貼ってもらえると理解が早いです。

@mikkame

mikkame commented Apr 12, 2016

Copy link
Copy Markdown
Contributor Author

問題の発生するtlfファイルを添付します。

example.tlf.zip
v0.8.0で出力を行った場合は以下のように
example(v0.8.0).pdf

修正済みバージョンでは以下のように出力されるようになります。
example(fixed).pdf

@mikkame

mikkame commented Apr 12, 2016

Copy link
Copy Markdown
Contributor Author

久々に確認したんですが、修正済みは線未指定のものも描画されてしまっていますね・・・
要再修正ですねorz
Issueに起こした方がよろしいでしょうか?

@hidakatsuya

Copy link
Copy Markdown
Contributor

@mikkame

ありがとうございます。なるほど、理解しました

久々に確認したんですが、修正済みは線未指定のものも描画されてしまっていますね・・・
要再修正ですねorz

あ、ほんとですね。

Issueに起こした方がよろしいでしょうか?

いえ、お手数ですが、修正していただいてこの branch (fixed-background-not-drawn) に普通にコミットを重ねていただければ OK です。

$ git checkout fixed-background-not-drawn
~ 修正 ~
$ git commit -a
$ git push origin fixed-background-not-drawn

その後レビューさせていただきます!

@hidakatsuya

Copy link
Copy Markdown
Contributor

@mikkame

久々に確認したんですが、修正済みは線未指定のものも描画されてしまっていますね・・・

上記現象ですが、修正済みでない方 (example(v0.8.0).pdf) でも発生してますね。この PR が原因ではなく、 master でも起こるのかもしれません。確認してみます。

@hidakatsuya

Copy link
Copy Markdown
Contributor

@mikkame

久々に確認したんですが、修正済みは線未指定のものも描画されてしまっていますね・・・

この問題、master の方でも発生していました。その際、ご指摘いただいた背景が描画されない問題についても併せて修正しました。そのため、この PR は close します。PR はこちらです #18

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants