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

New rendering pipeline #89

Merged
merged 8 commits into from
Jun 5, 2014
Merged

Conversation

rs22
Copy link
Contributor

@rs22 rs22 commented Jun 3, 2014

I think, it's ready for prime time 😉
There are no tests yet, maybe we should add at least some until thursday.

The Tanzmaus-Image (http://de.selfhtml.org/html/grafiken/anzeige/img.htm) is still not displayed correctly - this seems to be unrelated to the new pipeline, and I'm not sure, what the problem is in detail.
(It probably has to do with embedding submorphs in the TextMorph and their positioning: BitBltDisplayScanner>>placeEmbeddedObject:)
If problems with displaying images occur more frequently, we should make a bug/user story out of it, but I've not seen any other websites with this problem yet.

@Steditor
Copy link
Contributor

Steditor commented Jun 5, 2014

Works for me. I am still trying to think of a meaningful test, but have not yet been successful.

But so far: ⛵

@PFischbeck
Copy link
Contributor

Works for me as well 🐬

PFischbeck added a commit that referenced this pull request Jun 5, 2014
@PFischbeck PFischbeck merged commit 982673f into master Jun 5, 2014
@MrSerth
Copy link
Contributor

MrSerth commented Jun 5, 2014

The methods displayImagePage:, displayPlainTextPage: and processDocument: in Scamper are not working anymore. Are you able to change the lines accessing the two variables formattedPage and backgroundColor? In addition, the accessors are useless (also formattedPageSelection) and the step-method has errors.

@Steditor
Copy link
Contributor

Steditor commented Jun 5, 2014

Those are known issues that we have not yet dealt with, because those methods haven't been used by us so far. But this has to be done, therefore: #94

@MrSerth MrSerth deleted the rendering/new-rendering-pipeline branch June 5, 2014 13:26
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.

4 participants