Few functions and fixes to the gantt

[WurstMitBrot]

Hello,

i added a few functions i needed for a project.

• I made dependencies editable. You now can create new dependencies using the pop up and delete them by clicking on the arrows. These actions also fire events
• Editing the tasks and dependencies is optional
• The months in the pop up are now localized
• The pop up don't overlay the bars anymore
• Added german localization
• changed mouse events while draging bars from offsetX to clientX

[core44]

Think this line in frappe-gantt.js should be outside the else condition. It's currently breaking custom_popup_html because they don't get shown.

        	// SJ fix popup overlaying bars
        	this.parent.style.display = 'block';
        }

[Anurag810]

Please resolve the conflicts and make the changes requested by @core44

[TimVosch]

This PR fixed an issue where if I dragged a task to the right (future) it would actually save it's date a few months in the past.

I assume this has to do with changed mouse events while draging bars from offsetX to clientX which @WurstMitBrot mentioned.

(I have currently hacked this in my ERPNext server)

[parsimonyERP]

+1 Please merge as soon as possible.

[WurstMitBrot]

Hi @netchampfaris,

I'm sorry i just saw your suggestions today. Thank you for your feedback. I changed everything as desired.

[ofebles]

merged already?

[WurstMitBrot]

@ofebles still waiting for a reaction. Fixed all the suggestions and never heared something.

[ofebles]

@ofebles still waiting for a reaction. Fixed all the suggestions and never heared something.

Hi @WurstMitBrot , if i get your branch and overwrite in node modules my frappe-gantt folder , should this works? Its urgent for me. Any help? I got this issue related with all tasks are showing in the same init/end dates range. Im using it in erpnext v12.

[wpleasant]

Why has this issue been unresolved for two years? I've sold my team on moving our operation to erpNext and the very first item, the Gantt Chart, the office manager tries to use is broken, and has been broken for two years.

#frappe/erpnext#19348

[WurstMitBrot]

@netchampfaris and @Anurag810 do i have to fix something?

[sahil48]

@netchampfaris and @Anurag810

Is there a reason why no one is reviewing it, or clarifying what changes are needed for this to get added?

[WurstMitBrot]

@netchampfaris and @Anurag810 fixed merge conflicts. Is there any chance this will be added?