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

Textbox resize #3525

Closed
superczan opened this issue Dec 12, 2016 · 5 comments · Fixed by #4052
Closed

Textbox resize #3525

superczan opened this issue Dec 12, 2016 · 5 comments · Fixed by #4052
Milestone

Comments

@superczan
Copy link

Hi guys, I encountered a bug in 1.7.1. The issue is with textbox resize in group.

Steps to reproduce

Add multiple textbox on canvas. Select more than one of those textbox. Trying to resize the group creates problems.

Expected Behavior

The group should not resize below the longest word length.

Actual Behavior

Textbox when resized in groups are unpredictable. The group resize lets the resize to go beyond the smallest word length and even flip the textbox around. Sometimes when resizing the group, textbox in the group arbitrarily expands into infinite length.

@asturur
Copy link
Member

asturur commented Dec 12, 2016

is a known defect since textbox are out..i do not like the behaviour at all.

@superczan
Copy link
Author

Ok. Sorry if this is redundant then. Any chance of fix for this defect soon?

@kuldipem
Copy link
Contributor

kuldipem commented Jan 2, 2017

Problem occurs to me also on kitchensink,

  1. add a textbox
  2. add other objects
  3. select multiple objects
  4. just move the group
  5. click on blank canvas

@kuldipem
Copy link
Contributor

kuldipem commented Jan 2, 2017

I was investing about this issue and found something strange, I hope, it will help you

  • before moving in group , the textbox 's scaleX value was 0.5
  • after moving in group, the textbox 's scaleX value was 0.9999999

In developer console

var tb=canvas.getActiveObject();
tb.getScaleX();
0.5
var tb=canvas.getActiveObject();
tb.getScaleX();
0.9999999999999999

@asturur
Copy link
Member

asturur commented Jul 1, 2017

ok i finally changed this. Text can scale, just in 2.0 release, i m sorry, not going to backport this in the old 1.x branch

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants