Skip to content

[v2.0] Improved Accessibility#303

Draft
jaylikesbunda wants to merge 7 commits into
two-point-zerofrom
accessibility2.0
Draft

[v2.0] Improved Accessibility#303
jaylikesbunda wants to merge 7 commits into
two-point-zerofrom
accessibility2.0

Conversation

@jaylikesbunda
Copy link
Copy Markdown
Member

What's new (Author - fill this out)

  • Added a new Accessibility settings menu with Font Size, Bold Text, High Contrast, Reduced Motion, Epilepsy Warning, and Input Repeat Speed options.
  • Added accessibility configuration to the setup wizard.
  • Added centralized accessibility font helpers and wired font scaling through display/UI views.

Changed

  • Input repeat timing is configurable for joystick and T-Deck keyboard input.
  • Reduced motion disables or shortens view fades, carousel transitions, popup fades, and splash delay.

Removed

Verification (Author - fill this out)

  • I have tested Primary device(s) (list exact HW + config): TEmbedCC1101
  • I have tested potentially affected devices (and/or list potential affected devices not available to you):
  • I have wrapped device specific code with #ifdef CONFIG_... or similar: [ ] Yes / [X] N/A
  • I have updated Hugo Docs with any new or changed info for end users: [ ] Yes / [ ] N/A

Linked Issues

  • Closes #

Checklist (Reviewer - don't fill this out)

  • Code builds, flashes, and feature verified with no functional issues on listed device(s)
  • [a lil] Changes reviewed for unintended impact on other devices/targets

@jaylikesbunda jaylikesbunda self-assigned this Apr 28, 2026
@jaylikesbunda
Copy link
Copy Markdown
Member Author

@copilot resolve the merge conflicts in this pull request

Copy link
Copy Markdown

Copilot AI commented May 4, 2026

@copilot resolve the merge conflicts in this pull request

The merge conflicts have already been resolved in commit f8a3d16 (Merge branch 'two-point-zero' into accessibility2.0). The CI build also completed successfully after the merge.

@jaylikesbunda
Copy link
Copy Markdown
Member Author

@copilot resolve the merge conflicts in this pull request

Resolve conflicts by keeping accessibility font scaling (accessibility_get_font_*)
alongside new design_tokens.h and gui_anim.h from two-point-zero. Combined slide
transitions with reduced motion checks. Added SD backup settings alongside
accessibility settings in the SettingsType enum. Included both error_popup.h and
settings_sd_backup.h in commandline.c.

Co-authored-by: jaylikesbunda <133104319+jaylikesbunda@users.noreply.github.com>
Copy link
Copy Markdown

Copilot AI commented May 4, 2026

@copilot resolve the merge conflicts in this pull request

Resolved in commit 70807215. The branch was 16 commits behind two-point-zero (which added slide transitions, SD backup settings, design tokens, and UI polish). Conflicts were in 6 files:

  • settings_manager.h — kept both accessibility settings (FONT_SIZE, REDUCED_MOTION, HIGH_CONTRAST, etc.) and SD backup settings (EXPORT/IMPORT_SETTINGS_SD)
  • commandline.c — included both error_popup.h and settings_sd_backup.h
  • options_view.c — kept accessibility font-size-aware implementation
  • display_manager.c — included all three headers; combined the slide transition logic from two-point-zero with the reduced-motion checks from this branch (reduced motion skips the animation entirely, otherwise uses slide or fade)
  • keyboard_screen.c — included both accessibility_fonts.h and design_tokens.h
  • main_menu_screen.c — kept dynamic ANIM_DURATION (0 when reduced motion enabled) and accessibility_get_font_body() for carousel labels

@jaylikesbunda
Copy link
Copy Markdown
Member Author

@copilot resolve the merge conflicts in this pull request

Keep accessibility settings (font size, reduced motion, high contrast, bold text,
epilepsy warning, input repeat speed) alongside new two-point-zero features:
menu item borders setting, dns_server include in options_screen, heap caps include.
SettingsCategoryId type conflict resolved in favor of existing int16_t types to
maintain compatibility with accessibility settings screen code.

Co-authored-by: jaylikesbunda <133104319+jaylikesbunda@users.noreply.github.com>
Copy link
Copy Markdown

Copilot AI commented May 13, 2026

@copilot resolve the merge conflicts in this pull request

Resolved in commit e900448. The branch was 62 commits behind two-point-zero. Conflicts were in 4 files:

  • settings_manager.h — kept all accessibility settings enum values (FONT_SIZE, REDUCED_MOTION, HIGH_CONTRAST, BOLD_TEXT, EPILEPSY_WARNING, INPUT_REPEAT_SPEED) and added the new SETTING_MENU_ITEM_BORDERS from two-point-zero; same for the FSettings struct fields
  • settings_manager.c — merged all accessibility NVS keys/load/save/persist/getter/setter functions alongside the new menu_item_borders setting from two-point-zero
  • main_menu_screen.c — kept accessibility_get_font_small() for grid card labels
  • options_screen.c — included both <stdint.h> (accessibility branch) and dns_server.h/esp_heap_caps.h (two-point-zero); kept accessibility-compatible SettingsItem field types and font_size_options/repeat_speed_options arrays

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants