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, not ready] Update docs front page following design review #1617

Merged
merged 13 commits into from
Nov 8, 2023

Conversation

stichbury
Copy link
Contributor

@stichbury stichbury commented Oct 31, 2023

Description

This PR forms part of the ongoing work to enable a documentation subproject for Kedro-Viz. See #1524 for more details (and the kedro-org/kedro#2600).

Development notes

Updated the docs front page following design review in #1592.

  • Added graphic used by README.md on the repo
  • Revised an emoji
  • Added a link to the live demo
  • Changes to the top left corner from @tynandebold

Currently waiting to make a final chunk of changes to align this PR with meganav improvements as suggested by Steph in design review. So this is not yet ready to merge.

QA notes

Rebuilt docs and inspected

Checklist

  • Read the contributing guidelines
  • Opened this PR as a 'Draft Pull Request' if it is work-in-progress
  • Updated the documentation to reflect the code changes
  • Added new entries to the RELEASE.md file
  • Added tests to cover my changes

Signed-off-by: Jo Stichbury <[email protected]>
@stichbury stichbury marked this pull request as ready for review November 1, 2023 15:52
@stichbury stichbury marked this pull request as draft November 1, 2023 15:55
Copy link
Member

@tynandebold tynandebold left a comment

Choose a reason for hiding this comment

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

Looking great. Three comments from me, none of which I consider to be blocking, so I'll approve now.

docs/source/index.md Show resolved Hide resolved
docs/source/index.md Outdated Show resolved Hide resolved
docs/source/conf.py Outdated Show resolved Hide resolved
@stichbury stichbury changed the base branch from main to viz-docs-update November 8, 2023 14:09
@stichbury stichbury marked this pull request as ready for review November 8, 2023 14:11
@stichbury
Copy link
Contributor Author

I'm closing this PR by merging it into my other changes in #1628.

@stichbury stichbury merged commit 44bec27 into viz-docs-update Nov 8, 2023
2 of 4 checks passed
@stichbury stichbury deleted the update-docs-front-page branch November 8, 2023 14:12
stichbury added a commit that referenced this pull request Nov 13, 2023
* something to start off

Signed-off-by: Jo Stichbury <[email protected]>

* Fix permalink icon

Signed-off-by: Juan Luis Cano Rodríguez <[email protected]>

* Revise some links to framework docs

Signed-off-by: Jo Stichbury <[email protected]>

* Add live tracking demo to open in new tab

Signed-off-by: Jo Stichbury <[email protected]>

* Remove code to load old CSS

Signed-off-by: Jo Stichbury <[email protected]>

* [Draft, not ready] Update docs front page following design review (#1617)

* Update docs front page

Signed-off-by: Jo Stichbury <[email protected]>

* Remove nav_version next to Kedro logo in top left; add Kedro logo

Signed-off-by: Tynan DeBold <[email protected]>

* Revise project name and author name

Signed-off-by: Jo Stichbury <[email protected]>

* Use full kedro icon and type for logo

Signed-off-by: Tynan DeBold <[email protected]>

* Add mobile nav close btn

* Move code for adding btn to event listener

* Add css for mobile nav image background

* Update close button for mobile nav

---------

Signed-off-by: Jo Stichbury <[email protected]>
Signed-off-by: Tynan DeBold <[email protected]>
Co-authored-by: Tynan DeBold <[email protected]>
Co-authored-by: Vladimir <[email protected]>
Co-authored-by: vladimir-mck <[email protected]>

* A few content improvements

Signed-off-by: Jo Stichbury <[email protected]>

* Update layout.html

* Update layout.html

* Delete docs/source/kedro-horizontal-color-on-light.svg

* Update conf.py

* Update layout.html

Add datasets subproject link

* Update Shareable Viz docs with sections on creating access key and billing. (#1631)

* Update shareable viz docswith sections on creating access key and billing

Signed-off-by: Nero Okwa <[email protected]>

* Update docs/source/share_kedro_viz.md

Co-authored-by: Tynan DeBold <[email protected]>

* Update docs/source/share_kedro_viz.md

Co-authored-by: Tynan DeBold <[email protected]>

* Update docs/source/share_kedro_viz.md

Co-authored-by: Tynan DeBold <[email protected]>

* Update docs/source/share_kedro_viz.md

Co-authored-by: Tynan DeBold <[email protected]>

* Update docs/source/share_kedro_viz.md

Co-authored-by: Tynan DeBold <[email protected]>

* Update docs/source/share_kedro_viz.md

Co-authored-by: Tynan DeBold <[email protected]>

* Update docs/source/share_kedro_viz.md

Co-authored-by: Tynan DeBold <[email protected]>

* Update share_kedro_viz.md

---------

Signed-off-by: Nero Okwa <[email protected]>
Co-authored-by: Tynan DeBold <[email protected]>
Co-authored-by: Jo Stichbury <[email protected]>

* Update docs/source/conf.py

* Update docs/source/preview_datasets.md

Co-authored-by: Juan Luis Cano Rodríguez <[email protected]>

* Update docs/source/visualise_charts_with_matplotlib.md

Co-authored-by: Juan Luis Cano Rodríguez <[email protected]>

* Update docs/source/share_kedro_viz.md

Co-authored-by: Juan Luis Cano Rodríguez <[email protected]>

* Update docs/source/visualise_charts_with_plotly.md

Co-authored-by: Juan Luis Cano Rodríguez <[email protected]>

* Update docs/source/preview_datasets.md

Co-authored-by: Juan Luis Cano Rodríguez <[email protected]>

* Suggested variable removal

Signed-off-by: Jo Stichbury <[email protected]>

* Fix links

Signed-off-by: Jo Stichbury <[email protected]>

---------

Signed-off-by: Jo Stichbury <[email protected]>
Signed-off-by: Juan Luis Cano Rodríguez <[email protected]>
Signed-off-by: Tynan DeBold <[email protected]>
Signed-off-by: Nero Okwa <[email protected]>
Co-authored-by: Juan Luis Cano Rodríguez <[email protected]>
Co-authored-by: Tynan DeBold <[email protected]>
Co-authored-by: Vladimir <[email protected]>
Co-authored-by: vladimir-mck <[email protected]>
Co-authored-by: Nero Okwa <[email protected]>
Co-authored-by: Juan Luis Cano Rodríguez <[email protected]>
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.

3 participants