Adapt home and repositories pages from umich-arclight by ssciolla · Pull Request #22 · mlibrary/boxwalker

ssciolla · 2026-06-22T14:51:52Z

This PR aims to resolve JIRA ARC-135

Resources

…Arclight::Repository

…verriding templates for index, home, adding HomepageHelper and CatalogHelper, adding image config and files, adding um_arclight translation file

Arc 135 repositories view edits

…ion.bootstrap; fix spacing in _home.html.erb

…his Site row in _home.html.erb

…home Tweak home spacing and code style

…mb divide to base.scss

…_sim

ARC-134 refactor SCSS

ssciolla · 2026-06-22T15:03:04Z

-}
-
-// Arclight, imported from arclight frontend package
-@import 'arclight/app/assets/stylesheets/arclight/application';


Do we no longer need this, or was it moved elsewhere?

For now, I've left it out of our update with the assumption we don't need it. To make it easy for now there's just a flat directory, "stylesheets"

ssciolla · 2026-06-22T15:07:16Z

+
+  /* ---------------------- */
+  /* SEARCH BOX ON HOMEPAGE */
+  /* ---------------------- */


Remove the large comment or the search bar itself? Doesn't it make sense to keep it under "Find Archival Materials."

Yeah, sorry, I think I just meant remove the commented code above this, not the Search box stuff.

When CSS is minified in production all comments are removed. These comments are helpful in development.

rshiggin · 2026-06-22T17:01:26Z

I'd ideally like to remap color variables from Bootstrap to our design system variables. This would take a small chunk of code in bootstrap.scss.

@use 'bootstrap/scss/bootstrap' with (
  $primary: #1d7491,           // your teal-400
  $success: #20a848,           // your green-400
  $danger:  #d93838,           // your pink-400
  $warning: #ffcb05,           // your maize-400 (replaces dandelion!)
  $body-bg: #fff,
  $font-family-base: var(--font-base-family)
);

@use 'bootstrap-icons/font/bootstrap-icons';
@use "blacklight-frontend/app/assets/stylesheets/blacklight/blacklight";

Note: all the default Bootstrap 5 being pulled into our project is stored in node_modules/bootstrap.

rshiggin · 2026-06-22T17:10:04Z

re: SCSS deprecation warnings.

We're going to continue to see deprecation warnings from "@import" until "Project Blacklight" updates it's project to be fully Dart Sass compatible, which it will be forced to do with BS6.

Let's consider silencing these in package.json by adding "--quiet-deps"
"build:css:compile": "sass ... --no-source-map --load-path=node_modules --quiet-deps"

I've commited this f0eab7c

…nce file

…ect not fully compatible with Dart Sass

Update and pare down helpers

…use of old @import spec

ssciolla · 2026-06-22T19:44:49Z

