Review: Article: "Building a mobile app"

baonlq / html5rocks

Automatically exported from code.google.com/p/html5rocks

0 stars 0 forks source link

Review: Article: "Building a mobile app" #880

Open GoogleCodeExporter opened 8 years ago

GoogleCodeExporter commented 8 years ago

Link to revision:
http://code.google.com/r/dmitrymonin-html5rocks/

Purpose of code changes:
I added a new article: "Building a mobile app", which covers following topics:
- State of HTML5 on Mobile phones 
- Choose of JavaScript framework
- Mobile specific CSS Styling
- UX Considirations
- Perfomance and Handling of Offline states

Original issue reported on code.google.com by dmitry.m...@gmail.com on 8 May 2012 at 8:23

GoogleCodeExporter commented 8 years ago

Original comment by ericbide...@html5rocks.com on 9 May 2012 at 7:23

Changed state: Available
Added labels: External, new, tu

GoogleCodeExporter commented 8 years ago

Original comment by ericbide...@html5rocks.com on 9 May 2012 at 7:24

Added labels: Tutorial, Type-Patch
Removed labels: Type-CodeReview, tu

GoogleCodeExporter commented 8 years ago

Boris can you look this over?

Original comment by ericbide...@html5rocks.com on 9 May 2012 at 7:29

Changed state: as
Added labels: Mobile

GoogleCodeExporter commented 8 years ago

Eric/Dmitry, any way to conveniently stage this for review purposes, or do I 
need to check out that branch and do the dance?

Original comment by s...@google.com on 9 May 2012 at 10:15

GoogleCodeExporter commented 8 years ago

Hi Boris, I would like to assist, but I don't know usual review process. Do you
want to merge my fork into separate branch of main repository? If it can help, 
I attached .diff file with my changes.

Original comment by dmitry.m...@gmail.com on 10 May 2012 at 12:45

Attachments:

building-a-mobile-app.diff

GoogleCodeExporter commented 8 years ago

Thanks for the article, Dmitry. Some notes:

- HTML section is mostly covered here: 
http://www.html5rocks.com/en/mobile/mobifying/#toc-meta
- JS section omits many libraries. Why did you pick these specifically? Either 
this has to be a lot more comprehensive, or you need to justify why you've 
chosen to pick them specifically.
- CSS: you recommend not to use web fonts in "mobile". Why? Modern Android and 
iOS devices are capable.

DEALING WITH NON PERSISTENT CONNECTION is nice.

In general, it would be awesome if you could figure out a more specific topic 
for this, because there's a fair amount of overlap with 
http://www.html5rocks.com/en/mobile/mobifying.

Original comment by s...@google.com on 25 May 2012 at 12:36

GoogleCodeExporter commented 8 years ago

Hi Boris, 
thank you for your feedback and  useful advices. I've modified my article 
according them. Take a look.

Original comment by dmitry.m...@gmail.com on 7 Jun 2012 at 3:51

GoogleCodeExporter commented 8 years ago

Original comment by ericbide...@html5rocks.com on 24 Jul 2012 at 12:36

Changed state: Assigned