Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
Patch ported from Drupal 7: #1295176: [D7] Notice: Undefined index: default in ckeditor_admin_profile_form() - when editing CKeditor profile
Comment | File | Size | Author |
---|---|---|---|
#3 | Screenshot.png | 36.55 KB | AlexisWilke |
Comments
Comment #1
dczepierga CreditAttribution: dczepierga commentedChanges commited to GIT.
Greetings
Comment #2
dczepierga CreditAttribution: dczepierga commentedComment #3
AlexisWilke CreditAttribution: AlexisWilke commentedWhat was fixed here?!
I have two problems with the latest addition... (i.e. the editing of the toolbar using JavaScript)
1. It doesn't work at all (see attached image)
2. It tries to load jquery.ui scripts (you MUST use jquery_ui or you'll break many sites like mine!)
So at this time I cannot edit the bar... and saving (which I did not try) will probably break the existing bar.
Comment #4
AlexisWilke CreditAttribution: AlexisWilke commentedThere is a start, with these fixes I can see the buttons and click on them as expected.
However, I still cannot define a bar (i.e. the objects show as if they were being moved but
I have no clue what the jquery-1.4.4.min.js is for and as far as I know the jquery-ui.js is automatically loaded by the jquery scripts.
I would move the sort.js one directory up because it's not part of the jQuery environment (yes, it uses it, but no, it's not a jQuery script, it's for the CKEditor.)
Thank you.
Alexis
Comment #5
AlexisWilke CreditAttribution: AlexisWilke commentedFor others who also have a problem with editing using the new JavaScript toolbar, you will notice that the textarea we had for a little while with the JavaScript arrays is gone. This is actually just hidden with the CSS code.
You can show the textarea called edit-toolbar back by commenting out the edit-toolbar display: none instructions as follow.
File: ckeditor/ckeditor.css
Comment #6
dczepierga CreditAttribution: dczepierga commented@AlexisWilke, if u found new bug pls open new issue - if u read related issue in first post u will know what was fixed.
I moved your bug to this issue: #1311926: [D6] Add jquery_ui support to the Drag & Drop toolbar configuration.
First of all, we must load newer jquery because a lot of people use Jquery which is build into Drupal - it's too old to work well with this what we need here.
Also we cannot only use
jquery_ui_add()
function, because not all users have jquery_ui project.So in summary i add support for jquery_ui module and if CKEditor module detect it everything is loaded as u write. Pls update to last DEV and check that all works as u expect.
Really thx for help with that.
Greetings
Comment #7
mkesicki CreditAttribution: mkesicki commented