Project

General

Profile

Actions

Bug #73090

closed

pageIdSelector in RTE link popup does not show when enabled

Added by Franz Kugelmann about 8 years ago. Updated over 5 years ago.

Status:
Closed
Priority:
Must have
Assignee:
Category:
Link Handling, Site Handling & Routing
Start date:
2016-02-03
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
7
PHP Version:
Tags:
Complexity:
medium
Is Regression:
Yes
Sprint Focus:
Stabilization Sprint

Description

Enable the little field for adding the id of a page directly in RTE page link popup does not work currently in 7.6.2.
It is disabled by default since 7LTE, but enabling it in pageTSConfig has no effect.
How to enable in PageTSConfig:
RTE.default.buttons.link.pageIdSelector.enabled = 1

This does not work because only the configuration for pageIdSelector is added, but the key pageIdSelector itself is not added to the allowed linkAttributes.

Attached a change that results in the field being shown. Probably this is not the final solution but helps to understand the problem.
Additionally there is probably css missing or wrong HTML-Code, since the field is not added in line with other fields, see screenshot.


Files

pageIdSelector.png (19.2 KB) pageIdSelector.png Franz Kugelmann, 2016-02-03 09:58
rteLinkPopupPageIdSelector.diff (695 Bytes) rteLinkPopupPageIdSelector.diff Franz Kugelmann, 2016-02-03 09:58

Related issues 1 (1 open0 closed)

Related to TYPO3 Core - Bug #71298: Fix missing/broken RTE features after Link Handler API mergeAcceptedMarkus Klein2015-11-02

Actions
Actions #1

Updated by Markus Klein about 8 years ago

Thanks for the fix.

We do not handle patches here, but we use Gerrit (https://review.typo3.org) to handle change requests.
Please push your patch there. Feel free to ask for help here or preferably on Slack.

Actions #2

Updated by Markus Klein about 8 years ago

  • Category changed from Backend User Interface to Link Handling, Site Handling & Routing
  • Status changed from New to Accepted
  • Assignee set to Markus Klein
  • Priority changed from Should have to Must have
  • Complexity set to medium
  • Is Regression changed from No to Yes
Actions #3

Updated by Gerrit Code Review about 8 years ago

  • Status changed from Accepted 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 https://review.typo3.org/46460

Actions #4

Updated by Franz Kugelmann about 8 years ago

Allright :)

https://review.typo3.org/#/c/46460/
Thanks for caring!

Actions #5

Updated by Gerrit Code Review about 8 years ago

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

Actions #6

Updated by Markus Klein about 8 years ago

  • Target version changed from next-patchlevel to Candidate for patchlevel
Actions #7

Updated by Markus Klein about 8 years ago

  • Sprint Focus set to Stabilization Sprint
Actions #8

Updated by Gerrit Code Review about 8 years ago

Patch set 1 for branch TYPO3_7-6 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/46903

Actions #9

Updated by Franz Kugelmann about 8 years ago

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

Updated by Benni Mack over 5 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF