Skip to content
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

Fixed Editor issues #60

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Fixed Editor issues #60

wants to merge 2 commits into from

Conversation

igmoweb
Copy link

@igmoweb igmoweb commented Jul 11, 2016

  • inkblot mods were not properly loaded when wp-admin is under SSL
  • Adding full stylesheet in editor was preventing WP Editor to work

When using add_editor_style is better to load just a few styles instead of the full style.css as many things can go wrong in the editor (like this case)

I've also removed home_url('/') and replaced it for admin_url() when loading inkblot mods in the editor as it was throwing a mixed content error in JS console when wp-admin is under SSL. This will add the inkblot-mods under SSL if needed.

Thanks

- inkblot mods were not properly loaded when wp-admin is under SSL
- Adding full stylesheet in editor was preventing WP Editor to work
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant