Project

General

Profile

Feature #1362

プラグイン拡張のためのHTML属性の追加

Added by Akira Saito over 4 years ago. Updated over 4 years ago.

Status:
終了(Closed)
Priority:
通常(Normal)
Assignee:
Target version:
Start date:
01/07/2015
Due date:
% Done:

100%

Estimated time:

Description

システム開発でWork Timeを利用していますが、
チケットの名称が被ってしまい、識別のために詳細を見るといったことが、時々あります。

私はQuick Viewというプラグインを公開していますが、
このプラグインではチケットの詳細を(jQueryの)ダイアログ表示することができるので、
連携できると少し便利になります。

具体的には、cssのclassを各要素に追加したり、HTMLのカスタムデータ属性を追加したり(例えばdata-issueでチケット番号を識別可能にする)
して頂けると、連携しやすいと考えています。

よろしければパッチを見ていただけませんでしょうか?


Files

_user_day_table.html.erb.patch (662 Bytes) _user_day_table.html.erb.patch Akira Saito, 01/07/2015 10:49 PM

Associated revisions

Revision 378:d5921b94a879 (diff)
Added by Tomohisa Kusukawa over 4 years ago

fixes #1362 プラグイン拡張のためのHTML属性の追加

Revision 30ba283f (diff)
Added by Tomohisa Kusukawa over 4 years ago

fixes #1362 プラグイン拡張のためのHTML属性の追加

Revision 385:b37efd09dd08 (diff)
Added by Tomohisa Kusukawa over 4 years ago

fixes #1362 プラグイン拡張のためのHTML属性の追加

Revision 906fd6b1 (diff)
Added by Tomohisa Kusukawa over 4 years ago

fixes #1362 プラグイン拡張のためのHTML属性の追加

Revision 386:7ffa6369c961 (diff)
Added by akira saito over 4 years ago

fixes #1362 プラグイン拡張のためのHTML属性の追加

Revision 8d5066a3 (diff)
Added by akira saito over 4 years ago

fixes #1362 プラグイン拡張のためのHTML属性の追加

History

#1

Updated by Tomohisa Kusukawa over 4 years ago

ご連絡ありがとうございます。
軽微な変更で副作用もないと思いますので
採用させて頂きたいと思います。
ただ最近時間がなかなか取れず、少しお待ち頂くと思いますが、ご容赦頂けますでしょうか。

#2

Updated by Akira Saito over 4 years ago

時期は問題ありません。

よろしくお願い致します!

#3

Updated by Tomohisa Kusukawa over 4 years ago

  • Status changed from 新規(New) to 解決(Resolved)
  • % Done changed from 0 to 100

更新履歴 commit:d5921b94a879 で適用されました。

#4

Updated by Tomohisa Kusukawa over 4 years ago

  • Assignee set to Akira Saito

対応が遅く大変恐縮です。

commit:d5921b94a879 でパッチの内容を(手動で)適用しましたので、
確認して頂ければと思います。

ちょっと思ったのは、
① 'data-issue' のハイフンはアンダーバーにしてシンボル :data_issue にした方が良くないか
② <%= issue_id %> はダブルクォートで囲んだ方が良いのではないか
なのですが、如何でしょうか。

#5

Updated by Akira Saito over 4 years ago

ご対応ありがとうございます。返信が遅れてすみません。
ローカルで確認したいので、少し待ってください。

ご指摘については以下のように考えています。

① 'data-issue' のハイフンはアンダーバーにしてシンボル :data_issue にした方が良くないか

 HTML5で、data-*属性が独自属性として正式に利用できるようになり、
 DOMやjQueryもAPIを備えていますが、
 WorkTime Pluginが外のPluginと連携するのであれば、
 より標準的なI/Fを利用するほうが良いと思い、この形で提案させていただきました。
 こちらとしては、(URLをパースする以外の方法で)チケットのIDが取れさえすれば不都合はありません。

② <%= issue_id %> はダブルクォートで囲んだ方が良いのではないか

 こちらについてはその通りだと思います。

#6

Updated by Akira Saito over 4 years ago

実際に連携させてみました。(git://git.sourceforge.jp/gitroot/quickedit/quick_view.git)

試したところ、Daily Reportでチケット工数ありの場合に、
wt_done_ratioクラス属性が無く、連携が動作しませんでした。
元々のパッチになかったのが問題だったのですが、追加頂くことは出来ますか?

それと、'data-issue'と、:'data-issue'という記述が混在しているので、
統一すべきかなと思いました。

また、add ticketのsubmit時に生成されるチケット行にも、
クラス属性とdataxxx属性(data-にするかdata_にするかはお任せします)が欲しいです。
できればmonthly reportの方にも。。。
patchを用意したほうが良ければ、ご連絡ください。

#7

Updated by Tomohisa Kusukawa over 4 years ago

ご確認頂き、ありがとうございます。
# 対応が遅くて恐縮です。

もろもろあるので、具体的なコードで示して頂けると助かります。
できれば、パッチよりもPull Requestの方が助かるのですが、
(貢献も明確になりますし)ハードルが高い用であればパッチでも大丈夫です。

#8

Updated by Akira Saito over 4 years ago

それではPull Requestで試してみます。
Pull Requestは初めてなのと、別の作業もしているので、ちょっと遅れます。

#9

Updated by Tomohisa Kusukawa over 4 years ago

時間ができましたので対応してみます。
コミット後にまたご確認をお願い出来ますでしょうか。

#10

Updated by Akira Saito over 4 years ago

プルリクストを送りましたが、かぶってしまった上に、ミスっています。
取り消せないのでしょうか。。。

日毎工数表について、問題が解消されたことを確認しました。

2度目のプルリクエストには、月次レポートの修正要望も含めています。
採用いただければ嬉しいです。

#11

Updated by Tomohisa Kusukawa over 4 years ago

  • Target version set to 0.3.0

お願いしてたのに、かぶってしまい申し訳ありませんでした。

最後のプルリクエストを取り入れさせて頂きました。

#12

Updated by Tomohisa Kusukawa over 4 years ago

  • Status changed from 解決(Resolved) to 終了(Closed)

Version 0.3.0 に含めさせて頂きますね。

#13

Updated by Akira Saito over 4 years ago

お久しぶりです。

こちらもQuick View 0.1.0で、連携機能をリリースさせていただきました。
ご協力ありがとうございました。

Also available in: Atom PDF