power-media / prado3

Automatically exported from code.google.com/p/prado3
Other
0 stars 0 forks source link

THtmlArea Insert Image Bug #358

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
What steps will reproduce the problem?
1.Insert Image using THtmlArea.

What is the expected output? What do you see instead?
the image must appear.
I see the default 'no image' from IE.

What version of the product are you using? On what operating system?
PRADO 3.1.10
windows 7 32 bit

Please provide any additional information below.
Try to Insert Image From THtmlArea,
the first time you insert image, the image not appear,
click the image then right click insert/edit image, put image URL again, cilck 
update button, the image will be visible now.
the swf vedio will help to recognize the bug.

I test this bug on IE 7,8,9 and Firefox 5

Original issue reported on code.google.com by aabughaz...@gmail.com on 2 Aug 2011 at 11:31

Attachments:

GoogleCodeExporter commented 8 years ago
I rather doubt this has anything to do with Prado. It's more likely that - if 
at all - it's an issue with tinyMCE itself. However, since you provided no 
testcase code and no setup information (ie. for ex. what options you're passing 
to THtmlArea) nobody will be able to replicate your problem.

Btw. are you using absolute or relative urls? tinyMCE has its issues with 
relative urls. Therefore, you should always use absolute ones to avoid those 
pitfalls.

Also, there might be caching issues at work here. Try clearing your browser 
cache and re-test whether you're still experiencing the same problem.

Original comment by google...@pcforum.hu on 2 Aug 2011 at 8:55

GoogleCodeExporter commented 8 years ago
tiny_mce\tiny_mce\themes\advanced\js\image.js 
replace lines  93-96
93: ed.execCommand('mceInsertContent', false, '<img id="__mce_tmp" />', 
{skip_undo : 1});
94: ed.dom.setAttribs('__mce_tmp', args);
95: ed.dom.setAttrib('__mce_tmp', 'id', '');
96: ed.undoManager.add();

with:

tinymce.each(args, function(value, name) {
                if (value === "") {
                    delete args[name];
                }
            });

            ed.execCommand('mceInsertContent', false, tinyMCEPopup.editor.dom.createHTML('img', args), {skip_undo : 1});
            ed.undoManager.add();

Original comment by uaca...@gmail.com on 1 Dec 2011 at 5:26

GoogleCodeExporter commented 8 years ago
Closing since this is already fixed in trunk/ with r3065 (update of tinymce to 
version 3.4.7)

Original comment by ctrlal...@gmail.com on 2 Dec 2011 at 1:11

GoogleCodeExporter commented 8 years ago
[deleted comment]
GoogleCodeExporter commented 8 years ago
I change the file in "tiny_mce\tiny_mce\themes\advanced\js\image.js" and not 
work.

I search for another file with name "image.js" and I found one in 
"tiny_mce\plugins\advimage\js".
So, at line 171 in this file I change the code too.

Now it's work fine.

Thank you.

Original comment by aabughaz...@gmail.com on 5 Dec 2011 at 7:49