Improved the UI and updated two py files #159
Conversation
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
There was a problem hiding this comment.
Great job, @Priyaanshucodez! 🎉 Thank you for submitting your pull request. Your contribution is valuable and we appreciate your efforts to improve our project.
We will promptly review your changes and offer feedback. Keep up the excellent work! Kindly remember to check our contributing guidelines
There was a problem hiding this comment.
looks good, could you add more margins between the buttons and charts in amplifier-frequency.py and energy-frequency .py file @Priyaanshucodez
ok sir |
|
@dinxsh i tried to increase the Margin but somehow it wasn't looking good so i changed the UI to a simpler version opening three different tabs for each task . |
|
i created the pr @dinxsh sir and @Soumya-Kushwaha Ma'am ....Just have a look and I had a request that if you could assign this as a level 3 issue instead of level 2 since a lot of changes i had to do ..I would be really grateful ...Rest it was great collaborating on this project with both of you . |
|
@Soumya-Kushwaha Ma'am your review is left ...Kindly review and close the issue . |
Plz review it from your side |
|
@dinxsh Sir there has been no reply from @Soumya-Kushwaha Ma'am ...I emailed her too ..Kindly inform her from your side . |
|
@Soumya-Kushwaha Ma'am plz review the PR ..It's been over 1 week now |
|
Ma'am plz close this one ...@dinesh changed the level from 2 to 3 |
Improved The UI with reference to #157 and updated two more files (Amplitude visualizer and Energy visualizer)