Fix UI

[0x2E]

Fix: #152

[Copilot]

Pull Request Overview

This PR fixes UI issues (Fix: #152) by refactoring several page load functions and updating how global state is managed.

• Removed calls to depends() in individual page load functions across various routes.
• Updated the layout load to use Promise.all with new global state setter functions.
• Introduced new setter functions in the state module for managing feeds and groups.

Reviewed Changes

Copilot reviewed 9 out of 16 changed files in this pull request and generated no comments.

Show a summary per file

File	Description
frontend/src/routes/(authed)/search/+page.ts	Removed dependency declaration from the page load.
frontend/src/routes/(authed)/items/[id]/+page.ts	Removed dependency declaration from the page load.
frontend/src/routes/(authed)/groups/[id]/+page.ts	Removed dependency declaration from the page load.
frontend/src/routes/(authed)/feeds/[id]/+page.ts	Removed dependency declaration from the page load.
frontend/src/routes/(authed)/bookmarks/+page.ts	Removed dependency declaration from the page load.
frontend/src/routes/(authed)/all/+page.ts	Removed dependency declaration from the page load.
frontend/src/routes/(authed)/+page.ts	Removed dependency declaration from the page load.
frontend/src/routes/(authed)/+layout.ts	Updated to use new global state setters and added combined dependency declaration.
frontend/src/lib/state.svelte.ts	Added setter functions for feeds and groups and an unread count updater.

Files not reviewed (7)

• frontend/src/lib/components/ItemActionMarkAllasRead.svelte: Language not supported
• frontend/src/lib/components/ItemActionUnread.svelte: Language not supported
• frontend/src/lib/components/Sidebar.svelte: Language not supported
• frontend/src/routes/(authed)/+layout.svelte: Language not supported
• frontend/src/routes/(authed)/feeds/[id]/+page.svelte: Language not supported
• frontend/src/routes/(authed)/feeds/[id]/ActionMenu.svelte: Language not supported
• frontend/src/routes/(authed)/groups/[id]/+page.svelte: Language not supported

Comments suppressed due to low confidence (7)

frontend/src/routes/(authed)/search/+page.ts:4

• Removal of the 'depends' call here may lead to stale data if client cache invalidation relied on it. Ensure that the parent layout or another mechanism covers the necessary dependency tracking.

export const load: PageLoad = async ({ url }) => {

frontend/src/routes/(authed)/items/[id]/+page.ts:7

• The removal of the 'depends' call might affect automatic revalidation for this route; verify that this behavior is now handled at a higher level.

export const load: PageLoad = ({ params }) => {

frontend/src/routes/(authed)/groups/[id]/+page.ts:8

• Without the 'depends' call, consider whether dependency tracking for data updates is now fully managed by the parent layout; if not, this could result in stale group data.

export const load: PageLoad = async ({ url, params }) => {

frontend/src/routes/(authed)/feeds/[id]/+page.ts:7

• Ensure that removing 'depends' here does not break client-side cache invalidation or data refresh for the feed, as the global dependencies may need to compensate for this removal.

export const load: PageLoad = async ({ url, params }) => {

frontend/src/routes/(authed)/bookmarks/+page.ts:4

• Confirm that the elimination of the 'depends' call does not inadvertently affect the revalidation logic for bookmarks; ensure that any required dependency tracking is now handled elsewhere.

export const load: PageLoad = async ({ url }) => {

frontend/src/routes/(authed)/all/+page.ts:4

• Verify that the removal of the 'depends' call from this page load function does not compromise the intended refreshing of data, since dependency management may now be centralized.

export const load: PageLoad = async ({ url }) => {

frontend/src/routes/(authed)/+page.ts:4

• Removing the 'depends' call here could affect revalidation; please confirm that the global dependency handling in the layout suffices for UI update requirements.

export const load: PageLoad = async ({ url }) => {