Skip to content

Conversation

@zoidyzoidzoid
Copy link
Contributor

@zoidyzoidzoid zoidyzoidzoid commented Jul 17, 2017

Fixes #322, fixes #3992

Visible here: https://kubernetes-docs.zoidbergwill.com/docs/home/

To do:

  • Make the dropdown pretty
  • Fix the versioned docs URLs in the dropdown, once we know how many versions of docs back we wanna host?

This change is Reviewable

@k8s-ci-robot k8s-ci-robot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label Jul 17, 2017
@pwittrock
Copy link
Member

/unassign

@pwittrock
Copy link
Member

/assign @chenopis

@ahmetb
Copy link
Member

ahmetb commented Jul 18, 2017

@zoidbergwill do you mind deleting your kubernetes docs site? These get indexed on search engines and compete with the kubernetes.io website. Your Pull Request automatically gets a Netlify site if you click "Show all checks".

@zoidyzoidzoid
Copy link
Contributor Author

@ahmetb In my branches I have https://github.com/zoidbergwill/kubernetes.github.io/commit/f3c8c3c8a801a9112ca38d6ddc4aea9ad8689231 and I was using them to test before I created the PR, but they're deleted now.

@ahmetb
Copy link
Member

ahmetb commented Jul 18, 2017

Ah no worries, then.

@chenopis
Copy link
Contributor

@zoidbergwill Thanks for working on this! Can you do me a favor and check the "Allow edits from maintainers" (see image below) so that I can make changes? I want to switch the URL paths to the subdomains I am setting up.

allow-maintainers-to-make-edits-sidebar-checkbox

@zoidyzoidzoid
Copy link
Contributor Author

@chenopis Done, sorry, thought I'd done that.

@chenopis
Copy link
Contributor

@zoidbergwill No worries. Thanks!

@chenopis chenopis added the do-not-merge DEPRECATED. Indicates that a PR should not merge. Label can only be manually applied/removed. label Jul 27, 2017
@chenopis
Copy link
Contributor

@zoidbergwill It works for me, but I think there needs to be some sort of background for the dropdown so that the version numbers don't get drowned out. This is what it looks like for me:

Top of Page

screen shot 2017-07-27 at 10 23 42 am

Mid-page Scroll

screen shot 2017-07-27 at 10 23 56 am

@chenopis chenopis mentioned this pull request Jul 27, 2017
4 tasks
@k8sio-netlify-preview-bot
Copy link
Collaborator

k8sio-netlify-preview-bot commented Aug 5, 2017

Deploy preview ready!

Built with commit 9d81192

https://deploy-preview-4386--kubernetes-io-master-staging.netlify.com

@chenopis chenopis added Docs Review: Open Issues and removed do-not-merge DEPRECATED. Indicates that a PR should not merge. Label can only be manually applied/removed. labels Aug 7, 2017
@chenopis chenopis changed the title WIP: Version Selector Version Selector Aug 8, 2017
@chenopis
Copy link
Contributor

chenopis commented Aug 8, 2017

@zoidbergwill I'm going to implement this in the other release branches before I release this on master.

@chenopis chenopis added the do-not-merge DEPRECATED. Indicates that a PR should not merge. Label can only be manually applied/removed. label Aug 8, 2017
@steveperry-53
Copy link
Contributor

@zoidbergwill, @chenopis, When you scroll down on a page, the background color of the dropdown changes from black to white. Is that the intention? The white background doesn't do a very good job of separating the dropdown from the page content. Seems like keeping the black background would be better.

@zoidyzoidzoid
Copy link
Contributor Author

@chenopis Awesome, I was thinking about doing some active/hover stuff, but it's difficult with how the padding and positioning currently works.

@zoidyzoidzoid
Copy link
Contributor Author

@steveperry-53 It's what the navbar does and is just matching that. Because of how the positioning and padding stuff is currently happening, it's hard to add a box-shadow that would look good.

I've played with it a bit here, but I think it'd be difficult to get it to match the navbar properly:
image

I could maybe move it up a bit though and make it look like border/shadow just around the dropdown.

I also tried to do hover colour changing over the links in the dropdown but they look a bit gross because of the same padding stuff.

@steveperry-53
Copy link
Contributor

@zoidbergwill, A border would be nice to have, but I'd say the dropdown is also OK as is. Thanks for your work on this.
/lgtm

@chenopis chenopis added this to the 2017Q3 milestone Aug 10, 2017
@chenopis chenopis removed the do-not-merge DEPRECATED. Indicates that a PR should not merge. Label can only be manually applied/removed. label Aug 10, 2017
@chenopis
Copy link
Contributor

I'm deploying this now.

@chenopis chenopis merged commit fa691da into kubernetes:master Aug 10, 2017
@chenopis
Copy link
Contributor

closes #3992

@chenopis
Copy link
Contributor

closes #322

@zoidyzoidzoid zoidyzoidzoid deleted the versioned-docs branch August 27, 2017 15:14
jesscodez pushed a commit that referenced this pull request Sep 22, 2017
* Show current version of docs in nav

* Branch in edit link

* WIP version selector

* An actual working but not pretty version dropdown

* update URLs for versions

v1.7: https://kubernetes.io/docs/home/
v1.6: https://v1-6.docs.kubernetes.io/docs/home/
v1.5: https://v1-5.docs.kubernetes.io/docs/home/
v1.4: https://v1-4.docs.kubernetes.io/docs/home/

* fix v1.4 and v1.5 paths

* Tidier drop downs

* Make font visible always

* Make background match main nav always

* change to major.minor version number

* show only major.minor version in drop down list

* switch from githubbranch to docsbranch

* update githubbranch

* update githubbranch values

* Border radius for bottom of dropdown

* Tabs are hard
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cncf-cla: yes Indicates the PR's author has signed the CNCF CLA.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Versioned Docs Discussion We should be able to switch between docs versions

8 participants