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

Webkit2 4.0 port #99

Merged
merged 2 commits into from
Nov 29, 2019
Merged

Webkit2 4.0 port #99

merged 2 commits into from
Nov 29, 2019

Conversation

chimosky
Copy link
Contributor

No description provided.

@chimosky
Copy link
Contributor Author

@quozl kindly review.

@quozl
Copy link
Collaborator

quozl commented Jul 26, 2018

Thanks. Reviewed to efdc0ae. Tested on Ubuntu 17.10. Fixes #33.

Did this;

  • rebase to master, as master has moved quite a bit since you branched from it for this port; the rebase was clean and simple, and the command was git pull --rebase origin master,
  • reworked the zoom buttons, as they gave a log error when used; please review d53abd5,
  • avoided use of WebKit 3.0 API name; please review a532b13,
  • checked no new flake8 warnings or errors,
  • tested again,
  • captured screenshots of master vs this branch, for use in new issue, Broken scroll bar rendering after port to WebKit2 4.0 API #100,
  • force pushed.

After your review, plan to rewrite the history on this branch before merging. New issue won't block merge.

@quozl
Copy link
Collaborator

quozl commented Jul 26, 2018

Further testing exposes a latent bug; images are not shown. To reproduce, build the content, make a bundle, install it, start the activity, click on Home link, result is;

screenshot_ubb_2018-07-26_17 26 30

Now the _images directory is missing by design, as it is omitted from the bundle by setup.py. It works with WebKit 3.0 API browser because it accepts <img src="_images/file.png"> and is willing to look in the images directory, but the WebKit2 4.0 API browser insists on the correct file name.

ZIP format can handle symlinks, but they aren't created in the build process as far as I can see, and the exclusion in setup.py removes them. Any ideas?

@chimosky
Copy link
Contributor Author

chimosky commented Jul 26, 2018

Further testing exposes a latent bug; images are not shown. To reproduce, build the content, make a bundle, install it, start the activity, click on Home link, result is;

I can't reproduce, images are showing in mine.
Screenshot_from_2018-07-26_13-33-57.png

captured screenshots of master vs this branch, for use in new issue, #100,

Scrollbar rendering works fine in mine too.

I don't know why mine is Sugar 0.106 documentation after building the content and making a bundle.

@chimosky
Copy link
Contributor Author

chimosky commented Aug 7, 2018

@quozl, i tested and i don't get the same behaviour.

@quozl
Copy link
Collaborator

quozl commented Aug 7, 2018

Interesting, thanks. Please show me the contents of the bundle you built.

@chimosky
Copy link
Contributor Author

@quozl the image is here

@quozl
Copy link
Collaborator

quozl commented Dec 10, 2018

Thanks. In your bundle;

  • the images/ directory contains the images (not _images/),
  • the html/home_view.html file has an img tag that references _images/ (not images/).

This is technically invalid; the browser should try to open a directory _images but somehow opens a directory images.

So why is it working for you and not for me? Please suggest actions to diagnose.

@chimosky
Copy link
Contributor Author

@quozl can you test again with sugar v0.116, I just tested and I don't face any issue you mentioned above.

@quozl
Copy link
Collaborator

quozl commented Nov 29, 2019

Sure. Retested 1560ae9 by making a dist_xo bundle and installing it on he same system; Ubuntu 17.10 with Sugar 0.116.

  • the images in the Home View are not shown, as before in July 2018,
  • both your bundle and mine have an img tag with src set to "_images/Home_icon.png" yet there is no directory named "_images" in the directory "html", though there is a directory "images" next to the directory "html".

Neither of these are created by Sugar, so the version of Sugar is unsurprisingly ineffective.

chimosky and others added 2 commits November 30, 2019 10:22
- set zoom buttons insensitve when a zoom step is unavailable,

- limit to ten zoom steps,

- add accelerators (ctrl+-, ctrl++, ctrl+0),

Co-authored-by: James Cameron <[email protected]>
@quozl
Copy link
Collaborator

quozl commented Nov 29, 2019

Never mind, my mistake. I had not tested before your changes. Having done that, the same problem occurs, so there's no reason not to merge.

@quozl
Copy link
Collaborator

quozl commented Nov 29, 2019

Rebased.

@quozl quozl merged commit 8b6469c into godiard:master Nov 29, 2019
@quozl quozl mentioned this pull request Nov 29, 2019
@chimosky chimosky deleted the webkit2-4.0-port branch March 26, 2021 00:36
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.

2 participants