+      }
+    }
+
+    &.banner-shadow {


I think I removed a shadow block from homepage.scss for imgback. Not sure if that's a big deal but it didn't seem to be needed...

Probably not a big deal. I don't think the shadow for imgback is in the template.

For the banner the shadow does the angled transition between darker yellow and lighter.

…o homepage; remove ask a librarian, comment out alert in translation file

rshiggin and others added 30 commits June 12, 2026 14:28

Add thumbnails and repositories.yml from current findingaids

bff93f0

Begin migrating repository-related views from findingaids

0d258fa

Change index view to three columns

1773bd0

Fix missing city_state_zip_country by creating a helper, not opening …

a76277a

…Arclight::Repository

Start wiring up homepage by switching root to catalog#index, adding o…

6a10435

…verriding templates for index, home, adding HomepageHelper and CatalogHelper, adding image config and files, adding um_arclight translation file

Merge pull request #16 from mlibrary/ARC-135-repositories-view-edits

f92e821

Arc 135 repositories view edits

layout and style for homepage

2a0f8b1

Add more missing pieces for homepage

f953f63

Move sidebar into has query branch of conditional for now

041ce8c

Fix image position on homepage by adding relative to BS5 col-md-12

e07dfb0

Adding footer style into compiled SCSS

982a6b3

Move up closing div; indent some things

624a373

Add missing padding; remove rl-menu-title

6238490

Use viewport-container in app/views/layouts/blacklight/base.html.erb

b9bebd5

Use row instead of container-fluid

8bd6497

Add card.scss with padding and link title, and add import to applicat…

e5eea44

…ion.bootstrap; fix spacing in _home.html.erb

Add newline at EOF in card.scss; remove full-width class from About t…

6033a03

…his Site row in _home.html.erb

Make alerts a separate conditional row instead of nesting rows

7f2a307

mr -> me

4387001

Merge pull request #17 from mlibrary/ARC-135-repositories-view-tweak-…

ee2cf9c

…home Tweak home spacing and code style

Configure separate compiled stylesheets

2658ffa

Create files for compiling bootstrap.css and application.css

159fc7a

Delete arclight.scss as redundant and move its one rule for bread-cru…

9542273

…mb divide to base.scss

Add search box to home and navbar

626ff4c

Add some basic indexing instructions to README

d7812f6

CSS build pipeline

49d0320

Add title to arclight.en.yml

d0e21e5

Change collection_home_count query to use level_ssim instead of level…

06afd27

…_sim

Fix card height and image spacing in repositories view

3c5f01d

Merge pull request #20 from mlibrary/ARC-134-refactor-scss

31ee8b9

ARC-134 refactor SCSS

ssciolla commented Jun 22, 2026

View reviewed changes

Comment thread app/assets/stylesheets/application.scss Outdated

rshiggin added 2 commits June 22, 2026 12:39

Update format to Dart Sass

50eb012

Refactor bootstrap variables; replaces dandelion with maize variants

4709d02

rshiggin and others added 15 commits June 22, 2026 13:15

New colors.scss file with Library's design system tokens

ee7bc3a

Moves BS configs and functions for custom SCSS into standalone refere…

db6d891

…nce file

Silence css:compile deprecation notices in default Bootstrap builds

f0eab7c

Rollback changes to Bootstrap compile to use old version; parent proj…

644664c

…ect not fully compatible with Dart Sass

Delete old version of file

562926c

Add back _homepage.scss, refactoring and trimming down

04329dc

Use Rails config_for to load settings.yml and fix ContactLinkHelper

c8c0307

Pare down and rename DulArclightHelper to UMArclightHelper

35eb162

Remove extra margin under homepage warning banner

10afd59

UMArclightHelper -> UmArclightHelper

dad7e7b

Remove deprecated warning types and unused buttons

cdac1b3

Add newlines at EOFs

6801b43

Please rubocop

bf24386

Merge pull request #23 from mlibrary/ARC-135-repositories-view-helpers

e2babb9

Update and pare down helpers

Add additional flag to silence deprecation warnings for Blacklight's …

d022e42

…use of old @import spec

ssciolla commented Jun 22, 2026

View reviewed changes

ssciolla and others added 4 commits June 22, 2026 17:22

Add newlines at EOFs; move homepage search button styling from base t…

6c6cb56

…o homepage; remove ask a librarian, comment out alert in translation file

Change comment in _header_navbar.html.erb

0cde545

Add newlines at EOFs; tweak some comments

e212ecc

Add updated Brakeman gem to Gemfile

8dba734

rshiggin changed the title ~~Adopt home and repositories pages from umich-arclight, refactoring/updating as needed~~ Adapt home and repositories pages from umich-arclight Jun 23, 2026

ssciolla marked this pull request as ready for review June 23, 2026 15:21

rshiggin approved these changes Jun 23, 2026

View reviewed changes

ssciolla merged commit 6e26ade into main Jun 23, 2026
5 checks passed

Uh oh!

Conversation

ssciolla commented Jun 22, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

ssciolla Jun 22, 2026

Choose a reason for hiding this comment

Uh oh!

rshiggin Jun 22, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

ssciolla Jun 22, 2026

Choose a reason for hiding this comment

Uh oh!

rshiggin Jun 22, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

ssciolla Jun 22, 2026

Choose a reason for hiding this comment

Uh oh!

rshiggin Jun 22, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

rshiggin commented Jun 22, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

rshiggin commented Jun 22, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

ssciolla Jun 22, 2026

Choose a reason for hiding this comment

Uh oh!

rshiggin Jun 22, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

ssciolla commented Jun 22, 2026 •

edited

Loading

rshiggin Jun 22, 2026 •

edited

Loading

rshiggin commented Jun 22, 2026 •

edited

Loading

rshiggin commented Jun 22, 2026 •

edited

Loading