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

fix: Added Pagination & Updated sidebar with counts #271

Open
wants to merge 22 commits into
base: main
Choose a base branch
from

Conversation

jagadeeshgade008
Copy link
Contributor

@jagadeeshgade008 jagadeeshgade008 commented Mar 5, 2024

  • added pagination for features,attributes, segments
  • updated nav bar to side in desktop UI along with counts
image

@fahad19
Copy link
Member

fahad19 commented Mar 14, 2024

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):

  • keep the top bar like we had before, and also keep the new sidebar
  • on top bar's right side, include the revision info and also a button that links to the Git repo

this does lead to more design related questions, but I am open to hearing your feedback first.

@jagadeeshgade008
Copy link
Contributor Author

@fahad19 How about adding the version number in the left sidebar bottom instead of adding two different menus

@fahad19
Copy link
Member

fahad19 commented Mar 23, 2024

@jagadeeshgade008: that could work too 👍

@jagadeeshgade008
Copy link
Contributor Author

@fahad19 How can we get the version number
I have tried building the static site it is not available in search-index.json

@fahad19
Copy link
Member

fahad19 commented Apr 21, 2024

I believe a bit of extra work would be needed to make it available in generated search-index.json:

@jagadeeshgade008
Copy link
Contributor Author

Screenshot 2024-04-22 at 14 16 18 @fahad19 updated revision Can you review this


React.useEffect(() => {
fetch("/search-index.json")
.then((response) => response.json())
.then((data) => {
setRevision(data.revision);
Copy link
Member

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;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@jagadeeshgade008
Copy link
Contributor Author

@fahad19 updated code changes

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.

None yet

2 participants