[Enhancement] #83 Improvement of Streamlit Interface for School/Center Searching #88
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
dhirajraut1
suggested changes
Apr 30, 2024
… and hide index of the filtered table
|
Thank you for this. These dont seem to work -
|
added 2 commits
May 4, 2024 21:42
…nto Enhancement_Streamlit_Interface_83
|
@sapradhan Completed reviewed changes Do not use thousand separator comma (27,056) in scode / cscode But this issue Use horizontal layout for radio options for efficient use of screen space: For this i have added a custom css. This does not work in firefox is not replicated in either browsers in my machine . I have tested in chrome, brave,edge and firefox . Below is the image of firefox: |
|
thank you, try to fix other issues reported in #83 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fixes #83
Task Done:
Instead of four radio buttons options scode | school | cscode | center, could we just keep two school | center and show the code along side the school/center name.
Also make it searchable by both name and code.
Use horizontal layout for radio options for efficient use of screen space: For this i have added a custom css
Do not use thousand separator comma (27,056) in scode / cscode
Show Only Unique School,Center,scode,cscode values in filter dropdown and hide index of the filtered table