Project

General

Profile

Actions

Bug #66854

closed

RTE Link Editor: unfolding pagetree element does not jump to anchor of the choosen element

Added by form4 GmbH & Co. KG over 9 years ago. Updated about 6 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
RTE (rtehtmlarea + ckeditor)
Target version:
-
Start date:
2015-05-08
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
6.2
PHP Version:
Tags:
Complexity:
no-brainer
Is Regression:
No
Sprint Focus:

Description

If you want to add a link in the RTE and you have a big pagetree.
By unfolding one tree element it reloads the frame. After reloading it should junp to the opened tree element.
But this does not happen.

The name attribute is set with 2 double quotation marks (name=""anchor""), but the anchor goes to the '#anchor'.
See screenshot.

Filename: /typo3/class.browse_links.php
both Funktions: PM_ATagWrap() -> this is existing 2 times
wrong: #$name = ' name="' . $bMark . '"';
right: $name = ' name=' . $bMark;

Greetings
Kerstin


Files

screenshot_firebug.PNG (6.74 KB) screenshot_firebug.PNG form4 GmbH & Co. KG, 2015-05-08 13:19
Actions #1

Updated by Mathias Schreiber over 9 years ago

  • Complexity set to no-brainer
Actions #2

Updated by Gerrit Code Review over 9 years ago

  • Status changed from New to Under Review

Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/39363

Actions #3

Updated by Gerrit Code Review over 9 years ago

Patch set 1 for branch TYPO3_6-2 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/39364

Actions #4

Updated by form4 GmbH & Co. KG over 9 years ago

Hi,
thanks for the fix patch.
I just checked this with my older TYPO3 Version 6.2.9, but the patch for branch TYPO3_6-2 works fine.

I would like to mention, that the file contains two classes and both classes have the method PM_ATagWrap() implemented in the same way.
I'm not sure if the other occurence should be patched too.

Do you think, this will be in the next Version 6.2.13?

Greetings
Kerstin

Actions #5

Updated by Andreas Fernandez over 9 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100
Actions #6

Updated by Benni Mack about 6 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF