I don’t know exactly how the idea of commenting pull requests here on the blog came to my mind. Maybe I’ve been influenced by an italian post on Raffaele Gaito’s blog (written by Alberto Cabas Vidani whom I thank) that quoted Gary Vee’s speech: “Document, don’t create”.
Actually, without knowing anything about this interesting stuff, it’s what I always did and continue to do: describe and document what I build without caring too much if the results are good or not (while I certainly keep studying to figure out how things should properly be made and improving my skills) but I never thought of document my pull requests too! They’re a lot and many of them unfortunately were on a project now closed and removed from github for various reasons.
Basically it’s just an API wrapper built for nodejs as an npm module for theTVDB open source project! You can pull it down with
npm install node-tvdb and start calling its methods to get information over your favourite tv shows!
The problems weren’t actually in the library itself, but in its documentation’s website that, on smartphones, was completely messed up!
If you read the package.json of the node-tvdb project, you notice that the website is being generated by jsdoc (a documentation automated generator) with a template created by the same guy that built this library.
Now follows a screenshot of the first one where some z-index were wrong and elements were overlapping on each other.
And here a screenshot of the second one. Here the
min-width property was causing the left part of the main div to not be rendered correctly on devices with small display.
Combined it was like this:
The problematic code in question for the z-index was this part (the lines marked with “-“ were problematic and have been replaced by the “+” lines):
And this was for the min-width problem:
This is the link of the pull request made with these changes.
I hope someone enjoyed this! See you again at the next post ;)