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

Prevent "UP | HOME" links from being hidden by TOC #152

Merged
merged 1 commit into from
Jul 27, 2022

Conversation

tfree87
Copy link
Contributor

@tfree87 tfree87 commented Jul 21, 2022

I finally created a pull request to fix the issue I posted (#150).

This change in the CSS prevents the HOME and UP links from being hidden when using ReadTheOrg theme.

I tested this change in the CSS and verified that it works by adding the following to my Org files:

#+OPTIONS: html-style:nil
#+HTML_HEAD: <link rel="stylesheet" type="text/css" href="https://tfree87.github.io/org-html-themes/src/readtheorg_theme/css/htmlize.css"/>
#+HTML_HEAD: <link rel="stylesheet" type="text/css" href="https://tfree87.github.io/org-html-themes/src/readtheorg_theme/css/readtheorg.css"/>

#+HTML_HEAD: <script src="https://ajax.googleapis.com/ajax/libs/jquery/2.1.3/jquery.min.js"></script>
#+HTML_HEAD: <script src="https://maxcdn.bootstrapcdn.com/bootstrap/3.3.4/js/bootstrap.min.js"></script>
#+HTML_HEAD: <script type="text/javascript" src="https://fniessen.github.io/org-html-themes/src/lib/js/jquery.stickytableheaders.min.js"></script>
#+HTML_HEAD: <script type="text/javascript" src="https://fniessen.github.io/org-html-themes/src/readtheorg_theme/js/readtheorg.js"></script>

The above code produced the expected result.

Thanks for all your work on this neat project!

Aligning the links for page up and home to the right side of the page prevents them from being hidden by the table of contents.
@fniessen fniessen merged commit ceeb696 into fniessen:master Jul 27, 2022
@fniessen
Copy link
Owner

Dear @tfree87 , thank you very much!

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