Add documentation for the Neo4j plugin#514
Conversation
Issue: #507 Currently the supported version range is 5.20–5.20 which looks a bit silly, but we will update that once we establish the lower bound.
|
The link checker is reporting 2 URLs that must be broken until we make the release. |
Codecov Report✅ All modified and coverable lines are covered by tests. 🚀 New features to boost your workflow:
|
niegrzybkowski
left a comment
There was a problem hiding this comment.
Nitpicks and css shenanigans which would be good to fix
|
|
||
| .jelly-table-tight + div > div > table td, th { | ||
| padding-left: 0.5em !important; | ||
| padding-right: 0.5em !important; |
There was a problem hiding this comment.
I'm guessing this is so the table doesn't need a horizontal scrollbar, but on my PC it still shows up, which is not the best look on the first page.
There was a problem hiding this comment.
Looking at the CSS, it's probably because the th has min-width pinned at at 5rem
There was a problem hiding this comment.
I don't think there is a way around this, entirely. The way table sizing works in HTML is... complicated and bad. This simply increases the chances that stuff will fit into the table and be visible, at least.
There are complex hacks around this, but do we really need it?
There was a problem hiding this comment.
It's not critical but I think it looks kinda ugly
There was a problem hiding this comment.
Changing it to min-width: 4rem works
Issue: #507
Currently the supported version range is 5.20–5.20 which looks a bit silly, but we will update that once we establish the lower bound.