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
fix: Added Pagination & Updated sidebar with counts #271
base: main
Are you sure you want to change the base?
Conversation
jagadeeshgade008
commented
Mar 5, 2024
•
edited
edited
- added pagination for features,attributes, segments
- updated nav bar to side in desktop UI along with counts
entitiesCount in App component
ListFeatures, and ListSegments components
SearchInput components
ListFeatures, and ListSegments components
thanks so much for the PR, @jagadeeshgade008! for awareness, since #274 we have a revision number that is numeric. do you have any opinion where to put this info? my suggestions (which you can reject):
this does lead to more design related questions, but I am open to hearing your feedback first. |
@fahad19 How about adding the version number in the left sidebar bottom instead of adding two different menus |
@jagadeeshgade008: that could work too 👍 |
@fahad19 How can we get the version number |
I believe a bit of extra work would be needed to make it available in generated
|
@fahad19 updated revision Can you review this |
|
||
React.useEffect(() => { | ||
fetch("/search-index.json") | ||
.then((response) => response.json()) | ||
.then((data) => { | ||
setRevision(data.revision); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
@@ -439,6 +439,7 @@ export interface LastModified { | |||
} | |||
|
|||
export interface SearchIndex { | |||
revision: string; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
@fahad19 updated code changes |