Skip to content

Test on Blacklight 9#1633

Merged
corylown merged 1 commit intomainfrom
test-on-bl9
Feb 9, 2026
Merged

Test on Blacklight 9#1633
corylown merged 1 commit intomainfrom
test-on-bl9

Conversation

@corylown
Copy link
Contributor

@corylown corylown commented Jan 29, 2026

Includes much of #1628
Drops Blacklight 8

@corylown corylown force-pushed the test-on-bl9 branch 4 times, most recently from b9f4dd4 to 46ec421 Compare February 2, 2026 16:28
@corylown corylown changed the title Test on bl9 Test on Blacklight 9 Feb 2, 2026
@corylown corylown marked this pull request as ready for review February 2, 2026 18:41
Copy link
Contributor

@seanaery seanaery left a comment

Choose a reason for hiding this comment

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

This is great - thanks for working through these changes! I see that the three test environments in CI are all now using BL9 and all have passing tests. I can get the app to run without error now using this branch. The UI will definitely need some subsequent work to catch up to all the BL9 changes :-)

@corylown corylown merged commit 4bc3e9c into main Feb 9, 2026
4 checks passed
@corylown corylown deleted the test-on-bl9 branch February 9, 2026 14:43
@corylown
Copy link
Contributor Author

corylown commented Mar 4, 2026

@seanaery did you have any particular issues or changes in mind when you mentioned:

The UI will definitely need some subsequent work to catch up to all the BL9 changes

@seanaery
Copy link
Contributor

seanaery commented Mar 6, 2026

@corylown I did -- mostly some obvious visible layout issues, but there's also probably some under-the-hood refactoring to do, like switching to using CSS custom properties (variables) instead of Sass variables. And now that BL9 fixes native advanced search, I want to ensure that a newly-generated arclight app's advanced search "just works." I'll aim to make some tickets today that include screenshots.

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.

2 participants