docs: design plan for immediate-apply Internet Settings with deferred restart#2491
Open
merchako wants to merge 3 commits into
Open
docs: design plan for immediate-apply Internet Settings with deferred restart#2491merchako wants to merge 3 commits into
merchako wants to merge 3 commits into
Conversation
… restart Proposes moving the Internet Settings dialog from a mandatory "Save and restart" button to immediate-apply per setting, tagging restart-requiring settings, and offering a Restart now / Restart later choice with a persistent reminder (including in the empty-state UI) so users are not stranded after dismissing the prompt. Co-authored-by: Claude <noreply@anthropic.com> Session-URL: https://claude.ai/code/session_018JRuUNqsC5vnCZAXYyZ1SG
Fixes the format:check CI failure (emphasis markers and table padding). Co-authored-by: Claude <noreply@anthropic.com> Session-URL: https://claude.ai/code/session_018JRuUNqsC5vnCZAXYyZ1SG
Wires in the Applying Changes, Triggers and Actions, Settings & State, and Dismissal Patterns guideline articles as relative links to their eventual merged paths, plus a References section noting each is pending on an open PR. Co-authored-by: Claude <noreply@anthropic.com> Session-URL: https://claude.ai/code/session_018JRuUNqsC5vnCZAXYyZ1SG
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.
Summary
Adds a design proposal (
docs/specs/2026-07-02-internet-settings-restart-ux-design.md) for reworking the Internet Settings dialog in theparatext-registrationextension.Today the dialog bundles internet-use mode, proxy config, and server selection behind a single "Save and restart" button that always restarts the app. This violates Platform.Bible's immediate-apply principle and forces a restart even when the changed setting may not need one.
The proposal recommends:
InternetAccessstate live, so this is a frontend change.)Notes
PlatformScrTextCollection.cs:47is suggestive but the networking consumers live inParatextData.dll).AI-assisted — session
🤖 Generated with Claude Code
Generated by Claude Code
This change is