Skip to content

Conversation

redstrate
Copy link
Contributor

Previously, we sent users to the top of the page whenever they click on a property that refers to a global enum/struct in our documentation. However the page is already filled with permalinks we can use, so I made it so we assume there's a permalink.

If the permalink doesn't exist, it would just send them back to the top of the page anyway which was the previous status quo.

@ogoffart ogoffart requested a review from NigelBreslaw October 5, 2025 11:27
@ogoffart ogoffart added the a:documentation Improvements or additions to documentation (infrastructure and text itself) (mN,bS) label Oct 5, 2025
@ogoffart
Copy link
Member

ogoffart commented Oct 5, 2025

Please use === instead of ==, that should fix the CI

@NigelBreslaw
Copy link
Member

Once the === issue above is fixed we can approve and merge this in.

Previously, we sent users to the top of the page whenever they
click on a property that refers to a global enum/struct in our
documentation. However the page is already filled with permalinks
we can use, so I made it so we assume there's a permalink.

If the permalink doesn't exist, it would just send them back to
the top of the page anyway which was the previous status quo.
@redstrate redstrate force-pushed the work/josh/struct-enum-permalinking branch from 99e6145 to 969f3d2 Compare October 7, 2025 13:39
@redstrate
Copy link
Contributor Author

Sorry about the delay, fixed that now

@NigelBreslaw
Copy link
Member

Thanks!

@NigelBreslaw NigelBreslaw merged commit 1c3569c into slint-ui:master Oct 7, 2025
41 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a:documentation Improvements or additions to documentation (infrastructure and text itself) (mN,bS)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants