CodeIt icon indicating copy to clipboard operation
CodeIt copied to clipboard

Change the color of time complexity chart.

Open multiverseweb opened this issue 1 year ago • 10 comments

In light mode, the color of time complexity chart should be black.

image

multiverseweb avatar Oct 01 '24 20:10 multiverseweb

Hello, Please assign it to me. i would like to work on it.

nithinrvs avatar Oct 01 '24 20:10 nithinrvs

Hi @multiverseweb , I think you need something like this

image

I've already started working on it, It'll be great if you assign this issue it to me....

ShivanshPlays avatar Oct 01 '24 23:10 ShivanshPlays

Hi I would really love an opportunity to contribute on this issue!

Hustler004 avatar Oct 02 '24 15:10 Hustler004

Hi any update? @multiverseweb

ShivanshPlays avatar Oct 03 '24 15:10 ShivanshPlays

@ShivanshPlays yes, that is what I want, assigning it to you. The color does not need to be black exactly, its looking a bit odd. Make it a bit lighter.

multiverseweb avatar Oct 03 '24 15:10 multiverseweb

Okay, I am on it.

ShivanshPlays avatar Oct 03 '24 15:10 ShivanshPlays

assign please @multiverseweb

ankurrrrr7 avatar Oct 03 '24 18:10 ankurrrrr7

@ankurrrrr7 look for the issues that have not been assigned yet.

multiverseweb avatar Oct 03 '24 19:10 multiverseweb

@multiverseweb bhaiya karliya abhi karoon pr fix hogya yeh!

ankurrrrr7 avatar Oct 03 '24 19:10 ankurrrrr7

@ankurrrrr7 okay, go ahead.

multiverseweb avatar Oct 03 '24 21:10 multiverseweb

hello sir @multiverseweb , can you pls assign me this task.

BaniSinghgithub avatar Oct 22 '24 06:10 BaniSinghgithub

Sure @BaniSinghgithub, assigned.

multiverseweb avatar Oct 25 '24 00:10 multiverseweb

Any updates @BaniSinghgithub?

multiverseweb avatar Oct 27 '24 22:10 multiverseweb

@multiverseweb sorry totally missed this one, can you assign me, I'll raise the PR by tomorrow

ShivanshPlays avatar Oct 27 '24 22:10 ShivanshPlays

Sure @ShivanshPlays, make sure the color of the grid lines and numbers on the graph should change. Unlike adding background like this: image

The PR will be merged on First Come First Serve basis.

multiverseweb avatar Oct 27 '24 23:10 multiverseweb

Hi @multiverseweb , I think you need something like this

image

I've already started working on it, It'll be great if you assign this issue it to me....

@ShivanshPlays Try changing the color of numbers on the axes too.

multiverseweb avatar Oct 27 '24 23:10 multiverseweb

understood

ShivanshPlays avatar Oct 27 '24 23:10 ShivanshPlays

Hey @ShivanshPlays, are you working on this issue or shall we close this?

multiverseweb avatar Nov 06 '24 02:11 multiverseweb

@multiverseweb If the approach meets the expected standards, please consider merging this enhancement. Thank you for the opportunity to contribute!

Screenshot 2024-11-08 182835

raavan4m3 avatar Nov 08 '24 13:11 raavan4m3

@multiverseweb I have completed the changes as requested and have submitted a pull request for review. Please take a look at it when you have a chance.

Let me know if there are any further changes or adjustments needed.

raavan4m3 avatar Nov 08 '24 13:11 raavan4m3

Hey @raavan4m3 actually this is what is required: image

I hope you can change the color of graph like this

multiverseweb avatar Nov 09 '24 05:11 multiverseweb

@multiverseweb Screenshot 2024-11-09 123056

raavan4m3 avatar Nov 09 '24 07:11 raavan4m3

@raavan4m3 Yes, this looks good

multiverseweb avatar Nov 09 '24 07:11 multiverseweb

@multiverseweb I have completed the changes as required and have submitted a pull request for review. Screenshot 2024-11-09 123056

raavan4m3 avatar Nov 09 '24 12:11 raavan4m3