cuny-academic-commons / cbox-theme

Default theme for Commons In A Box
GNU General Public License v2.0
20 stars 15 forks source link

Issue with Theme options #256

Open jmerandy opened 7 years ago

jmerandy commented 7 years ago

I am getting a perpetual spinning wheel in the theme options when I attempt to update the logo. Just updated the theme (1.0.15) and Wordpress is up to date (4.7.3–en).

screen shot 2017-04-11 at 3 41 37 pm
boonebgorges commented 7 years ago

Thanks for the report. Can you open your browser's JS console and report any errors/warnings you see there?

On 04/11/2017 03:27 PM, jmerandy wrote:

I am getting a perpetual spinning wheel in the theme options when I attempt to update the logo. Just updated the theme and Wordpress is up to date.

screen shot 2017-04-11 at 3 41 37 pm https://cloud.githubusercontent.com/assets/27307673/24929236/48e01c9e-1ed3-11e7-8732-d0df800cbf23.png

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/cuny-academic-commons/cbox-theme/issues/256, or mute the thread https://github.com/notifications/unsubscribe-auth/AAPDYxPZLknaCQzO1k9XJedq9ypqszEKks5ru-IpgaJpZM4M6m1S.

jmerandy commented 7 years ago
screen shot 2017-04-13 at 10 08 35 am
r-a-y commented 7 years ago

Hi Jesse,

Can you try adding the following to wp-config.php?

define( 'INFINITY_DEV_MODE', true );

Then reload a page (any page will do) and remove the code above. Then see if your CBOX Theme Options work as expected again. If it does, remove the line.

If that doesn't work, what plugins are you using? There is a chance that one of your plugins is registering some JS files that conflict with the CBOX Theme Options page.

jmerandy commented 7 years ago

Here are the plugins I am using that are network activated:

Akismet Anti-Spam BP Groupblog BP MPO Activity Filter Commons in a Box Cookies for Comments Draw Attention Pro Image Widget Multisite Enhancements Unconfirmed User switching Wordpress MU Domain Mapping

I tried deactivating each, one at a time and then reloading the theme editor page to no avail.

I also added the line of code you share and could not get the theme editor to function properly

On Thu, Apr 13, 2017 at 12:15 PM, r-a-y notifications@github.com wrote:

Hi Jesse,

Can you try adding the following to wp-config.php?

define( 'INFINITY_DEV_MODE', true );

Then reload a page (any page will do) and remove the code above. Then see if your CBOX Theme Options work as expected again.

If that doesn't work, what plugins are you using? There is a chance that one of your plugins is registering some JS files that conflict with the CBOX theme options page.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/cuny-academic-commons/cbox-theme/issues/256#issuecomment-293944150, or mute the thread https://github.com/notifications/unsubscribe-auth/AaCuma8GUFqAJB0NGDqg-rxP2FeCl88Nks5rvkoRgaJpZM4M6m1S .

--

Jesse Merandy Director of the Digital Media Lab

Bard Graduate Center: Decorative Arts, Design History, Material Culture 38 West 86th Street, New York, NY 10024 T 212 501 3061 | F 212 501 3065 | E jesse.merandy@bgc.bard.edu jesse.merandy@bgc.bard.edu

jmerandy commented 7 years ago

It looks like the issue is actually only with the image upload functionality built into the theme designer. Anywhere where this is an option there is no ability to upload or adjust the images set. The logo is just the one place where I was trying to alter the image.

r-a-y commented 7 years ago

I just gave the uploader a quick test and it works.

Check to see if the /wp-content/uploads/exports/cbox-theme/dynamic-admin.js file is generated.

If not, try adding back the INFINITY_DEV_MODE line in wp-config.php, load a few pages and check if the /wp-content/uploads/exports/cbox-theme/dynamic-admin.js file is regenerated.

If the file exists, verify that the icextOptionUploader error is gone by opening up the file and checking to see if icextOptionUploader exists in the JS.

If it does exist, go back to the CBOX Theme Options page and everything should work.

jmerandy commented 7 years ago

Sorry I am so late on following up on this. I had some major projects that sidetracked me. I am hoping we can pick up the thread. I am seeing the file /wp-content/uploads/exports/cbox-theme/dynamic-admin.js where it is supposed to be. I am also doing a search in the dynamic-admin.js file and am not seeing the icextOptionUploader clause. Any thoughts on how to proceed?

On Thu, Apr 13, 2017 at 3:49 PM, r-a-y notifications@github.com wrote:

I just gave the uploader a quick test and it works.

Check to see if the /wp-content/uploads/exports/ cbox-theme/dynamic-admin.js file is generated.

If not, try adding back the INFINITY_DEV_MODE line in wp-config.php, load a few pages and check if the /wp-content/uploads/exports/ cbox-theme/dynamic-admin.js file is regenerated.

To verify that the icextOptionUploader error is gone, open up the JS file and see if icextOptionUploader exists in the file.

If it is, go back to the CBOX Theme Options page and everything should work.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/cuny-academic-commons/cbox-theme/issues/256#issuecomment-294004175, or mute the thread https://github.com/notifications/unsubscribe-auth/AaCumUaEl_fbv8snqq0rW7qwWnWYMPe4ks5rvnxmgaJpZM4M6m1S .

--

Jesse Merandy Director of the Digital Media Lab

Bard Graduate Center: Decorative Arts, Design History, Material Culture 38 West 86th Street, New York, NY 10024 T 212 501 3061 | F 212 501 3065 | E jesse.merandy@bgc.bard.edu jesse.merandy@bgc.bard.edu

r-a-y commented 7 years ago

@jmerandy - Sorry for the late reply. Maybe you can delete the dynamic-admin.js file and try regenerating the file to see if the uploader portion is added to the JS.

See my previous comment for some details to regenerate the JS file: https://github.com/cuny-academic-commons/cbox-theme/issues/256#issuecomment-294004175