Skip to content

Conversation

@dchiller
Copy link
Collaborator

@dchiller dchiller commented Jan 29, 2025

Closes #929.

In addition to various search fixes, this PR reorganizes files in the nginx container to reduce complexity of the directory structure. Updates build processes to account for this change.

Initial frontend builds were failing after 9f0372e. This updates the babel configuration for correct building.
With recent Marionette changes, View `events` methods are attached before the view is initialized, so debouncing the setQuery function must happen after the event infrastructure is initialized.

A separate dev development installation of underscore is also no longer necessary.
View uses pre-rendered content, so child views need to be shown in initialization.
The CollectionView's sort method is what includes the rendering of child views, so we shouldn't overwrite with nothing (if we need a different sorting, we can add a new comparator function later).
By default, the view listens to events on the view's el.
…ng music

Temporarily fix an error in volpiano-text alignment whereby missing music that has no accompanying words causes an exception in the process of adjusting missing music spacing to accompanying words. This fix is temporary until the volpiano-display-utilities module is implemented on Cantus Ultimus.
@dchiller dchiller changed the title I929 search fixes Clean-up search after Marionette upgrade; rearrange nginx folder Jan 29, 2025
@dchiller dchiller merged commit 71f03f8 into DDMAL:main Jan 29, 2025
2 checks passed
@dchiller dchiller deleted the i929-search-fixes branch January 29, 2025 17:15
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.

Marionette upgrades caused a variety of issues in the search interface

1 participant