tlienart / PkgPage.jl

Create a beautiful landing page for your package in less than 10 minutes.
https://tlienart.github.io/PkgPage.jl/
MIT License
116 stars 6 forks source link

Docs is not clickable #23

Closed Roger-luo closed 4 years ago

Roger-luo commented 4 years ago

I created a package page for my package here: https://rogerluo.me/Comonicon.jl/

but somehow when I set

@def add_docs  = true
@def docs_url  = "https://rogerluo.me/Comonicon.jl/dev/"

the Docs cannot be clicked, tho when I inspect the actual link the href is set to https://rogerluo.me/Comonicon.jl/dev/ any idea how to fix this?

Roger-luo commented 4 years ago

It seems can be fixed if I disable the logo. The LOGO span somehow is too large and it covers the package name and the Docs

tlienart commented 4 years ago

Thanks for the report! so can I summarise the issue as follows?:

@zdroid would you mind having a look at this one? thanks!!

zlatanvasovic commented 4 years ago

This is also an issue with PkgPage.jl's page, since "PkgPage.jl" can't be clicked.

The problem is that we were using navbar-brand incorrectly: https://getbootstrap.com/docs/4.5/components/navbar/#supported-content. It is not supposed to be used with 2 links, but just with one. I suggest we move Docs out of navbar-brand and put it maybe on the right side?

tlienart commented 4 years ago

since "PkgPage.jl" can't be clicked.

is this a browser issue? I have no problem clicking it.

I suggest we move Docs out of navbar-brand and put it maybe on the right side?

sounds good, would you mind opening a PR with this? thanks! 🙏

zlatanvasovic commented 4 years ago

You can click it, but only on 10px of its right side. After that, you can't.

I'll send a fix soon. :smile:

It turns out the needed fix was just to set one dimension to a concrete value (based on Bootstrap docs). max-height and height are identical in this case anyway (the logo will be 25px).

tlienart commented 4 years ago

@Roger-luo this should be fixed & released (thanks Zlatan!) let us know if you still have an issue

Roger-luo commented 4 years ago

Thanks for the quick fix!