Fix: remove dots from expanded images#1368
Conversation
shahar1
left a comment
shahar1
left a comment
There was a problem hiding this comment.
We need to use only official releases, and the pointer that you suggested does not seem to be one, unfortunately. If the latest version fixes it and it doesn't break anything else - then we could proceed from there.
|
Hey
Thanks for explaining!
I understand now that we need to use official Docsy releases only.
I’ll check whether the latest Docsy release already fixes this and update
accordingly.
…On Sat, 3 Jan 2026 at 20:19, Shahar Epstein ***@***.***> wrote:
***@***.**** requested changes on this pull request.
We need to use only official releases
<https://github.com/google/docsy/releases>, and the pointer that you
suggested does not seem to be one, unfortunately. If the latest version
fixes it and it doesn't break anything else - then we could proceed from
there.
—
Reply to this email directly, view it on GitHub
<#1368 (review)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/BDF5MBXVFKOHV5KZGGZBVMT4E7JJNAVCNFSM6AAAAACQQQYAGSVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZTMMRUGM3DANBSGA>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
|
I’ve updated the Docsy submodule to point to the latest official release (v0.13.0). Please let me know if this resolves the issue correctly or if any further changes are needed. |
Hmm. build failed. |
|
I checked this on my side, and it appears that the dot is actually part of the image itself 😅 https://github.com/apache/airflow-site/blob/main/landing-pages/site/static/images/feature-image.png |
|
Solved differently |
4 participants
This PR removes the dots displayed under expanded images.
The dots were rendered via Bootstrap carousel indicators. The fix is implemented as a project-level SCSS override in the Docsy theme, without modifying Docsy core styles.
Only the Docsy submodule pointer is updated.
Fixes #222