Skip to content

Made BarChart Clickable to ProblemSet Page#1

Open
KanniShashankh wants to merge 4 commits intoApoorvaRajBhadani:mainfrom
KanniShashankh:KanniShashankh-MakePopup_clickable
Open

Made BarChart Clickable to ProblemSet Page#1
KanniShashankh wants to merge 4 commits intoApoorvaRajBhadani:mainfrom
KanniShashankh:KanniShashankh-MakePopup_clickable

Conversation

@KanniShashankh
Copy link
Copy Markdown

No description provided.

@ApoorvaRajBhadani
Copy link
Copy Markdown
Owner

Thank You for doing this! Can you please share a screen recording of the exact functionality? That would be helpful.

@KanniShashankh
Copy link
Copy Markdown
Author

Thanks @ApoorvaRajBhadani , Here is a quick demo. Whenever we click on the rating graph, it takes us to the problemset page with that rating selected. :)

Screen.Recording.2023-08-19.at.11.30.39.PM.mov

@ApoorvaRajBhadani
Copy link
Copy Markdown
Owner

Will review it in few days. Sorry for the delay.

@ApoorvaRajBhadani
Copy link
Copy Markdown
Owner

ApoorvaRajBhadani commented Sep 20, 2023

Hey what I would suggest is - if we click on the bar then the problems opened should be of the said difficulty but the problems should be what is solved by that user?

#2 is also requesting something similar.

@KanniShashankh
Copy link
Copy Markdown
Author

I think we can get the list of problems solved by a user, but im not sure the problemset has filters based on problem name/problemID

@07nainankit
Copy link
Copy Markdown

I think we can get the list of problems solved by a user, but im not sure the problemset has filters based on problem name/problemID

no issue it's not necessary that we need the problems filtered in problem set , it can be any way. Just a simple list will be good enough.

@ApoorvaRajBhadani
Copy link
Copy Markdown
Owner

@ApoorvaRajBhadani
Copy link
Copy Markdown
Owner

The code has a logical error. You are assuming user would have solved problems with subsequent rating with a difference of 100, but this is not always true.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants