bitsondatadev commented on code in PR #9642:
URL: https://github.com/apache/iceberg/pull/9642#discussion_r1478097135


##########
site/nav.yml:
##########
@@ -32,18 +32,18 @@ nav:
   - Talks: talks.md
   - Vendors: vendors.md
   - Project: 
-    - Join: community.md
+    - Community: community.md
     - Spec: spec.md
     - View spec: view-spec.md
     - Puffin spec: puffin-spec.md
     - Multi-engine support: multi-engine-support.md
     - How to release: how-to-release.md
     - Terms: terms.md
-  - Concepts:
-    - Catalogs: catalog.md
-  - ASF:
-    - Sponsorship: https://www.apache.org/foundation/sponsorship.html
-    - Events: https://www.apache.org/events/current-event.html
-    - License: https://www.apache.org/licenses/
-    - Security: https://www.apache.org/security/
-    - Sponsors: https://www.apache.org/foundation/thanks.html
+    - ASF: 
+      - Sponsorship: https://www.apache.org/foundation/sponsorship.html
+      - Events: https://www.apache.org/events/current-event.html
+      - License: https://www.apache.org/licenses/
+      - Security: https://www.apache.org/security/
+      - Sponsors: https://www.apache.org/foundation/thanks.html
+  - Concepts: 

Review Comment:
   No, I am nesting ASF under the projects header rather than ASF having their 
own header.
   
   The main issue wanting the change is how mkdocs renders headers. The 
previous site we had a hovering dropdown, we're avoiding that now for 
[accessibility concerns](https://www.w3.org/WAI/tutorials/menus/flyout/) and so 
clicking on the header requires loading a local page where the sub pages are 
rendered on the left.
   
   Unfortunately since all the ASF menu items are links, it just brings you to 
the first sponsorship link rather than show you all the options (you can try 
this on the current site by clicking the header). To remedy this I have nested 
these links in the nav menu under project as shown in the second picture above.
   
   This will effectively just remove the ASF from the top nav items which 
honestly is too crowded as Fokko mentions.
   



-- 
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.

To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org
For additional commands, e-mail: issues-h...@iceberg.apache.org

Reply via email to