Skip to content

Conversation

@fantkolja
Copy link
Collaborator

@fantkolja fantkolja commented Dec 5, 2025

@fantkolja fantkolja requested a review from a team as a code owner December 5, 2025 14:53
@netlify
Copy link

netlify bot commented Dec 5, 2025

Deploy Preview for lucid-bassi-1dc305 ready!

Name Link
🔨 Latest commit 061444f
🔍 Latest deploy log https://app.netlify.com/projects/lucid-bassi-1dc305/deploys/6936ef52cecbff0008bf3e83
😎 Deploy Preview https://deploy-preview-295--lucid-bassi-1dc305.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@Rob-Hazelcast
Copy link
Contributor

So this stops the top-left search box getting covered by the site banner, but only by removing the site banner. Not sure that's desirable?

Still got the bug where clicking a parent item in the side nav opens the item above.

@fantkolja fantkolja marked this pull request as draft December 5, 2025 18:25
@fantkolja
Copy link
Collaborator Author

@Rob-Hazelcast @oliverhowell Sorry guys, still working on it, will ping you when ready

@fantkolja fantkolja changed the title Static header for swaggerUI Sticky header Dec 5, 2025
@fantkolja fantkolja changed the title Sticky header Static header for SwaggerUI Dec 5, 2025
@fantkolja fantkolja marked this pull request as ready for review December 5, 2025 19:11
@fantkolja
Copy link
Collaborator Author

fantkolja commented Dec 5, 2025

@Rob-Hazelcast @oliverhowell Tried everything I could think of, but I couldn't keep the header fixed on the swagger pages. What I suggest is to scroll it like in the preview... Please check if it's OK...

Seems I got it working after all...

@fantkolja fantkolja changed the title Static header for SwaggerUI Sticky header for SwaggerUI Dec 7, 2025
fantkolja added a commit to hazelcast/hazelcast-docs that referenced this pull request Dec 8, 2025
@fantkolja fantkolja merged commit 109f7c2 into master Dec 8, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants