Skip to content

More darkmode examples#38058

Merged
mdo merged 15 commits intomainfrom
more-darkmode-examples
Apr 28, 2023
Merged

More darkmode examples#38058
mdo merged 15 commits intomainfrom
more-darkmode-examples

Conversation

@mdo
Copy link
Copy Markdown
Member

@mdo mdo commented Feb 15, 2023

Tackles a few more examples. Thought I opened this previously, but apparently not.

@mdo mdo requested a review from a team as a code owner February 15, 2023 02:40
Copy link
Copy Markdown
Member

@julien-deramond julien-deramond left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Few quick comments. I haven't done the complete review, I'll continue later.

site/content/docs/5.3/examples/dashboard-rtl/index.html and site/content/docs/5.3/examples/blog-rtl/index.html should be updated as well to reflect site/content/docs/5.3/examples/dashboard/index.html and site/content/docs/5.3/examples/blog/index.html modifications.

@mdo mdo force-pushed the more-darkmode-examples branch from b987bf1 to 3babf1e Compare April 26, 2023 04:38
@julien-deramond
Copy link
Copy Markdown
Member

Minimal modifications have been done for blog-rtl/index.html via a0fb9ef.
There are discrepancies in terms of content (missing content in the RTL version) but it comes from previous modifications. Can be tackled in another PR.
Used Google Translate for the new content in "Recent Posts".

@julien-deramond
Copy link
Copy Markdown
Member

The same work has been done for dashboard-rtl/index.html via e01664e.
I hope that I haven't forgotten anything.
For missing translations, used Google Translate as well.

@julien-deramond
Copy link
Copy Markdown
Member

julien-deramond commented Apr 26, 2023

Global comment to take into account here or in a separate PR: because of include_js: falses that are in some examples, the theme selector can't be used. So maybe drop all include_js: false + handle differently (or remove) {{ if ne .Page.Params.include_js false -}} in site/layouts/_default.

If done here, #38512 can be closed.

@mdo mdo force-pushed the more-darkmode-examples branch from 4fca984 to c9e57bc Compare April 27, 2023 23:55
@mdo mdo requested a review from a team as a code owner April 27, 2023 23:55
@mdo mdo merged commit e468daa into main Apr 28, 2023
@mdo mdo deleted the more-darkmode-examples branch April 28, 2023 01:13
romankupchak93 pushed a commit to romankupchak93/bootstrap that referenced this pull request Jan 5, 2024
* WIP darkmode examples

* Fix product css error, redo parts of blog

* wip dashboard

* updated rtl generated

* fix up some examples, namely dashboard

* jumbotron and sign-in fixes

* more fixes

* more

* Fixes from code review

* Remove duplicate `data-bs-target` in dashboard/index.html

* Modify `blog-rtl/index.html`

* Modify `dashboard-rtl/index.html`

* Modify `carousel-rtl/index.html`

* Keep JS on all pages for switcher

* Remove Feather from dashboard examples

---------

Co-authored-by: Julien Déramond <juderamond@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

No open projects
Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants