DOC Update sponsor page: reorganize sponsors and add BNP Paribas Group #32642
Conversation
- Reorganize sponsors into tiers: Founding (Inria), Gold (Chanel), Silver (BNP Paribas Group), Bronze (NVIDIA) - Remove logos from Past Sponsors section, convert to full-width text format - Convert Other contributions section to bullet points - Add BNP Paribas Group logo and update sponsor information - Add AXA, BNP Cardif, and Dataiku to past consortium sponsors grid - Update probabl description to mention sponsorship program management - Update footer funding logos - Simplify sponsor descriptions for consistency
There was a problem hiding this comment.
Thanks for the PR, this seems fine to me!
Having someone outside Probabl give a quick look would be nice, e.g @betatim @lucyleeow or @OmarManzoor?
The main change is the reorg of the funding section to display different tiers (gold, silver, etc ...) and update the information:
- Replace bnp-paribas.png with bnp-paribas.jpg - Add NVidia funding entry for Tim Head - Add JavaScript to make external links open in new tab
ArturoAmorQ
changed the title
doc/about.rst
Outdated
| } | ||
| }); | ||
| } | ||
| // Also handle links anywhere on the page if on about.html |
There was a problem hiding this comment.
What does 'handle links' mean?
There was a problem hiding this comment.
That second part isn’t really needed here. It’s just a defensive fallback I added in case the page didn’t wrap its content in a #about element, or if the same script were reused elsewhere. Since on about.html everything is already inside the #about section, we can safely remove that second block. I guess.
Co-authored-by: Lucy Liu <jliu176@gmail.com>
doc/about.rst
Outdated
| } | ||
| }); | ||
| } | ||
| // Also handle links anywhere on the page if on about.html |
There was a problem hiding this comment.
That second part isn’t really needed here. It’s just a defensive fallback I added in case the page didn’t wrap its content in a #about element, or if the same script were reused elsewhere. Since on about.html everything is already inside the #about section, we can safely remove that second block. I guess.
4 participants
Reference Issues/PRs
What does this implement/fix? Explain your changes.
Any other comments?