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

wrap text within svg or hide text #9

Open
vincentvikram opened this issue Nov 7, 2015 · 5 comments
Open

wrap text within svg or hide text #9

vincentvikram opened this issue Nov 7, 2015 · 5 comments
Assignees
Labels
Milestone

Comments

@vincentvikram
Copy link
Owner

right now the text spills outside the svg making the diagram look unreadable and ugly. There seems to be a solution https://stackoverflow.com/questions/4991171/auto-line-wrapping-in-svg-text

@TechMky
Copy link
Contributor

TechMky commented Aug 15, 2018

I contacted the support team of Jointjs and came to know that the textWrap attribute is not supported in Jointjs v0.9.
Although the recent commit by @vincentvikram , 62cdd2f has solved the issue #20 , the overall structure of the project is still as per the version 0.9.

So now the workaround for this issue as of now is to use breakText attribute. This is again not a fully fledged solution and hence I need to know how to take this forward.
@vincentvikram sir, should I go on with breakText ?

@vincentvikram
Copy link
Owner Author

vincentvikram commented Aug 15, 2018 via email

@TechMky
Copy link
Contributor

TechMky commented Aug 15, 2018

Yes, I am thinking of something similar.

@TechMky
Copy link
Contributor

TechMky commented Aug 17, 2018

@vincentvikram there is a function called createNewItem() which is defined in the app-bundle.js which again comes from the gulp command. Now the issue is that the function is defined in the app-bundle.js itself (as I can't find that function anywhere else in the code). So, after running the gulp command, the changes made will get overwritten.
I propose to move that function definition into misc folder in a separate file.

P.S. Here is the outcome if we go with the breakText() method.
capture

@vincentvikram
Copy link
Owner Author

vincentvikram commented Aug 21, 2018 via email

@TechMky TechMky mentioned this issue Aug 21, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants