Skip to content

feat(lxl-web): Revised startpage and improved appbar accessibility - #1646

Merged
johanbissemattsson merged 29 commits into
developfrom
feat-lxlweb-revised-start-and-improved-appbar-accessibility
Jun 25, 2026
Merged

feat(lxl-web): Revised startpage and improved appbar accessibility#1646
johanbissemattsson merged 29 commits into
developfrom
feat-lxlweb-revised-start-and-improved-appbar-accessibility

Conversation

@johanbissemattsson

@johanbissemattsson johanbissemattsson commented Jun 22, 2026

Copy link
Copy Markdown
Contributor

Revised PR replacing #1635

Description

Solves

A visual refresh of the start page (adding a hero image and refining styling) and improved accessibility of the app bar (changes to the expanded search is saved for a later PR).

Skärmavbild 2026-06-23 kl  11 52 12

Summary of changes

  • Revised start page with hero image and refined styling
  • Prevent layout shifting on start page
  • Improve accessibility of the app bar
    • Better tab-ordering
    • Larger app bar contents on larger breakpoints
    • Visible labels on smaller breakpoints
  • Update search context
  • Subpath imports for app.css
  • Remove gradient on featured lists
  • Add support for rendering search result lists as grids
  • Prevent flickering expanded search when resetting
  • Add skip to content component
  • Remove autfocus for now which could cause some accessibility issues
  • Added @sveltejs/enhanced-img for optimized hero images-
  • Keep query when navigating (reverting change in feat(lxl-web): Redesigned start page with hero image #1635)
  • Keep search field visible on /find routes (reverting change in feat(lxl-web): Redesigned start page with hero image #1635)

@jesperengstrom

Copy link
Copy Markdown
Contributor

Nice! A smooth experience now I think. Liking that the flicker is removed 👍

Discovered some minor bugs/visual regressions. Will list them below. Would not say that any of them is blocking?

@jesperengstrom

jesperengstrom commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

When you delete pills from mobile modal menu and close it, it's still there until reload. A regression.
gif5

@jesperengstrom

jesperengstrom commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

A bit crammed "special collections" section on frontpage + 'explore' button is on top of text
.
Image (2)

@jesperengstrom

jesperengstrom commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

Vertical scroll on some cards on front page cuts last line
.
Image (3)

@jesperengstrom

Copy link
Copy Markdown
Contributor

Fjärrlån menu item and button no longer match in color. Black text on green is not sufficient color contrast
.
Skärmavbild 2026-06-23 kl  15 00 49

@johanbissemattsson

johanbissemattsson commented Jun 23, 2026

Copy link
Copy Markdown
Contributor Author

Would not say that any of them is blocking?
When you delete pills from mobile modal menu and close it, it's still there until reload. A regression.

Good find, I would say that first one is blocking so I fixed it in 16c4c0. EDIT: 175c5a5

I agree with the others but they are more aesthetical so they could probably be saved for later.

@johanbissemattsson

Copy link
Copy Markdown
Contributor Author

Fjärrlån menu item and button no longer match in color. Black text on green is not sufficient color contrast

My bad, this is also fixed in 9a52c0e

@olovy

olovy commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

Still:

vertical scroll on start page
Skärmbild från 2026-06-24 12-18-07

below 1000 px width
Skärmbild från 2026-06-24 12-08-57

above 1000 px width - button is a bit crammed?
Skärmbild från 2026-06-24 12-18-49

@johanbissemattsson

Copy link
Copy Markdown
Contributor Author

Still:

vertical scroll on start page

below 1000 px width

above 1000 px width - button is a bit crammed?

These should be fixed in 69febcb and 61c9732.

@johanbissemattsson
johanbissemattsson merged commit 580a390 into develop Jun 25, 2026
3 checks passed
@johanbissemattsson
johanbissemattsson deleted the feat-lxlweb-revised-start-and-improved-appbar-accessibility branch June 25, 2026 06:40
olovy added a commit that referenced this pull request Aug 19, 2026
Restores the behavior from before PR #1646 (580a390)
The whole start page should still be displayed when subset (_r) is set
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.

3 participants