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

toc2: update for html export + notebook menus set to the left #803

Merged
merged 2 commits into from
Nov 28, 2016

Conversation

jfbercher
Copy link
Member

  • Since previous PR, the highlighting colors for the sections in toc with selected/edited/running cells are user configurable and are set directly by javascript (config is read from system). However, when exporting to html, the style is not defined. This PR simply adds a default config (for hover highlighting) which is used by documents exported to html and overwritten in the live notebook.

  • In Centered title/menus when using toc2-sidebar view looks odd #650, @Kevin-McIsaac suggested to move title/menus/icons on left.
    Indeed, since the introduction of the toc sidebar, the toc2 extension resizes the main container to fit the whole window and add the sidebar. Centered menus then sometimes look odd. I agree with the suggestion. The PR adds a small styling in toc css to move the menu to the left. However, I am not sure if it should be done here, or left to the users, or put as a configuration option. Thoughts?

@jfbercher
Copy link
Member Author

jfbercher commented Nov 27, 2016

Moving title/menus/icons to the left:

However, I am not sure if it should be done here, or left to the users, or put as a configuration option. Thoughts?

Finally, I have put this as a user configurable parameter (with a default to true, that is "move")

@jcb91
Copy link
Member

jcb91 commented Nov 28, 2016

I have put this as a user configurable parameter (with a default to true, that is "move")

sounds good to me!

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