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

Update site to improve navigation #68

Merged
merged 6 commits into from
Sep 29, 2020
Merged

Conversation

anik120
Copy link
Collaborator

@anik120 anik120 commented Sep 28, 2020

Improves navigation within the site by:

  • Expanding the section navigation menu (on the left)
  • Including the page navigation menu (on the right)
  • Reducing the size of description for individual docs to make them cleaner

Improves navigation to SDK site by:

  • Including the Build menu from the sdk site as a top level menu

@openshift-ci-robot openshift-ci-robot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Sep 28, 2020
@anik120 anik120 changed the title [WIP] Change Menu and navigation Update site to improve navigation Sep 29, 2020
@openshift-ci-robot openshift-ci-robot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Sep 29, 2020
<section class="of-masthead of-masthead--subpage">
<h1 class="of-heading of-heading--xl">About OLM</h1>
<h1 class="of-heading of-heading--xl">Building Operators using the Operator SDK</h1>
Copy link
Member

Choose a reason for hiding this comment

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

is this intentional? converting the copy over to sdk?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@ecordell yes. The about section wasn't doing much for us, it just seemed like a complete waste of space we can utilize instead to draw attention to the other of project.

Copy link
Member

@ecordell ecordell left a comment

Choose a reason for hiding this comment

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

/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Sep 29, 2020
@anik120 anik120 merged commit 7ed0934 into operator-framework:master Sep 29, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm Indicates that a PR is ready to be merged.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants