Closed (fixed)
Project:
Wysiwyg Linebreaks
Version:
8.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Issue tags:
Reporter:
Created:
14 Feb 2013 at 22:47 UTC
Updated:
4 Jan 2014 at 02:58 UTC
Jump to comment: Most recent
Comments
Comment #1
geerlingguy commentedOkay, working on stuff now. I've opened an 8.x branch, and have the module at least show up on the modules page: http://drupalcode.org/project/wysiwyg_linebreaks.git/commit/e78ad0b
I'm cleaning things up, and will hopefully be able to find a way to listen for the editor module's 'attach' event, and if it's called, and Wysiwyg linebreaks is installed and configured, our JS will be attached and run...
I might need to add a configure page where users can choose whether to force linebreaks or convert linebreaks. And is there a way I could allow setting this per-Wysiwyg editor? I guess I need to explore how editor.module is the same as/different than wysiwyg.
Comment #2
geerlingguy commentedAlso opened up follow-up issue for 7.x/6.x bugfixes: #2066419: Fix module name and missing variable declaration in linebreaks.js.
Comment #3
geerlingguy commentedIt works! See: http://drupalcode.org/project/wysiwyg_linebreaks.git/commit/417eacd
Next step is making the conversion from tags back to linebreaks (done when the editor is switched off (like when a node is saved) work correctly (so users can choose whether they want the 'force linebreaks' or 'convert linebreaks' functionality, like in D6/D7.
The usability/architecture is already a jillion times better, since we don't have to provide a button and some extra empty CSS files and such in D8. Nice!
Comment #4
geerlingguy commentedThe change record CKEditor module added: WYSIWYG in core! was key to completing the module port from D7 to D8, especially the section Provide additional CKEditor plugins.
With the third commit for this issue, I've implemented the
settingsForm()method ofCKEditorPluginConfigurableInterface, and am using the new configuration setting when the linebreaks.js file is loaded to determine whether to force linebreaks or convert them.Just need to do another once-over on the D8 branch, and make sure I have everything cleaned up, and then I'll be able to close this issue and release a beta (though it's working great for me and can be stable once D8 is stable). Yay!
Comment #5
geerlingguy commentedI just finished the rest of the work in porting the module (updated README.md and removed extraneous files)... but I can't seem to get my pushed commits and new tag (8.x-1.x-beta5) to appear here, even though git is reporting my branches are in sync.
Anyways, a couple more quick things I need to do now:
[Edit: Well, davereid found my missing commit by browsing git manually (see: http://drupalcode.org/project/wysiwyg_linebreaks.git/commit/fcaf3dd), but for some reason it seems the synchronization of the git repo to the project page was/is broken?]
Comment #6
geerlingguy commentedPosted follow-up issue in the infrastructure queue: #2066525: Pushed git commits and tags are delayed, unable to create release nodes. Hopefully that gets resolved soon!
Comment #7
geerlingguy commentedBecause of the problem reported in the previous comment, I had to tag a new release and add that release, so the first beta is beta2 :)
See release node: https://drupal.org/node/2067297
Yay!
Comment #8
wim leers#4: happy to see that the change record proved useful to you! :)
So, what was your overall impression of the API provided by
ckeditor.module? :)Comment #9
wim leersAlso, I'd appreciate it if you could chime in on #1933916-17: [meta] How do we want to facilitate enabling of CKEditor for sites upgraded from Drupal 7 to Drupal 8? :) Thanks for porting!
P.S.: don't hesitate to ping me if you have problems/questions with this module or another WRT
editor.moduleorckeditor.modulein Drupal 8!Comment #10
geerlingguy commented@Wim Leers: So, so SO much better than working with Wysiwyg plugins in D7. Wysiwyg was an amazing first step in D6/D7 in terms of making a modular Wysiwyg system for Drupal... but focusing on CKEditor (along with editor.module) looks like it's going to make the architecture and pluggability so much simpler.
On the flip side, I found that some of the documentation around editor.module (and especially it's responsibilities vs. ckeditor.module and any other editors that want to work with editor.module) was lacking/unclear. I hope I can help with that here and there, when I get time :)
Comment #11
wim leers#10: great! About
editor.moduledocs being lacking. First, I assume you did see/read the corresponding change notice at https://drupal.org/node/1911614. Second, where do you find them lacking? I can definitely inject a paragraph there to explain thateditor.moduleis a light glue module that provides basic infrastructure that any text (WYSIWYG) editor module would need (to attach/detach a text editor, but also integration with Drupal 8's new dialog system), whereas it is up to the modules using the infrastructure provided byeditor.moduleto allow for text (WYSIWYG) editor-specific to do things that are specific to them. Such asckeditor.moduleallowing for other modules to provide CKEditor plugins.Other than that, it might be wise to update that change notice with the changes introduced in #1894644: Unidirectional editor configuration -> filter settings syncing and #1886566: Make WYSIWYG editors available for in-place editing.
How does that sound?
Comment #12
geerlingguy commentedThat sounds good - I didn't even know https://drupal.org/node/1911614 existed, so that was helpful. (I probably should've searched for 'WYSIWYG' on the change notice page. Silly me!).
Comment #13
geerlingguy commentedUpdating tags and titles.