search

multiverseweb / CodeIt

CodeIt is a software solution tool designed to streamline and enhance the coding experience for developers. It addresses several common challenges faced when working with coding platforms, such as LeetCode, and offers a range of features to improve code management, security, and performance analysis.
https://codeittool.netlify.app
MIT License
50 stars 99 forks source link

Change color of time complexity chart #16 #334

Closed raavan4m3 closed 2 weeks ago

raavan4m3 commented 2 weeks ago

The toggleThemeBtn function is designed to switch the theme of a time complexity chart between dark and light modes and to update the visibility of theme-toggle buttons accordingly. If currentTheme is "dark": Hides the dark theme button and shows the light theme button. Sets the gridColor to a semi-transparent white (rgba(255, 255, 255, 0.1)) for the chart’s grid lines, suitable for dark mode. Updates the x- and y-axis grid colors of the complexityChart object to the defined gridColor. If currentTheme is not "dark" (assumes "light" theme): Sets gridColor to black, suitable for light mode. Updates the x- and y-axis grid colors of the complexityChart object to the defined gridColor. Shows the dark theme button and hides the light theme button. Screenshot 2024-11-09 123056

netlify[bot] commented 2 weeks ago

Deploy Preview for codeittool ready!

Name Link
Latest commit 37a00802e218769f1a32af835b590d520d84c999
Latest deploy log https://app.netlify.com/sites/codeittool/deploys/67306f0e3e247400089ae8ec
Deploy Preview https://deploy-preview-334--codeittool.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

multiverseweb commented 2 weeks ago

Hey @raavan4m3, the chart looks fine now but you have made the changes in resources/common_functions.js (it works on all tabs) which has disturbed the theme feature in tabs other than Complexity Analyser. This can be solved by checking whether the chart exists on the page or not using an if condition in js file. Kindly fix this issue so that I can merge the PR.

raavan4m3 commented 2 weeks ago

@multiverseweb yes i know what i have done in line number 172 it should be const textColor =isDark? "white":"black" ; that has been changed to const textColor =white" ; wait it will fix it

raavan4m3 commented 2 weeks ago

@multiverseweb yes i have done the changes pls look into it and let me know if somthing is not upto mark

multiverseweb commented 2 weeks ago

@multiverseweb yes i know what i have done in line number 172 it should be const textColor =isDark? "white":"black" ; that has been changed to const textColor =white" ; wait it will fix it

@raavan4m3 I think you misunderstood the problem that I stated. Let me show it to you here:

https://github.com/user-attachments/assets/a37e3ba3-d95f-4a78-93a1-5b8d99180a18

The theme toggle works fine for time complexity analyser but now it is not working in code obfuscator, code refiner, home and other tabs.