Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[1.5.x] hugo modules (bootstrap included) #167

Merged
merged 2 commits into from
Jan 24, 2025

Conversation

zetxek
Copy link
Owner

@zetxek zetxek commented Jan 23, 2025

Note PS: the version was meant to be 2.0.0 but due to complications with go modules, the version has become 1.5.0.
See #171 for more information.

As mentioned in #158 (reply in thread), I'm leaning more to keeping bootstrap in the theme - to make it easier to use.

This PR is a "else" for #166, another version of how the theme can support hugo modules without removing bootstrap - therefore keeping compatibility with git submodules, as well as "download, copy and play".

I made sure that the theme files are used by overriding/playing with the color:
2025-01-23 19 30 29

Aside of that, respect #166 , the other change is a bit of update on the README.

- Commented out Bootstrap and Popper.js module imports in config.toml and hugo.toml to address discussions on theme usage.
- Enhanced README.md to improve clarity on theme features, installation methods, and Bootstrap integration, emphasizing accessibility and performance improvements.
@zetxek zetxek requested review from mnordhaus and dcorto January 23, 2025 12:34
@zetxek zetxek merged commit 0108e15 into 2.x-hugo-modules Jan 24, 2025
1 check passed
@zetxek zetxek deleted the 2.x-hugo-modules-bootstrap-included branch January 24, 2025 17:06
@zetxek
Copy link
Owner Author

zetxek commented Jan 24, 2025

Merging to #166 - I think that to modularize the repo we can go in smaller steps, and see what makes sense for the best compatibility :)

@zetxek zetxek changed the title 2.x hugo modules (bootstrap included) [1.5.x] hugo modules (bootstrap included) Jan 26, 2025
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.

2 participants