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
Refactor/2318/migrate pre render service 2 #2980
Refactor/2318/migrate pre render service 2 #2980
Conversation
shaman-apprentice
commented
Aug 11, 2022
•
edited
edited
- fix loading indicator isn't working #2977
- ref Migrate all Angular Components and Services #2318 (migrate codeMap.preRender.service)
- explicit state actions on which to render
- throttle rendering (reducing total initial calls to render of default dev map from 58 to 1)
- a very rough performance analysis through chrome's performance profile shows no real difference (uploading a 1.704kb and a 28.596kb file)
- delete code for re-selecting building after map change as this didn't work and wasn't missed
…oading indicators ref #2318
70cf60d
to
ad4b8c3
Compare
@shaman-apprentice is it intentional that this is still a draft? I guess so as there are still some open ToDos? |
…-pre-render-service-2
…-pre-render-service-2
…-pre-render-service-2
[CodeCharta Analysis] Kudos, SonarCloud Quality Gate passed! |
[CodeCharta Visualization] SonarCloud Quality Gate failed. |
visualization/app/codeCharta/state/effects/renderCodeMapEffect/renderCodeMap.effect.ts
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I haven't noticed any different behavior than with the non-migrated components (plus the loading gif works again). So I give you a go! 😃