ADG-483 Remove Twitter and Facebook links from footer and content - #502
Merged
christophdubach merged 2 commits intoSep 16, 2026
Merged
Conversation
✅ Deploy Preview for accessibility-developer-guide ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Contributor
There was a problem hiding this comment.
🟡 Changes recommended
Retain the supported RSS link and clarify or separate the unrelated contributor-roster changes.
Get a fresh assessment by requesting another Copilot review.
Pull request overview
Removes Facebook and Twitter links from the footer, documentation, and configuration.
Changes:
- Removes social links and sharing references.
- Removes the unused Twitter configuration field.
- Updates contribution and contributor information.
File summaries
| File | Summary |
|---|---|
src/components/global/footer/footer.hbs |
Removes Facebook and Twitter footer links. |
README.md |
Removes social-media links. |
pages/introduction/how-to-use/README.md |
Removes social update content. Nit (3 votes): retain an RSS-only sentence. |
pages/introduction/about/README.md |
Updates contact and contributor information. Nit (1 vote): clarify or separate unrelated roster changes. |
pages/contribution/README.md |
Replaces social-sharing instructions. |
config.js |
Removes the unused Twitter configuration field. |
Review details
Suppressed comments (1)
pages/introduction/about/README.md:85
- This hunk also changes the contributor roster and employment/title details, which is unrelated to removing Facebook and Twitter links and is not reflected in the PR title. Please split these edits or update the PR description and verify the roster changes so the review scope is explicit.
- **Regina Reusser**, accessibility consultant, Access for all.
- **Roman Alt**, accessibility consultant, Access for all.
- Files reviewed: 6/6 changed files
- Comments generated: 1
- Review effort level: Lite
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
| @@ -39,12 +39,3 @@ At the bottom of each content element, links to its sub-elements (chapters or pa | |||
| Being a comprehensive tutorial, the contents of this guide build up on each other sequentially. So we highly recommend you to start reading from the very beginning, then make your way continuously through the guide (use the **Previous** and **Next** links at the bottom to do so). Alas, internal links between different contents are provided sparingly. | |||
|
|
|||
| However, we have given our best to make the individual content elements as self-standing and self-explanatory as possible. This allows you to browse them on their own as a quick reference. Use the content navigation sidebar to quickly access different contents. | |||
christophdubach
deleted the
feature/ADG-483_remove-twitter-and-facebook
branch
September 16, 2026 15:42
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #483