Change icons to svg #69
Conversation
| @@ -17,17 +17,17 @@ def normal_mode_input_fields(setting, lang) | |||
| content += content_tag(:p) do | |||
| content_tag(:label, k) + | |||
| text_field_tag("settings[custom_messages][#{k}]", v.to_s) + | |||
| link_to_function('', '$(this).closest("p").remove();', class: 'icon icon-del clear-key-link') | |||
| link_to_function(sprite_icon('del'), '$(this).closest("p").remove()', class: 'icon icon-del clear-key-link') | |||
There was a problem hiding this comment.
本題とはそれますが、
link_to_functionを通すと$(this).closest("p").remove();; return false;のようにセミコロンが自動で入るようだったのでセミコロンなしに直しています。
| @@ -1,6 +1,6 @@ | |||
| <div> | |||
| <%= select_tag 'select-key', available_message_options(@setting, lang), id: 'key-selector' %> | |||
| <span class='icon icon-help'><%= l(:text_description_of_search_box) %></span> | |||
| <span class='icon icon-help'><%= sprite_icon('help', l(:text_description_of_search_box)) %></span> | |||
There was a problem hiding this comment.
icon-help は必要ないと思います。
現状では icon-help は、古いPNG画像にフォールバックするためのものにすぎません。
https://github.com/redmica/redmica/blob/7eb368c8fb93cac296321ff1b44890237271352f/app/assets/stylesheets/application.css#L1844
There was a problem hiding this comment.
全体的に既存のアイコンを表示していたクラスはそのままにしていましたが、よく考えたらRedmine6より前との互換性はどのみちないのでクラスを維持しても意味が無いですね
他のクラスも合わせて修正します
There was a problem hiding this comment.
.iconにはスタイルが適用されているため.iconのみ残します
|
@hidakatsuya |
| caption: :label_custom_messages, html: { class: 'icon icon-edit' } | ||
| requires_redmine version_or_higher: '3.2' | ||
| caption: :label_custom_messages, html: { class: 'icon' }, icon: 'edit' | ||
| requires_redmine version_or_higher: '6.0' |
変更漏れでした 🙇 |
This pull request will change all icons in the plugin to support Redmine 6 or later svg icons.
Redmine version supported by this plugin is 6.0