Closed kwiato closed 1 week ago
Feedback:
When navigating between pages in dark mode, the light mode briefly flashes for a few milliseconds. We can work on this as part of a separate PR. To evaluate if it's a blocker for releasing.
Review the footer in responsive mode:
Before:
It's important to clean up the codebase before the release:
We are currently defining colors in two separate files: base/_variables.scss and variables/_colors.scss. To improve maintainability, we should consolidate these into a single file. Additionally, we should remove any unused variables and colors.
Styles are only defined in this project for all projects, so it should be safe to update them as long as we test them. If you'd prefer not to address this in the current PR, we can handle it as part of a separate one.
Icons that are not using the new icons library:
Left sidebar expand button (fas fa-bars
):
Tables checks (fa fa-check
) and cross marks (fa fa-times
)
Hero box & topic documentation examples: /examples/hero-box
& /examples/topic-box
LGTM. Let's wait for @annastuchlik review before merging.
New issues derived from the changes in this PR:
To decide if we want to include them in the next release.
@kwiato @dgarcia360 Thank you :) We can now merge and move on :)
@dgarcia360 why cant I see dark node on the Theme site? https://sphinx-theme.scylladb.com/stable/
@tzach We need to merge this PR: https://github.com/scylladb/sphinx-scylladb-theme/pull/1216 We'll do it today. The new version contains major changes, so we have do release the new theme with separate PRs.
What I mean is that Dark Theme is part of the new theme that needs to be released yet.
Some comments: