Skip to content

Conversation

@yoonjs2
Copy link
Contributor

@yoonjs2 yoonjs2 commented Jun 12, 2016

What is this PR for?

In documentation about storage, a custom hashtag navigation used to nav to each paragraph but scrolling is not working properly (it cannot fully scroll to headers of paragraph) when clicking each menu at nav bar.

But after PR #995 by @AhyoungRyu, custom hashtag nav can be changed into anchor.js style so I fixed nav for documentation to use this anchor feature to make it scroll properly.

What type of PR is it?

Bug Fix, Documentation

Todos

How should this be tested?

  1. Apply this patch
  2. Build docs site using jekyll and browse localhost:4000
  3. Click top nav -> Click "More" -> Click "Git storage", "S3 Storage", and so on.

Screenshots (if appropriate)

before
after

Questions:

  • Does the licenses files need update? No
  • Is there breaking changes for older versions? No
  • Does this needs documentation? No

# Conflicts:
#	docs/interpreter/cassandra.md
# Conflicts:
#	docs/interpreter/cassandra.md
# Conflicts:
#	docs/interpreter/cassandra.md
# Conflicts:
#	docs/interpreter/cassandra.md
#	docs/interpreter/spark.md
#	docs/manual/interpreters.md
# Conflicts:
#	dev/merge_zeppelin_pr.py
#	docs/quickstart/explorezeppelinui.md
#	zeppelin-server/src/main/java/org/apache/zeppelin/rest/LoginRestApi.java
#	zeppelin-web/.jshintrc
#	zeppelin-web/src/app/notebook/notebook.controller.js
#	zeppelin-web/src/app/notebook/paragraph/paragraph.controller.js
#	zeppelin-web/src/components/navbar/navbar.controller.js
#	zeppelin-web/src/components/navbar/navbar.html
# Conflicts:
#	docs/assets/themes/zeppelin/js/docs.js
@minahlee
Copy link
Member

I just tried #1004 and it scroll to expected position. Since #1004 is using a lot of anchor, would you mind to close this pr so we can reduce burden of changing every anchor that is used in #1004?

@yoonjs2
Copy link
Contributor Author

yoonjs2 commented Jun 15, 2016

@minahlee No problem :)

@yoonjs2 yoonjs2 closed this Jun 15, 2016
@yoonjs2 yoonjs2 deleted the fix-navigation-for-storage-doc branch June 18, 2016 08:42
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