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(global search bar): display result data with embedded search bar #1712

Merged
merged 1 commit into from
Dec 2, 2024

Conversation

dchiller
Copy link
Contributor

@dchiller dchiller commented Nov 18, 2024

Create two new extendable templates (single_column_with_search_bar.html and filterable_table_with_search_bar.html) for two common use cases with an embedded search bar: one more generally for the embedded search bar and one for views that display filterable tables.

Adjust all current views to use these templates where relevant.

Put the search bar in a bootstrap column so that dynamic search result previews fill the correct horizontal space.

Closes #1703

Old look of the Genre list page while using the global search bar

image

New look of the Genre list page while using global search bar

image

dchiller added a commit to dchiller/CantusDB that referenced this pull request Nov 21, 2024


Follow DDMAL#1712 and conform pagination styling of the browse chants templates to other filterable table templates.
Copy link
Contributor

@lucasmarchd01 lucasmarchd01 left a comment

Choose a reason for hiding this comment

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

Very nice!

Create two new extendable templates for two common use cases with an embedded search bar: one more generally for the embedded search bar and one for views that display filterable tables.

Adjust all current views to use these templates where relevant.

Put the search bar in a bootstrap column so that dynamic search result previews fill the correct horizontal space.

Refs: DDMAL#1703
@dchiller dchiller force-pushed the i1703-global-search-bar-results branch from 892cc3c to 0561179 Compare December 2, 2024 15:15
@dchiller dchiller merged commit e4dddb8 into DDMAL:develop Dec 2, 2024
1 check passed
@dchiller dchiller deleted the i1703-global-search-bar-results branch December 2, 2024 15:18
dchiller added a commit to dchiller/CantusDB that referenced this pull request Dec 3, 2024


Follow DDMAL#1712 and conform pagination styling of the browse chants templates to other filterable table templates.
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.

Chant details don't show up in search bar results on genre and service pages
2 participants