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

Caret iframe support / refactor default document occurrences #1152

Open
wants to merge 2 commits into
base: howling-mad
Choose a base branch
from

Conversation

evindor
Copy link

@evindor evindor commented Apr 4, 2014

Not ready to merge yet, i'm opening this PR for a discussion. We started it here.

At first i would like to know if this is an issue which needs to be fixed. Right now caret is not rendering in case of aloha being fired up from top document on an element which is in iframe. Is this a use case for aloha? Should we fix it?

While digging in it i found out that somewhere Aloha uses default document, and this is subject to change for sure. In the current state this PR contains some document refactoring, fixes iframe issue and brings in one more sandbox case with iframe.

@GenticsDev
Copy link
Contributor

Can one of the admins verify this patch?

@MKuenzi
Copy link

MKuenzi commented May 30, 2014

Development cycle wise when would I see a fix like this rolled down to .25? I am trying to implement Aloha in this same fashion and I would love to see this working in the stable release.

Upon looking further, I'm seeing aloha.js is radically different from .25? I'm guessing this is just going to be part of a 1.0 release then? And in that case, any time frame for the 1.0 release?

@evindor
Copy link
Author

evindor commented Jun 3, 2014

@petrosalema

@petrosalema
Copy link
Member

@evindor Yes, we should replace all references to the global document object with ownerDocument as you've started to do. This will solve some of the iframe issues.

@petrosalema
Copy link
Member

@MKuenzi Yes, Aloha Editor in the "howling-mad" branch is a very different beast. We've revolutionised our approach to editing. We will be announcing soon the launch date for this new API.

@MKuenzi
Copy link

MKuenzi commented Jun 3, 2014

@petrosalema Thanks for the response. Is there anywhere I can check out a build of this? Otherwise I can build this but I'm running into issues. I have started a thread if you have time to help me https://getsatisfaction.com/aloha_editor/topics/building_howling_mad

@petrosalema
Copy link
Member

@MKuenzi, to build:

  1. Got into the root directory in the "howling-mad" branch
  2. Create a "build" directory
  3. Simply run ./build.sh to see build options.

@cprerovsky cprerovsky force-pushed the howling-mad branch 2 times, most recently from cf709a2 to 9c8c31f Compare September 10, 2014 15:52
@Jotschi Jotschi changed the title Caret iframe support / refactor default document occurancies Caret iframe support / refactor default document occurrences Oct 31, 2014
@Jotschi Jotschi added this to the 2.0.x milestone Oct 31, 2014
@GenticsDev
Copy link
Contributor

Can one of the admins verify this patch?

1 similar comment
@GenticsDev
Copy link
Contributor

Can one of the admins verify this patch?

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.

5 participants