Skip to content

Conversation

@M46N3
Copy link

@M46N3 M46N3 commented May 28, 2025

Added a new environment variable to support new dark mode functionality.

Fixes #198
Fixes #218

Updated docs PR docker-library/docs#2577

@M46N3
Copy link
Author

M46N3 commented Sep 8, 2025

Any chance this can be merged soon? Maybe together with upgrade to 5.4.0 #221?

@TimWolla
Copy link
Owner

TimWolla commented Sep 8, 2025

Hi! Thank you for the PR. Unfortunately I've not yet had the time to investigate this and I don't want to commit to a particular “easy” solution that might not be the best solution within the context of this Docker image.

@stasadev
Copy link

Wouldn't it be simpler to handle it like this?

if [ -n "$ADMINER_DESIGN" ]; then
	# Only create link on initial start, to ensure that explicit changes to
	# adminer.css after the container was started once are preserved.
	if [ ! -e .adminer-init ]; then
-		ln -sf "designs/$ADMINER_DESIGN/adminer.css" .
+		for css_file in adminer.css adminer-dark.css; do
+			if [ -f "designs/$ADMINER_DESIGN/$css_file" ]; then
+				ln -sf "designs/$ADMINER_DESIGN/$css_file" adminer.css
+				break
+			fi
+		done
	fi
fi

This avoids introducing an extra variable.

@M46N3
Copy link
Author

M46N3 commented Jan 22, 2026

I am not sure persisting changes to a theme is desired as a default? If personal changes are needed it could maybe be solved with mounting your own css?

The extra variable is required to give the user a choice between light and dark mode.

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.

Wrong CSS of ADMINER_DESIGN issue about ADMINER_DESIGN=dracula

3 participants