Bug #18549
closedProblems using link wizard in content elements
0%
Description
Users were used to add a link to e. g. the header of a content element
(Bild 1.png).
If you click on the link wizard icon a new window opens and shows the
link wizard (Bild 2.png). The tab "External URL" is active. The field
URL is pre-filled with the host address of the host TYPO3 is running on.
Write a different URL into the field URL now (Bild 3.png) and click "Set
Link".
Back in the content element (Bild 4.png) you will see that the field
"Link" indicates that it was changed, but the field is empty.
I think this is a bit strange.
Tested with 4_2 branch on Firefox 2.0.0.13 on Mac OS X 10.5.
Branch 4_1 also does not work correctly.
(issue imported from #M8001)
Files
Updated by Andreas Otto † over 16 years ago
Using the link wizard which is provided by the htmlarea rte causes no problems.
But htmlarea rte is using a different class for the link wizard afaik.
Updated by Steffen Kamper over 16 years ago
the link wizard in general is broken - same happens in Image option with link wizard
Updated by Steffen Kamper over 16 years ago
it's broken in 4.2 initial version as well
Updated by Ralf Hettinger over 16 years ago
I've created a patch for 4.2 that should solve the problems of #18549 (this one... which is a due to namespace congestion of JS functions) and #18549 (which is reconnection of link type not properly implemented for the wizard). Furthermore it completes #18549 (new link type folder wasn't implemented for the link wizard).
Will submit it to core list in a minute...
... at the moment, I haven't got the time to backport it to 4.1 - maybe that'S a nice task for so else? ;)
Updated by Steffen Kamper over 16 years ago
I did some improvements in 8001_v2.diff
Updated by Steffen Kamper over 16 years ago
ther was a typo and ad a small issue with empty link params, please use 8001_v3.diff
Updated by Ralf Hettinger over 16 years ago
added 8001_v4.patch, which
- solves #14135 again,
- handles title, class and target for all types,
- reimplements the update button
Updated by Ralf Hettinger over 16 years ago
thanks to Steffen Kamper for testing.
Added in 8001_v5.patch:
- current link is saved if changing the tab
- cosmetic correction of email address field
Updated by Ralf Hettinger over 16 years ago
8001_v6.patch is just a slight bugfix of _v5
Updated by Ralf Hettinger over 16 years ago
8001_w-folder-select.patch now
- handles content of the tab 'folder' better (i.e. lists folders)
- has current tab set to 'page' and no current link set, if no link set yet.
Updated by Steffen Kamper over 16 years ago
... and should be the last one, as all is fixed now - thx Ralf!
Updated by Andreas Otto † over 16 years ago
Well, I don't think so.
I have just applied the patch to Revision 3552 on branch 4.2 and tested on Safari.
If no link is set, the the wizard starts in the page tab.
Selecting another tab is not possible.
If I enter a URL in the link field of the record and save the record the wizard will use the tab external link.
Again, selecting another tab is not possible.
Updated by Steffen Kamper over 16 years ago
i confirm that on safari only (tested the windows version)
But i don't understand the behavior, as the JS-Part here is not accessed by Safari (jumpToUrl(...) )
is there a possibiolity to debug the js to see whats going wrong?
Does it work in RTE ?
Updated by Andreas Otto † over 16 years ago
Don't know how to use the Developer tools in Safari.
The link wizard in RTE is working fine.
Updated by Ralf Hettinger over 16 years ago
Guys,
this one was admittedly a bit hard to track down (Safari JS engine chokes on reserved word class used as variable)...
... please use 8001-safari-fixed.patch
Updated by Ralf Hettinger over 16 years ago
Updated by Ralf Hettinger over 16 years ago
8001-cleaned-n-updatebutton.patch
- is rather clean according the CGL (thanks to Ingo Renner for taking care of this)
- is more consistent on when showing the "update" button (thanks Andreas Otto for the hint)
Updated by Ralf Hettinger over 16 years ago
8001-ie6-fixed.patch
- bugfix: unselectable target in IE 6 due to multiple usage of same identifier in HTML source