Skip to content
This repository has been archived by the owner on Oct 6, 2020. It is now read-only.

NUXT ❤️ PWA #77

Merged
merged 3 commits into from
Jun 21, 2017
Merged

NUXT ❤️ PWA #77

merged 3 commits into from
Jun 21, 2017

Conversation

pi0
Copy link
Member

@pi0 pi0 commented Jun 20, 2017

Hi! Here is our latest hnpwa implementation with Nuxt.js framework built on top of Vue. Originally ported from vue-hackernews-2 so original design credits back to them :)

Appendix: it would be so awesome if it could be merged just before VueConf (tomorrow), me & nuxt team can be responsible for any immediate changes that required before merge.

interactive-em: '2.8s'
interactive-faster-3g: '2.4s'
lighthouse-link: https://www.webpagetest.org/lighthouse.php?test=170620_PG_a2a9feaf4ace07a61b2c6c2a171b1c79&run=1
wpt-em-link: https://www.webpagetest.org/result/170620_AM_f5e1ae2f305c1d81b44cfe0c9c930e35
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like the interactive times are do not quite match the report, and there was an issue with the main thread settling in the EM_3G test.

Started some new tests here:
em - https://www.webpagetest.org/result/170620_2Y_5f5fdbaae7d479adf52986a98fc8a720/
faster - https://www.webpagetest.org/result/170620_WP_e35e74fdc07b32752e1b642cf8e159f9/

Will take a look after they complete.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That's strange with webpagetest results. Things like https redirect should be passed too but failing here! Would you give some points why that mainThread problem happens? Isn't it active because of prefetching js files?

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The http->https problem is a known issue unfortunately, every implementation is impacted :(

See here for details: GoogleChrome/lighthouse#2363

@kristoferbaxter
Copy link
Collaborator

@addyosmani We've seen a few entries get caught in this stage, where a Consistently Interactive time cannot be recorded. I believe we should consider adding them.

@housseindjirdeh
Copy link
Collaborator

Aside from updating the interactive numbers, this PR looks good to me.

@kristoferbaxter agreed. I think we can update all the entries once they all record consistently interactive numbers (mentioned #60)

@pi0
Copy link
Member Author

pi0 commented Jun 20, 2017

@kristoferbaxter @housseindjirdeh Thanks for your time & quick response :)
I've just updated results. At least lighthouse for 3g is not having consistency interactive problem anymore.

Is there anything needed from our side before merge?

Copy link
Collaborator

@housseindjirdeh housseindjirdeh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good to me 👍

This is such an awesome app - @pi0 @alexchopin @atinux you folks did a outstanding job so thank you for submitting this 🎉

@housseindjirdeh
Copy link
Collaborator

housseindjirdeh commented Jun 21, 2017

Appreciate it if you would like to take a look @addyosmani as well but will be merging this in because of urgency (VueConf tomorrow)

@housseindjirdeh housseindjirdeh merged commit 70ba93a into tastejs:master Jun 21, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants