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

[4.13.100] Optimize initialization speed #2841

Merged
merged 16 commits into from
May 16, 2022
Merged

Conversation

yushih
Copy link
Collaborator

@yushih yushih commented May 2, 2022

Please review commit c5d2601 individual if it helps.

ee7d093 makes perceived loading speed faster by loading wallet data after the main UI starts to render.

80f8576 loads SVGs on-demand (some of them are huge but rarely used). But because of it, all the SVG import statements need to be rewritten: c5d2601 is just mechanical changing of all import syntax for SVGs. Also, flow type of SVG imports must be fixed by bd83cd2.

Other commits should be straightforward by the commit messages.

@yushih yushih force-pushed the yushi/startup-speed-1 branch from 69f3416 to fb837bf Compare May 5, 2022 17:28
@yushih yushih changed the title Make perceived loading speed faster by loading wallet data after the main UI starts to render Optimize initialization speed May 5, 2022
neuodev
neuodev previously approved these changes May 6, 2022
@neuodev neuodev dismissed their stale review May 6, 2022 07:59

Not all import statements for SVGs is updated! this might cause crash for the extension when navigating to these page

@yushih yushih force-pushed the yushi/startup-speed-1 branch 2 times, most recently from e34ac57 to 9992d54 Compare May 10, 2022 12:34
@yushih yushih force-pushed the yushi/startup-speed-1 branch from 9204e71 to 6719315 Compare May 11, 2022 07:33
@vsubhuman vsubhuman added this to the 4.13.100 milestone May 13, 2022
@vsubhuman vsubhuman changed the base branch from develop to production May 13, 2022 14:02
@vsubhuman vsubhuman self-assigned this May 13, 2022
@vsubhuman vsubhuman changed the title Optimize initialization speed [4.13.100] Optimize initialization speed May 13, 2022
@@ -39,7 +39,10 @@ export const generateAdaPaperPdf = async (

const width = 595.28;
const height = 841.98;
const doc = new Pdf({

const JsPdf = (await import('jspdf')).default;
Copy link
Contributor

Choose a reason for hiding this comment

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

nice

vsubhuman
vsubhuman previously approved these changes May 13, 2022
Copy link
Contributor

@vsubhuman vsubhuman left a comment

Choose a reason for hiding this comment

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

Thank you, @yushih !

Copy link
Contributor

@vsubhuman vsubhuman left a comment

Choose a reason for hiding this comment

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

/check

(accidental request changes)

vsubhuman
vsubhuman previously approved these changes May 13, 2022
Copy link
Contributor

@vsubhuman vsubhuman left a comment

Choose a reason for hiding this comment

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

/check

@@ -17,5 +17,6 @@ module.exports = {
'jest-canvas-mock'
],
// ignore manifest.test.js file, because it isn't a test
testPathIgnorePatterns: ['manifest.test.js']
testPathIgnorePatterns: ['manifest.test.js'],
modulePathIgnorePatterns: ['jormungandr'],
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Oops, forgot about this.

vsubhuman
vsubhuman previously approved these changes May 13, 2022
Copy link
Contributor

@Nebyt Nebyt left a comment

Choose a reason for hiding this comment

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

/check

Nebyt
Nebyt previously approved these changes May 14, 2022
Copy link
Contributor

@Nebyt Nebyt left a comment

Choose a reason for hiding this comment

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

/check

Nebyt
Nebyt previously approved these changes May 14, 2022
@vsubhuman
Copy link
Contributor

@vsubhuman vsubhuman merged commit 198c49e into production May 16, 2022
@vsubhuman vsubhuman deleted the yushi/startup-speed-1 branch May 16, 2022 10:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants