Front end dev

[KorenMary]

No description provided.

[codecov]

Codecov Report

Attention: 39 lines in your changes are missing coverage. Please review.

Comparison is base (4d14a2b) 87.60% compared to head (8661eeb) 88.65%.
Report is 1 commits behind head on main.

Files	Patch %	Lines
src/client/dcp_client/gui/napari_window.py	77.58%	26 Missing ⚠️
src/client/test/test_napari_window.py	84.61%	8 Missing ⚠️
src/client/dcp_client/utils/utils.py	94.91%	3 Missing ⚠️
src/client/dcp_client/gui/welcome_window.py	89.47%	2 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main      #49      +/-   ##
==========================================
+ Coverage   87.60%   88.65%   +1.04%     
==========================================
  Files          22       24       +2     
  Lines        1452     1736     +284     
==========================================
+ Hits         1272     1539     +267     
- Misses        180      197      +17     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[christinab12]

Hi Mariia, please take a look at my comments. The key take-aways here are:

• Always include comments on the functionality of your functions
• Go through your code once at the end of your development and remove any lines which are not being used
• If variables are used only within a class function you don't need to assign it as a class variable
• Good practise is to split code for front-end features and any functionality. It helps readability and keeps things clean!
  Please implement these changes and let me know if you have any questions! Thanks :)