vim-jp.github.io
vim-jp.github.io copied to clipboard
vim-jp
Patchesの表示が冗長
ページの右側に表示されるPatchesの表示が冗長に思います。
この例だと、patch 8.0.0454: compiler warnings for "always true" comparison の部分は不要だと思います。また、" は " として表示されるべきでしょう。
--- a/assets/javascripts/patch-list.js
+++ b/assets/javascripts/patch-list.js
@@ -5,7 +5,7 @@ $(function() {
$.each(items, function() {
$('<li/>').append(
$('<a/>').attr({'href': this.link, 'target': '_blank'}).text("Patch " + this.title)
- ).append($('<br/>')).append($('<span/>').text(this.description)).appendTo('#patches-list');
+ ).append($('<br/>')).append($('<span/>').text(this.description.replace(/^[^\n]*\n\n/, '').replace(/"/g, '"'))).appendTo('#patches-list');
});
});
});
とりあえず、こんなパッチを当てれば表示は改善されますが、これでいいのかは分かりません。
また、" だけではなく、<, >, & なども対応が必要かもしれません。
' もデコードが必要でした。
あと、Solution:の前に改行を入れた方が見やすそう。
--- a/assets/javascripts/patch-list.js
+++ b/assets/javascripts/patch-list.js
@@ -3,9 +3,10 @@ $(function() {
$.getJSON("//vim-jp.herokuapp.com/patches/json?callback=?", function(items) {
$.each(items, function() {
+ var desc = $('<div/>').html(this.description).text().replace(/^[^\n]*\n\n/, '').replace(/\n(?=Solution:)/, '<br/>')
$('<li/>').append(
$('<a/>').attr({'href': this.link, 'target': '_blank'}).text("Patch " + this.title)
- ).append($('<br/>')).append($('<span/>').text(this.description)).appendTo('#patches-list');
+ ).append($('<br/>')).append($('<span/>').html(desc)).appendTo('#patches-list');
});
});
});
修正をコミットしました。 http://vim-jp.herokuapp.com/patches/json の仕様がどうなっているのかは知りませんが、意図通り動いているように見えます。
twitterのvim_jpアカウントのツイートも改善されるとうれしいな
https://twitter.com/vim_jp/status/842492142790606848
8.0.0470: patch 8.0.0470:
