Skip to content

Group AnkiQuest settings into clear sections - #49

Merged
AldanaBRZ merged 2 commits into
float3:ankiquestfrom
AldanaBRZ:fix/organize-ankiquest-settings
Sep 28, 2026
Merged

AldanaBRZ merged 2 commits into
float3:ankiquestfrom
AldanaBRZ:fix/organize-ankiquest-settings

Conversation

@AldanaBRZ

@AldanaBRZ AldanaBRZ commented Sep 28, 2026 •

Copy link
Copy Markdown
Collaborator

Purpose / Description

AnkiQuest put more than twenty controls in one long settings list, making streak protection, notification delivery, and account options hard to find. The entry page now shows the leaderboard and five short sections: Account & connection, Study & streaks, Reminders & alerts, Friends & sharing, and App & data.

Approach

Move each existing control into one section while preserving its preference key, default, and behavior. Split notification options into event and delivery groups. Add section labels in the app's English, Spanish, German, French, and Portuguese resources.

How Has This Been Tested?

  • Focused Android Robolectric tests for settings navigation, control coverage, and notification health: 8 passed.
  • :AnkiDroid:ktlintCheck passed.
  • XML resources parsed and the old and new control keys compared: all 22 are retained exactly once.

@github-actions

github-actions Bot commented Sep 28, 2026 •

Copy link
Copy Markdown

Snapshot diff report vs ankiquest. Open screenshot-diff for diffs.

  • PreferencesScreenshotTest: 6 changes
All 6 changed screenshots

PreferencesScreenshotTest

  • AnkiquestAccountSettingsFragment_compare.png
  • AnkiquestMaintenanceSettingsFragment_compare.png
  • AnkiquestNotificationsSettingsFragment_compare.png
  • AnkiquestSettingsFragment_compare.png
  • AnkiquestSharingSettingsFragment_compare.png
  • AnkiquestStudySettingsFragment_compare.png

@AldanaBRZ
AldanaBRZ merged commit 3006aba into float3:ankiquest Sep 28, 2026
13 of 16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant