Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

highlightプラグインをjQueryで再実装 #53

Merged
merged 5 commits into from
Jun 20, 2011
Merged

highlightプラグインをjQueryで再実装 #53

merged 5 commits into from
Jun 20, 2011

Conversation

machu
Copy link
Member

@machu machu commented Jun 18, 2011

issue #38 を実装しました。
日記のタイトルを$jQuery.titleに設定しています。

add_js_settingsを使ったRubyからJavaScriptへの文字列渡しにて、それぞれでエスケープすると大変なのでto_jsonメソッドが使えると嬉しいです(別チケットで提案します)。

@machu
Copy link
Member Author

machu commented Jun 19, 2011

すみません。チェックが甘かったです。
確かにtitle部分を変えるメリットが無くなったのであれば、シンプルにハイライトだけでもいいですね。
title変更部分を削除してみました。

@tdtds
Copy link
Member

tdtds commented Jun 20, 2011

良いと思います~。ChangeLogつけてcommitして下さい。

machu added a commit that referenced this pull request Jun 20, 2011
highlightプラグインをjQueryで再実装
@machu machu merged commit 5c27622 into tdiary:master Jun 20, 2011
@tdtds
Copy link
Member

tdtds commented Apr 17, 2012

tDiary-users-talk:0520でのリクエストを受けて、 db5197a にてtitleの変更を復活しました

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