Thanks for your work on this.

Note that this theme is a clone of the Sphinx theme of the same name. For the most part, we simply copy the CSS and JavaScript as-is and then fashion our templates to output (mostly) the same HTML as the upstream theme. The only additional CSS we include in theme-extra.css is generally to make up for deficiencies where we are not able to match the upstream HTML (for example, Markdown tables differ from Rst tables in subtle ways).

So thanks for locating your changes in the correct location. However...

If this bug does not exist upstream, then my assumption is that our HTML is missing something (maybe a css class on an element or a wrapping element). Also, the fact that the HTML includes inline styles makes me wonder if that is because the upstream theme does as well. To be clear, I haven't checked on either in this case but it does give me pause and should probably be investigated before moving forward with this fix.

It is also possible that this bug did exist upstream but has since been fixed, in which case updating to a more recent version of upstream may be the best way to resolve the issue. Again, that should probably be investigated. That said, if this is the case, it may be quicker for us to apply your fix temporarily with a note in the comments to remove the fix when we update to a more recent version of upstream.

Finally, I am concerned about your inability to see your changes reflected in a served instance. Installing with --editable in a virtualenv is the correct way to accomplish that. Is it possible that the rules were overridden by the upstream rules due to having a lower specificity? If so, they should show up as disabled in your browser's inspect tool. Have you checked that? Or are they not showing up at all?

Thank you for the extensive reply.

If this bug does not exist upstream, then my assumption is that our HTML is missing something (maybe a css class on an element or a wrapping element)

I just checked, and in fact, the Sphinx demo instance has an additional specification that states a padding-bottom of even 3em:

However, I don't think that this would solve the issue. Personally, I think that having fixed sizes that match each other are better (because this way, the last navigation item perfectly aligns to the current-version container).

Furthermore, MkDocs does not make use of the versions container in a way that the Sphinx theme does. There, you can actually expand the container to display further versions to switch between those:

So I think it's fine to stick with these "monkey patches," because they work in every MkDocs configuration, as no other divs are in any way appended to that container. Yes, these might break in the future, but that breakage would be detected while testing the new upstream CSS rules, and fixing those is not that difficult, fortunately.

That said, if this is the case, it may be quicker for us to apply your fix temporarily with a note in the comments to remove the fix when we update to a more recent version of upstream.

I agree. I have documented what I changed in every inch, so removing the changes is fairly straightforward.

Finally, I am concerned about your inability to see your changes reflected in a served instance. Installing with --editable in a virtualenv is the correct way to accomplish that.

So, I'm using conda locally because I use Python almost exclusively for statistics and machine learning in my PhD. I created a new environment using conda, and this means that my PATH pointed towards the environment pip.

I did not have any MkDocs installed (the local fork was the only one) and the mkdocs command correctly points to a directory in said environment. This means I don't really know where this came from, because the theme_extra.css was the original file in every respect, my changes were simply not there. (I did double-check the files, of course!) My changes definitely work in the way I implemented these, but that was my reason for indicating that I may lack some knowledge with regard to Python. I simply have given up understanding the different ways of handling Python :/

MkDocs does not make use of the versions container in a way that the Sphinx theme does.

Ah, right. That is the difference which causes the bug. You're right, in this situation, patching the CSS rules is the way to go.

I'm using conda locally

Okay, that explains the issue. Conda does some weird stuff that I don't understand. We have occasionally received reports, but never with enough information to narrow down to something actionable. We would need something with a Conda environment to work on the issue. Until then, I don't recommend development on MkDocs in a Conda environment.

In any event, when I get some time, I'll pull your changes locally and confirm they work as advertised. So long as they do, I'll merge this. In the meantime, a note needs to be added to the release notes.

Okay, that explains the issue. Conda does some weird stuff that I don't understand.

Feel you. I only use it because conda-forge is the only repository where I can get data science libraries compiled for ARM64 macOS, and since I knew how to create environments there I just went with it. But yeah, this explains why it's not working. When I get time, I will make sure to try to understand the regular pip environments as well.

With regard to the release notes – you mean this file? Then I'll add it in the meantime.

With regard to the release notes – you mean this file? Then I'll add it in the meantime.

Yes, please add a line under the section "Other Changes and Additions to Version 1.2".

Got it, and committed.

Finally had the time to test this. This does not fix the issue. The last item is now completely hidden and the penultimate item appears to be the last item.

Here is a screenshot from before the fix:

And here is a screenshot after the "fix" is applied:

Note that the last item "License" is completely hidden. I am not able to scroll down any farther. As least previously it was accessible, albeit not fully exposed. Also, the "GitHub" and "next" links are not properly spaced apart anymore.

Thank you for testing it out, and I took that chance to try to set it up once again, and now I finally managed to install it in a way that my mkdocs-command is using the development version from the repository.

I was able to reproduce the bug you found and noticed that it was a wrong specificity in the ruleset (so that the base theme would override my changes).

I pushed the necessary changes; it now looks like this:

First page:

Any page in between:

Last page:

As you can see, now it should be fixed. I also took the liberty to move the repository link in between the Next/Previous-arrows. I figured this way it might look more symmetrical. If this is not desired, I'll undo this change.

I also took the liberty to move the repository link in between the Next/Previous-arrows. I figured this way it might look more symmetrical. If this is not desired, I'll undo this change.

Let's undo this.

Let's undo this.

Done.

Looks good. Thanks.