Skip to content

FR: Support dark mode for zoom-image plugin #2508

Closed

Description

@wohenbushuang

Feature request

Problem or desire

Support dark mode for zoom-image plugin

Proposal

Support dark mode for zoom-image plugin

Implementation

Support dark mode for zoom-image plugin, instead of fixed white background

Activity

  1. added this to the 5.x milestone on Nov 12, 2024
  2. BanerjeeAgniva commented on Dec 20, 2024

    @BanerjeeAgniva

    @sy-records Hey I am new to open-source and coding, can you tell me what files I need to look into for this issue, thanks!

  3. sy-records commented on Dec 23, 2024

    @sy-records
    Member

    Hi, @BanerjeeAgniva

    You can see https://github.com/docsifyjs/docsify/blob/develop/src/plugins/zoom-image.js

    In Dark Mode, a few style changes will do the trick.

  4. ZEviod commented on Dec 23, 2024

    @ZEviod

    Hello @sy-records, I’d like to work on adding this feature.
    If yes, could you please clarify a few details to ensure alignment?

    ->Should the dark mode support be automatic (based on system preferences) or configurable via plugin options?
    ->Are there any existing guidelines, color schemes, or design patterns for dark mode in this project?
    ->Would you prefer dark mode styling to be implemented using CSS variables, a separate stylesheet, or another approach?

  5. sy-records commented on Dec 24, 2024

    @sy-records
    Member

    Should the dark mode support be automatic (based on system preferences) or configurable via plugin options?

    Automated, judgment by media query. prefers-color-scheme: dark

    Are there any existing guidelines, color schemes, or design patterns for dark mode in this project?

    We support dark mode in the new version, see https://github.com/docsifyjs/docsify/deployments/Preview , https://docsify-preview-hshwchgkk-docsifyjs.vercel.app/preview/#/themes?id=themes

    Would you prefer dark mode styling to be implemented using CSS variables, a separate stylesheet, or another approach?

    I think adding some styling to zoom-image.js will do the trick!

  6. trusktr commented on Dec 25, 2024

    @trusktr
    Member

    @jhildenbiddle any preferences here that may be best aligned with Docsify and docsify-themeable?

  7. self-assigned this
    on Jan 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Type

No type

Projects

No projects

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions