The light/dark components has problems #5

Closed
opened 2022-05-22 06:32:24 +00:00 by emacsen · 4 comments
emacsen commented 2022-05-22 06:32:24 +00:00 (Migrated from gitlab.com)

The light/dark component has a few issues that need to get fixed before the site's launch.

The first and most obvious is that it uses Javascript, which might not be an issue on its own other than we promise no Javascript.

Even if we remove this promise, though, the selection doesn't persist across pages, likely because there's no session cookie or other setting. There should not be one, though, as we don't need session cookies for a static site. Instead, a second set of pages could be generated based on light/dark themes, just like we will do for languages.

Lastly, the component is unlabeled, where usually there is a symbol such as a sun and moon to indicate light/dark mode.

The light/dark component has a few issues that need to get fixed before the site's launch. The first and most obvious is that it uses Javascript, which might not be an issue on its own other than we promise no Javascript. Even if we remove this promise, though, the selection doesn't persist across pages, likely because there's no session cookie or other setting. There should not be one, though, as we don't need session cookies for a static site. Instead, a second set of pages could be generated based on light/dark themes, just like we will do for languages. Lastly, the component is unlabeled, where usually there is a symbol such as a sun and moon to indicate light/dark mode.
emacsen commented 2022-05-22 06:32:24 +00:00 (Migrated from gitlab.com)

assigned to @vosaul

assigned to @vosaul
vosaul commented 2022-05-23 13:20:08 +00:00 (Migrated from gitlab.com)

mentioned in merge request !2

mentioned in merge request !2
vosaul commented 2022-05-23 19:39:25 +00:00 (Migrated from gitlab.com)

mentioned in merge request !3

mentioned in merge request !3
emacsen commented 2022-06-23 05:36:18 +00:00 (Migrated from gitlab.com)

The light/dark mode still has issues, but I'll open a ticket for each one and close this.

The light/dark mode still has issues, but I'll open a ticket for each one and close this.
emacsen (Migrated from gitlab.com) closed this issue 2022-06-23 05:36:18 +00:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
babka/babka-social-blog#5
No description provided.