-
Notifications
You must be signed in to change notification settings - Fork 125
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Editorial: Update ATK URL #2309
base: main
Are you sure you want to change the base?
Conversation
✅ Deploy Preview for wai-aria ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
so.... initially i was thinking this would be good - replace all the links with the new link. but this doesn't actually fix the link in the referenced terms. that still goes to the old documentation page, and that's an issue in core aam and ARIA (all the specs i'd assume) as well. but also, core aam doesn't link to each of the platform pages within each feature table.... and that got me wondering why this spec does/should continue to do so? i'm thinking maybe the fix here is to fix the link in the informative references section, and instead change the atk links to use the reference link |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
see prior comment
Thanks @scottaohara; that makes sense. 👍 I’ll update the PR to do what you’re suggesting:
❧ One question: I found https://gnome.pages.gitlab.gnome.org/at-spi2-core/atk/ today; between that and https://docs.gtk.org/atk/, which should we use? |
@spectranaut do you know which we should be referring to, per @smockle's question above? seems the api version is the same in both, but the library version is different. |
…erences” section at the bottom of the page
Quick updates:
|
looks good to me. i am thinking we leave the link that's in the biblio.js already... we can tackle changing the other platform api links to instead link to their reference section link in a follow on PR, if we dont' want to just do those here as well. |
Initially filed as w3c/html-aam#560
https://gnome.pages.gitlab.gnome.org/atk/ (the URL currently referenced throughout html-aam) 404s, even after logging in to the GNOME GitLab instance1.
https://docs.gtk.org/atk/ (this PR’s replacement URL) seems to be the new location. GTK’s “API Docs” page links to this URL.Update: There are 3 candidates; details in #2309 (comment).Preview: https://deploy-preview-2309--wai-aria.netlify.app/html-aam/
❧
Test, Documentation and Implementation tracking
Once this PR has been reviewed and has consensus from the working group, tests should be written and issues should be opened on browsers. Add N/A and check when not applicable.Footnotes
That said, GNOME’s “Technologies” page still links to this URL. 🤷 ↩