• Type: Improvement
• Resolution: Won't Fix
• Priority: Minor
• Component/s: warnings-ng-plugin
• Labels:

  • help-wanted
  • newbie-friendly
  • user-experience

[JENKINS-59719] Add zoom.js on the details view

Collapse comment: Saurabh Yadav added a comment - 2020-06-16 19:25

Saurabh Yadav added a comment - 2020-06-16 19:25

What is the exact issue faced here?

Expand comment: Saurabh Yadav added a comment - 2020-06-16 19:25

Saurabh Yadav added a comment - 2020-06-16 19:25 What is the exact issue faced here?

Collapse comment: Ulli Hafner added a comment - 2020-06-16 20:38

Ulli Hafner added a comment - 2020-06-16 20:38

Ah, I saw this small tool in a demo and thought that it might make sense to include it in order to zoom into the charts with a special event. But it makes more sense to close this issue.

Expand comment: Ulli Hafner added a comment - 2020-06-16 20:38

Ulli Hafner added a comment - 2020-06-16 20:38 Ah, I saw this small tool in a demo and thought that it might make sense to include it in order to zoom into the charts with a special event. But it makes more sense to close this issue.

Collapse comment: Saurabh Yadav added a comment - 2020-06-16 20:43

Saurabh Yadav added a comment - 2020-06-16 20:43

Ok.

I am new here. Can someone guide me how to add this tool, i can surely try.

Thanks!

Expand comment: Saurabh Yadav added a comment - 2020-06-16 20:43

Saurabh Yadav added a comment - 2020-06-16 20:43 Ok. I am new here. Can someone guide me how to add this tool, i can surely try. Thanks!

Collapse comment: Ulli Hafner added a comment - 2020-06-16 20:52

Ulli Hafner added a comment - 2020-06-16 20:52

If you are interested, yes why not. I think the library just needs to be copied to https://github.com/jenkinsci/warnings-ng-plugin/tree/master/plugin/src/main/webapp/js.

Then it needs to be imported in https://github.com/jenkinsci/warnings-ng-plugin/blob/master/plugin/src/main/resources/io/jenkins/plugins/analysis/core/model/IssuesDetail/filled.jelly.

In the last step we then need to have a look on where to register the zoom event.

Expand comment: Ulli Hafner added a comment - 2020-06-16 20:52

Ulli Hafner added a comment - 2020-06-16 20:52 If you are interested, yes why not. I think the library just needs to be copied to https://github.com/jenkinsci/warnings-ng-plugin/tree/master/plugin/src/main/webapp/js . Then it needs to be imported in https://github.com/jenkinsci/warnings-ng-plugin/blob/master/plugin/src/main/resources/io/jenkins/plugins/analysis/core/model/IssuesDetail/filled.jelly . In the last step we then need to have a look on where to register the zoom event.

Collapse comment: Saurabh Yadav added a comment - 2020-06-18 18:31

Saurabh Yadav added a comment - 2020-06-18 18:31

Looks like I may need more help here to fix this.

1) which library to be copied ? Is it from here https://github.com/hakimel/zoom.js.

2) Imported in ? Need to change the code for filled.jelly ? 

3) Where to register the zoom event ? Can you please explain a little more.

Sorry, if I am asking too much. I am very new to coding.

Thanks!

Expand comment: Saurabh Yadav added a comment - 2020-06-18 18:31

Saurabh Yadav added a comment - 2020-06-18 18:31 Looks like I may need more help here to fix this. 1) which library to be copied ? Is it from here https://github.com/hakimel/zoom.js . 2) Imported in ? Need to change the code for filled.jelly ?  3) Where to register the zoom event ? Can you please explain a little more. Sorry, if I am asking too much. I am very new to coding. Thanks!