clarkead commented on a change in pull request #27369:
URL: https://github.com/apache/spark/pull/27369#discussion_r477525318
##########
File path: docs/_layouts/global.html
##########
@@ -20,12 +20,15 @@
<link rel="stylesheet" href="css/bootstrap.min.css">
<style>
body {
+ {% if page.url contains "/ml" or page.url contains "/sql" or
page.url contains "migration-guide.html" %}
Review comment:
@srowen Sorry about the delay getting an updated PR. I've pushed two
small tweaks to address the items in this thread. The first is to try and fix
the formatting on those pages with left navigation that had special logic.
I've removed the logic and fixed the width on the expand/collapse trigger.
The second change is with regards to the issue I saw on the
rdd-programming-guide.html and streaming-programming-guide.html pages. It
ended up being an issue where older versions of Bootstrap where setting the
max-width on images to 100%, but Bootstrap4 removed that. As a result larger
image files were blowing out the right side. I've added a css style to re-add
this in main.css.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
