-
Notifications
You must be signed in to change notification settings - Fork 14
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Implemented dynamic editing UI for admin page with slidepanel #57
base: master
Are you sure you want to change the base?
Conversation
$xoopsTpl->assign('scrollx', intval($_POST['scrollx'])); | ||
$accordion_active = (isset($_POST['accordion_active']) AND $_POST['accordion_active'] !== "" AND $_POST['accordion_active'] !== "false") ? intval($_POST['accordion_active']) : "false"; | ||
$xoopsTpl->assign('accordion_active', $accordion_active); | ||
$xoopsTpl->display("db:admin/ui.html"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I believe this is where it dynamically generates the markup from php.
Then within the markup it directs the save.php at formulize/ instead of /admin
@@ -182,7 +182,7 @@ | |||
|
|||
function sendFormData(thisformdata, ele_id) { | |||
if(!ele_id) { ele_id = 0 } | |||
$.post("save.php?ele_id="+ele_id, $(thisformdata).serialize(), function(data) { | |||
$.post("<{$xoops_url}>/save.php?ele_id="+ele_id, $(thisformdata).serialize(), function(data) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
After changing this line, when re-testing on browser, it continues to generate the following page source:
function sendFormData(thisformdata, ele_id) {
if(!ele_id) { ele_id = 0 }
-already cleared all cache, but still persists.
} else { | ||
$col_one = "<p id=\"subform-caption-f$fid-sf$subform_id\" class=\"subform-caption\"><b>" . trans($customCaption) . "</b></p>"; // <p style=\"font-weight: normal;\">" . _formulize_ADD_HELP; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The id attribute was recently added here, and it shouldn't be removed.
The tree control looks good! |
Conflicts: modules/formulize/include/formdisplay.php modules/formulize/templates/css/style.php
Currently investigating on the save function in the slidepanel.
When the slidepanel successfully renders, the "save your changes" button changes it's direction instead of admin/save.php, it points to formulize/save.php
Couple notes:
*** As soon as the major issues are dealt with, I will re-organize the plugin paths again