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

[DRAFT] Fix/favicon and active tab #6

Closed
wants to merge 17 commits into from
Closed

Conversation

Huongg
Copy link
Collaborator

@Huongg Huongg commented Jun 25, 2024

Huong Nguyen added 2 commits June 24, 2024 15:16
Signed-off-by: Huong Nguyen <huongg1409@gmail>
Signed-off-by: Huong Nguyen <huongg1409@gmail>
@astrojuanlu
Copy link
Member

I created the kedro-sphinx-theme repo from a template that uses the RTD PR Preview action 👍🏼 but the RTD project doesn't exist 👎🏼 hence the broken link.

Nothing to do here, with the SEO problems we have kedro-org/kedro#3741 better to not add another website to the mix.

Signed-off-by: Huong Nguyen <huongg1409@gmail>
@Huongg Huongg changed the title Fix/favicon and active tab [DRATF} Fix/favicon and active tab Jun 25, 2024
@Huongg Huongg changed the title [DRATF} Fix/favicon and active tab [DRATF] Fix/favicon and active tab Jun 25, 2024
@Huongg Huongg changed the title [DRATF] Fix/favicon and active tab [DRAFT] Fix/favicon and active tab Jun 25, 2024
Huong Nguyen added 14 commits June 25, 2024 10:06
Signed-off-by: Huong Nguyen <huongg1409@gmail>
Signed-off-by: Huong Nguyen <huongg1409@gmail>
Signed-off-by: Huong Nguyen <huongg1409@gmail>
Signed-off-by: Huong Nguyen <huongg1409@gmail>
Signed-off-by: Huong Nguyen <huongg1409@gmail>
Signed-off-by: Huong Nguyen <huongg1409@gmail>
Signed-off-by: Huong Nguyen <huongg1409@gmail>
Signed-off-by: Huong Nguyen <huongg1409@gmail>
Signed-off-by: Huong Nguyen <huongg1409@gmail>
Signed-off-by: Huong Nguyen <huongg1409@gmail>
Signed-off-by: Huong Nguyen <huongg1409@gmail>
Signed-off-by: Huong Nguyen <huongg1409@gmail>
Signed-off-by: Huong Nguyen <huongg1409@gmail>
Signed-off-by: Huong Nguyen <huongg1409@gmail>
@@ -19,8 +19,14 @@
extensions = [
"myst_parser",
"sphinx_copybutton",
'sphinx_favicon',
Copy link
Member

Choose a reason for hiding this comment

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

I think this is correct! And the only thing missing is adding "sphinx-favicon" to pyproject.toml as was done in https://github.com/kedro-org/kedro/pull/3547/files

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

ah great, thanks for the suggestion, let me try that

@Huongg Huongg closed this Jun 26, 2024
@Huongg Huongg deleted the fix/favicon-and-active-tab branch June 27, 2024 15:20
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