support Rouge #684

Merged
merged 10 commits into from Dec 30, 2016

Projects

None yet

2 participants

@takahashim
Collaborator
takahashim commented Nov 23, 2016 edited

cf. #485

  • fix stylesheet to support Rouge
  • fix tags of codes
@takahashim
Collaborator

config.ymlでpygmentsの代わりにrougeを指定して使います。

  • inline styleとclass設定(別途CSSで指定)のどっちがいいでしょうか?
  • 生成するタグについて、preだけではなくcodeを追加するべきという話も
@takahashim
Collaborator
takahashim commented Dec 10, 2016 edited

やっぱりinline styleは融通が利かない(配色を変えたりできない)のでclass設定方式(CSSで変更可)にしてみます

@takahashim takahashim added this to the 2.2.0 milestone Dec 10, 2016
@kmuto
Owner
kmuto commented Dec 11, 2016

あ、返答しそこねてたけどclassのほうがよいと思います。

@takahashim takahashim changed the title from support Rouge to [WIP] support Rouge Dec 24, 2016
@takahashim
Collaborator

(タグが壊れてたので要修正です…)

takahashim added some commits Nov 23, 2016
@takahashim takahashim support Rouge f51b8ce
@takahashim takahashim fix css
use classes instead of inline style attributes
f63c2e9
@takahashim takahashim fix sample config.yml c91529f
@takahashim takahashim support rouge in CSS 1edd3ba
@takahashim takahashim fix rouge 1848c66
@takahashim takahashim Merge branch 'master' into html-rouge
9bb924a
@takahashim takahashim fix styles
743ccd2
@takahashim takahashim changed the title from [WIP] support Rouge to support Rouge Dec 30, 2016
@takahashim
Collaborator

Rougeで行番号を追加させるには<table>にするしかできないようだったので、そのようにしています。

takahashim added some commits Dec 30, 2016
@takahashim takahashim remove comments and useless spaces a028bb9
@takahashim takahashim fix css in sample-book
e74eadb
@takahashim takahashim support //firstlinenum
cf3ad10
@takahashim
Collaborator

//firstlinenumにも対応したので、ここでマージしますね。

@takahashim takahashim merged commit 992c9b9 into master Dec 30, 2016

2 checks passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@takahashim takahashim deleted the html-rouge branch Dec 30, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment