Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Suggestion : Displaying the values of the bars for better visualization and understanding #168

Open
tkgowtham opened this issue Sep 3, 2023 · 8 comments
Assignees
Labels
enhancement New feature or request

Comments

@tkgowtham
Copy link

tkgowtham commented Sep 3, 2023

  • Idea :
    I would like to display the values of the bars that are being sorted just right below it, so it will be easier for user to understand what values are being sorted and how the sorting works in that particular algorithm.

  • How it could be implemented :
    This could be implemented by adding a box right below the bar. We could display the values of the bar respectively there. In each iteration, the values would move along with the red bar to show the user, how the algorithm sorts the values.

    It would also be diffcult to display values of larger size or larger count of values, since they can intersect each other, so we will also have to be able to give user the accessibilty to resize the window.

  • I would like to do this if you want to implement my suggestion.

@tkgowtham tkgowtham added the enhancement New feature or request label Sep 3, 2023
@LucasPilla
Copy link
Owner

LucasPilla commented Sep 19, 2023

@tkgowtham I like the idea, but I don't think it will look good with a lot of bars. Maybe we should add this feature if number of bars is less then X, what do you think?

@tkgowtham
Copy link
Author

tkgowtham commented Sep 19, 2023

Yes, I agree with your point. We can assume X to be less than or equal to 50. So, can I work on it ? If so, please assign me to this issue.

@dan424gg
Copy link

I would like to be assigned to this issue.

@LucasPilla

@nanashi-03
Copy link

How about a grid instead? It would be pretty easy to guess the numbers and it works for any amount of bars.

@dhruva2000
Copy link

Hey I can work on this I have some ideas for how to implement this

@LucasPilla LucasPilla assigned dhruva2000 and unassigned tkgowtham Nov 29, 2023
@LucasPilla
Copy link
Owner

Sure @dhruva2000, this issue is quite simple, we just need to find some way to show the bar values. Maybe just showing the values when the number of bars is small, otherwise it might get too cluttered.

@dhruva2000
Copy link

Got it thank you!

@dhruva2000
Copy link

Just attached my pull request onto this @LucasPilla! . I incorporated some of the changes from the previously opened pull request and added values to the bars again so I'm hoping this successfully captures the suggestion by @tkgowtham. I also added some padding and shifted some of the components to create more visual clarity in the app

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants