The attached patch will update a 6.x-1.x checkout to Drupal 7. The patch also includes some coding style fixes.
I recommend the http://drupal.org/project/ajax_markup module, it makes the preview button work with Markdown.
There are some remaining issues that I know of. Mainly CSS for popups when using the default admin theme "Seven".
| Comment | File | Size | Author |
|---|---|---|---|
| #10 | markdowneditor_drupal_7_1144534_7.patch | 44.46 KB | frjo |
| #10 | teaserbr.png | 210 bytes | frjo |
| #6 | markdowneditor_drupal_7_1144534_5.patch | 44.34 KB | frjo |
| #4 | markdowneditor_drupal_7_1144534_4.patch | 44.15 KB | frjo |
| markdowneditor_drupal_7.patch | 43.82 KB | frjo |
Comments
Comment #1
akamaus commentedWorks fine for me. The only problem if what 6.x-1.x-dev seems already includes code markup fixes, so many hunks on css and js files failed.
What's about making a 7.x branch?
Comment #2
frjo commentedI just retested the patch and it applies cleanly for me on a 6.x-1.x checkout.
Comment #3
Jean-Philippe Fleury commentedSubscribing
Comment #4
frjo commentedUpdated to fix a number of PHP notices.
Comment #5
frjo commented[double post]
Comment #6
frjo commentedAdded:
* a teser break button
* the "edit-menu-description" field to excludes
Comment #7
solipsist commentedAwesome! I'm opening a 7 branch so we can get all this packaged and available for everyone.
Comment #8
frjo commentedGreat to here!
CSS fixes for popups when using the default admin theme "Seven" is needed, they don't look so nice now. But other than that I believe the code works well.
Comment #9
solipsist commentedFinally getting around to this. The patch doesn't apply cleanly for some reason. What commit have you been working from?
Comment #10
frjo commentedMy bad, forgot to tell my editor not to strip whitespace at end of lines.
This version should apply cleanly to 6.x-1.x.
Also attached "teaserbr.png" from BUEditor, it's a handy button to have.
Comment #11
fenda commentedFriendly bump to this issue. Any progress on releasing this?
Comment #12
fenda commented@sollipist Did the patch appear cleanly for you?
Comment #13
simon_w108 commentedis the 7 branch available? where can I find it? thanks!
Comment #14
solipsist commentedIt applied cleanly and I've created a D7 development release. Please try it out.
Comment #15
dhalbert commentedThanks! Could you publish the 7.x release? It's not in the list of releases. (I'll use git if you don't want to.)
Comment #16
solipsist commentedWait a while. I don't think the packager has had enough time to generate the tarball.
Comment #17
Robin Millette commentedThere's now a dev release at least.
Comment #19
frjo commentedThere is now a stable 1.0 release and I have added new stuff to 7-dev that will become a 1.1 release, please test and report in issue #1834098: Please test 7.x-1.x so we can get a nice 7.x-1.1 release soon.