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

I find a Bug, createProgressBar: function (days, cls, desc, label, dataObj) miss a parameter day.id, #55

Open
lingpeiyong opened this issue May 9, 2016 · 2 comments

Comments

@lingpeiyong
Copy link

createProgressBar: function (days, cls, desc, label, dataObj) miss a parameter day.id,after the days;
I think this function is right, invoke this function give more parameter is the true bug occurs , because day.id is not use in this function.you maybe forget to remove day.id at the code of invoke this function

@jmgomez77
Copy link

That's right, I faced the same problem. I have fixed it by adding the parameter myself in the function implementation.

@jmgomez77
Copy link

`### Eclipse Workspace Patch 1.0
#P IdeaWeb
Index: src/main/webapp/resources/gantt/js/jquery.fn.gantt.js
===================================================================
--- src/main/webapp/resources/gantt/js/jquery.fn.gantt.js   (revision 1)
+++ src/main/webapp/resources/gantt/js/jquery.fn.gantt.js   (working copy)
@@ -940,7 +940,7 @@
             // **Progress Bar**
             // Return an element representing a progress of position within
             // the entire chart
-            createProgressBar: function (days, cls, desc, label, dataObj) {
+            createProgressBar: function (days, id, cls, desc, label, dataObj) {
                 var cellWidth = tools.getCellSize();
                 var barMarg = tools.getProgressBarMargin() || 0;
                 var bar = $('<div class="bar"><div class="fn-label">' + label + '</div></div>')
`

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

No branches or pull requests

2 participants