Skip to content

[RFC] Fix subform multiple tinymce elements alternative#39609

Closed
dgrammatiko wants to merge 4 commits intojoomla:4.2-devfrom
dgrammatiko:4.2-dev-tinyButtons
Closed

[RFC] Fix subform multiple tinymce elements alternative#39609
dgrammatiko wants to merge 4 commits intojoomla:4.2-devfrom
dgrammatiko:4.2-dev-tinyButtons

Conversation

@dgrammatiko
Copy link
Copy Markdown
Contributor

@dgrammatiko dgrammatiko commented Jan 11, 2023

Pull Request is an alternative to #39449 which is for Issue #39416 .

Summary of Changes

  • The buttons PHP code has been reworked to pass data to JS
  • There is no server side rendering of the modals, it would happen on the fly, when a user requests a modal
  • There is some code that tries to replace the editor id both for click action and for the modals callback, etc
  • due to CSR all the ids should be always correct for any subform combination

Testing Instructions

Actual result BEFORE applying this Pull Request

Expected result AFTER applying this Pull Request

Screen.Recording.2023-01-11.at.22.25.17.mov

Link to documentations

Please select:

  • Documentation link for docs.joomla.org:

  • No documentation changes for docs.joomla.org needed

  • Pull Request link for manual.joomla.org:

  • No documentation changes for manual.joomla.org needed

@wilsonge @Fedik what do you think?

@joomla-cms-bot joomla-cms-bot added NPM Resource Changed This Pull Request can't be tested by Patchtester PR-4.2-dev RFC Request for Comment labels Jan 11, 2023
@dgrammatiko dgrammatiko force-pushed the 4.2-dev-tinyButtons branch 3 times, most recently from 5ff326a to 34ae6ad Compare January 11, 2023 22:45
@Fedik
Copy link
Copy Markdown
Member

Fedik commented Jan 15, 2023

hou hou hou 🎅
no, sorry, that to much hacky, better keep that bug alive for some time 😄

@dgrammatiko
Copy link
Copy Markdown
Contributor Author

I mean I can add some events to let devs fill the dialog and remove the eval parts (I didn't introduce them now, they existed).

Anyways I though I will give it a try

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

NPM Resource Changed This Pull Request can't be tested by Patchtester RFC Request for Comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants