refactor: visualizer and quick-access components

mgechev / ngrev

Tool for reverse engineering of Angular applications

MIT License

1.57k stars 106 forks source link

refactor: visualizer and quick-access components #82

Closed vik-13 closed 3 years ago

vik-13 commented 3 years ago

Remove detaching app component. Now zone.js handles running change detection;
Add OnPush strategy to most components;
Fix issues with quick-access component (fuse search);
Refactor visualizer component;
Small improvements;

I know it's not necessary to have single module per component, especially for such small app. But as for me it makes component much more easier to understand what's going on there. I mean when you have one module for a few components, sometimes it's hard to say which imports are required for which component.

mgechev commented 3 years ago

Awesome refactoring! I'll take a look once I land ✈️

vik-13 commented 3 years ago

Updated

mgechev commented 3 years ago

Great job! That's a huge improvement compared to before. 🚀

vik-13 commented 3 years ago

Thanks! I'm really looking forward when other views are gonna be implemented... It would be great to get a helpful app at the end :)

mgechev commented 3 years ago

Next I'll be working on restoring the previous component view we had with the template that allows you to navigate to directives & components used inside of it.

vik-13 commented 3 years ago

Sounds great! I'm gonna create one PR soon. Found a solution to not reset the view in case of changing theme or other params. It's a bit tricky, but I'm testing it.. seems it works pretty smooth.