When switching profiles, profile configuration could be lost, so it would be nice to require confirmation.

Also, the "Remove" link seems to be redundant? It's the same as switching an input format to "none"?

CommentFileSizeAuthor
#4 wysiwyg-HEAD.editor-profile.patch1.4 KBsun
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

sun’s picture

Hm. Yes. I also thought about this a few times, but unfortunately, didn't come to a conclusion. Another possible approach would be to store the configuration per editor, for each input format. So if you change the editor to editorB, the configuration for editorA is kept. If you select editorA for this input format again, the original values are back again.

But then again, another consideration was that an admin might want to use the editor configuration of one input format in another input format upon switching the editor. I think that's the most common case: You set up a profile for "Filtered HTML", and later decide that the default input format should not have a full-blown editor. So you switch the editor to another editor, reconfigure the profile for "Filtered HTML", and ideally, you would like to assign the previous profile to "Full HTML" or whatever input format.

The latter would require that an admin is able to setup editor profiles first - regardless of input formats and associations. Afterwards, profiles can be assigned to and unassigned from input formats.

markus_petrux’s picture

You could for example, form_alter the input format page to offer the possibility to attach profile editors, maybe a select list element ¿?

That way, you can create as many editor profiles as you wish, and the association to input formats is made in the input format page itself?

sun’s picture

Title: Add confirmation page when switching editors associated to input formats » Previous profile configuration lost when assigning different editor

Well, yes, but that's slightly offtopic. Forms can be displayed, submitted, and processed anywhere in Drupal, so the location does not matter for this issue.

sun’s picture

Status: Active » Fixed
FileSize
1.4 KB

Thanks for reporting! Committed attached patch to all branches.

A new development snapshot will be available within the next 12 hours. This improvement will be available in the next official release.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.