Skip to content

Commit a393fc7

Browse files
authored
fix(docs): remove navbar padding breaking ui on mobile (#18402)
## Which issue does this PR close? NA ## Rationale for this change I just noticed that an existing css was forcing extra padding in the navbar, which resulted in breaking the navbar ui in certain small screen (mobile) devices. | Old | New (fixed) | |:---:|:------------:| | <img width="384" height="120" alt="Screenshot 2025-10-31 at 14-22-44 DataFrame API — Apache DataFusion documentation" src="https://github.com/user-attachments/assets/c4e23471-cfde-4988-b887-13060da9faf1" /> | <img width="384" height="117" alt="Screenshot 2025-10-31 at 14-23-19 DataFrame API — Apache DataFusion documentation" src="https://github.com/user-attachments/assets/152c7b8d-cf0e-4a62-a616-264398d074a1" /> | *notice how the right side navbar element is breaking out of the layout To replicate this issue (firefox-browser) : - Open https://datafusion.apache.org/ in "responsive design mode" (ctrl+shift+m) - Select "Galaxy S20 Android 11" or "iPhone 12/13 + Pro iOS 14.6" ...etc device. ## What changes are included in this PR? Removed an existing css that was responsible for this behavior.
1 parent 70e9155 commit a393fc7

File tree

1 file changed

+0
-8
lines changed

1 file changed

+0
-8
lines changed

docs/source/_static/theme_overrides.css

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -69,14 +69,6 @@ html[data-theme="dark"] .dark-logo {
6969
margin-left: auto;
7070
}
7171

72-
/* Ensure the logo is properly displayed */
73-
74-
.navbar-brand {
75-
height: auto;
76-
width: auto;
77-
padding: 0 2em;
78-
}
79-
8072
/* This is the bootstrap CSS style for "table-striped". Since the theme does
8173
not yet provide an easy way to configure this globally, it easier to simply
8274
include this snippet here than updating each table in all rst files to

0 commit comments

Comments
 (0)