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

Wysiwyg Editor / javascript may incompatible #87

Closed
Patrick1701 opened this issue Feb 5, 2014 · 5 comments
Closed

Wysiwyg Editor / javascript may incompatible #87

Patrick1701 opened this issue Feb 5, 2014 · 5 comments
Milestone

Comments

@Patrick1701
Copy link
Contributor

Hi Sebastien,
quite a long time... had about 5 Weeks over Christmas and still under heavy workload. :-)
Hope you are fine (and a happy new year ;-) )

I tried to use the Wysiwyg Editor and it seems there is a javascript problem, possibly a compatibiliy issue.

The page stops applying all jquery scripts, when Wysiwyg Editor is present on page.

I am not sure, if is a problem of yours or the Wysiwyg scripts, but I just want you to be informed there is a problem.

See screenshot attached.
unexpectederror

kind regards
Patrick

@sebfz1
Copy link
Owner

sebfz1 commented Feb 5, 2014

Hi Patrick,

I am going very well, thanks! I know what you mean by heavy workload, it is the same for me! :)

Thanks for reporting this issue. I did not noticed this issue, it works fine on the demo site...
@solomax did some changes in the latest version (wicket-jquery-ui-6.13.1) about compatibility issues. Could you please try this version?
Pay attention to read changelogs while upgrading:
https://github.com/sebfz1/wicket-jquery-ui/wiki/_pages

Best regards, talk to you soon,
Sebastien.

@sebfz1
Copy link
Owner

sebfz1 commented Feb 16, 2014

Hi Patrick,

Actually, this is a known issue in the bootstrap-wysiwyg project (since several month :s).
A pull request is available but has not been merged yet: mindmup/bootstrap-wysiwyg#68

@solomax, would you like to fork & merge the pull request by ourself?

Sebastien.

@solomax
Copy link
Contributor

solomax commented Feb 16, 2014

Hello Sebastien,
Will try to take a look at this pull request ASAP

@solomax
Copy link
Contributor

solomax commented Feb 21, 2014

I have sent the pull request
Hopefully it will be included into upcoming 6.14 :)

@sebfz1 sebfz1 added this to the 6.14.0 milestone Feb 21, 2014
@sebfz1
Copy link
Owner

sebfz1 commented Feb 21, 2014

Nice ! Thanks you very much Maxim ! 😄
I will merge it and release 6.14.0 over the WE...

Best regards,
Sebastien.

@sebfz1 sebfz1 closed this as completed Feb 21, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants