-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
image and video resize doesn't seem to be working #470
Comments
Yea, it's not working for me either, I'll fix this now I'm back from holiday! |
Happy holidays! Have fun:) |
Ok, should be fixed in new pre-release, can you confirm please? |
I found the same issue. Also, when Is that incorrect or am installing the pre-release wrong? |
@mgruesbeck You need @SimeonC , I have not tested this one yet since I need to merge my fork with it first. I will get to this tonight!! |
Thanks @episodeyang. Alright, I updated to pre-release. All textAngular files are loading with no errors but I'm still having resizing issues. If I click the image and drag the corner it will pass: However, if you click the image and use the percentages to resize no percent width are passed. For example if you select 100% on an image the output will be: I'll wait for @episodeyang's results. |
@mgruesbeck The percentages are aimed at a percentage of the original size of the uploaded image. So 100% is a 1:1 ratio of the image, I copied the logic off of summernote.org. |
The resize handle works well in firefox. However, the image popup does not show. In Chrome, the handle changes all of the images in the editor proportionally simultaneously. The popup shows properly though. I haven't tested it in other browsers though. |
Thanks guys. From what @episodeyang is saying it sounds like my issue is different. For example, when clicking 50% I would expect textAngular to pass: However, in my case, no percentage is passed and the output is this: I'll update you if I can figure it out. |
@mgruesbeck Ugh, that was my bad actually - I just checked and what you said is what should be happening and is what happens on my end when I actually try it. Possibly you have angular-sanitize instead of textAngular-sanitize loaded? That would strip out the width regardless. @episodeyang I see that now, looks like something weird is going on there... |
OK, try pre-release 21, that should have it fixed. |
@SimeonC , it works! There is just one problem though, it seems rangy crashes a lot. When it crashes, the whole app turn into a text view. What I have are two svg images that are 900kb each. Does anyone else experience crash? |
What's the error? I haven't had that problem yet - also which rangy version? |
Weird, I'll try look into this and see if I can find what's causing it, if I can replicate it that is! |
When you say it only occurs when a user uploads an I'm having trouble replicating this and might need some help with some extra steps please! |
yes! so in chrome, if I click on this svg a lot, it crashes the page. this however does not occur with the current demo on the website. On Tue, Jan 13, 2015 at 3:56 PM, Simeon Cheeseman [email protected]
|
Hi @SimeonC,
I just want to check, is the image resizing working? on my pre19, both the image resizing of video and image are not working.
Thanks,
Ge
The text was updated successfully, but these errors were encountered: