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

Update system/cms/themes/pyrocms/views/fragments/wysiwyg.php #1434

Merged
merged 1 commit into from
Jul 30, 2012
Merged

Update system/cms/themes/pyrocms/views/fragments/wysiwyg.php #1434

merged 1 commit into from
Jul 30, 2012

Conversation

Renatov
Copy link

@Renatov Renatov commented May 14, 2012

this change is because how to load the js with the new function

<script src = ""> ckeditor brings problems opening in internet explorer 6 back

@philsturgeon
Copy link
Member

What was the problem with the existing code? Was it building the file paths wrong?

@Renatov
Copy link
Author

Renatov commented May 20, 2012

hi phil, it happens that when loading rows ckeditor / ckeditor.js, ckeditor / adapters / jquery.js with the new function Asset :: get_filepath_js(), the load is incomplete in internet explorer which causes problems loading jkeditor plugin not only in internet explorer if no other browser and this acrecenta internet explorer versions from 6 onwards.
I work with pyrocms and done 2 projects for two companies, the addresses of these projects are www.serviciosponex.com (this I am using the library multilang.php that helps make a translator for the navigation links and things that go stored in the database), the other is www.spamediconostas.com, here my first deployment I had and good I'm unhappy to use flash as main background animation as I have brought many problems in different browsers.
Pyro puts emotion to programming.

@jerel
Copy link
Member

jerel commented Jun 20, 2012

Could you show us the url that is generated when this problem occurs? If there's a problem we need to fix get_filepath_js() instead as your fix will break if someone changes admin themes in the Pro version

@Renatov
Copy link
Author

Renatov commented Jul 3, 2012

Jerel Hello! I send you this response also put it to Adam, this is the way I found to solve the loading problem in internet explorer:

The link is this: #1430 (comment)

Regardss

philsturgeon pushed a commit that referenced this pull request Jul 30, 2012
Update system/cms/themes/pyrocms/views/fragments/wysiwyg.php
@philsturgeon philsturgeon merged commit f857ac8 into pyrocms:2.1/develop Jul 30, 2012
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.

3 participants