Skip to content
This repository was archived by the owner on Mar 29, 2025. It is now read-only.
This repository was archived by the owner on Mar 29, 2025. It is now read-only.

Move sorting preference to cookie to avoid SSR flash #219

Description

@leeoocca

Ok, so the problem with the sorting jump that you described is due to the fact that Remix renders the page server-side first, where it has no access to localStorage.

I'd propose to

  1. keep the scope for this PR on the new column
  2. create another PR to address
  • table styling (maybe also Votes column is too wide #160)
  • move from localstorage to cookies (bc also available server-side) or query params (not as persistent as localStorage, but shareable)

What do you think?

Originally posted by @leeoocca in #213 (review)

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions