[webkit-reviews] review granted: [Bug 186844] [WPE] Add page for webkit.org : [Attachment 343156] Patch

[bugzilla-daemon at webkit.org]

Michael Catanzaro <mcatanzaro at igalia.com> has granted Manuel Rego Casasnovas
<rego at igalia.com>'s request for review:
Bug 186844: [WPE] Add page for webkit.org
https://bugs.webkit.org/show_bug.cgi?id=186844

Attachment 343156: Patch

https://bugs.webkit.org/attachment.cgi?id=343156&action=review




--- Comment #2 from Michael Catanzaro <mcatanzaro at igalia.com> ---
Comment on attachment 343156
  --> https://bugs.webkit.org/attachment.cgi?id=343156
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=343156&action=review

> Websites/webkit.org/wpe.md:6
> +accelerated-content rendering, and simplicity of deployment in mind,

accelerated content, no hyphen

> Websites/webkit.org/wpe.md:11
> +In times when embedded devices become more powerful and ad-hoc
> +to endless applications in a diversity of industries,

There is a missing verb somewhere. This needs rephrased.

> Websites/webkit.org/wpe.md:22
> +* **To provide a no-frills, straight to the point, web-runtime

web  runtime, no hyphen

> Websites/webkit.org/wpe.md:27
> +  and constrains of embedded devices in consideration,

constraints

> Websites/webkit.org/wpe.md:42
> +  are implement in WPE with our goals in mind.

implemented

> Websites/webkit.org/wpe.md:44
> +* **To use hardware-acceleration wherever its advantageous:

hardware acceleration, no hyphen

> Websites/webkit.org/wpe.md:52
> +  through a backend-architecture.**

Either "backend-based architecture" or else "backend architecture" without the
hyphen

> Websites/webkit.org/wpe.md:54
> +  WPE has been designed with a backend architecture,

Um, now here the hyphen has disappeared. :P Be consistent!

> Websites/webkit.org/wpe.md:82
> +The WPE team is planning on making periodic stable releases and
> +these will be announced as soon as they are available.

These are nowadays available on https://wpewebkit.org/.