Skip to content

Improve custom sync server and AnkiDroid directory dialogs#20489

Open
dagangtj wants to merge 1 commit intoankidroid:mainfrom
dagangtj:improve-sync-server-dialogs
Open

Improve custom sync server and AnkiDroid directory dialogs#20489
dagangtj wants to merge 1 commit intoankidroid:mainfrom
dagangtj:improve-sync-server-dialogs

Conversation

@dagangtj
Copy link

Description

Improves the custom sync server and AnkiDroid directory dialogs as requested in #20487.

Changes

  • Capitalized 'url' to 'URL' in sync server dialog title
  • Added border to input boxes (URL, PEM certificate, Directory) for better visual consistency
  • Replaced 'OK' button with 'Save' button to match other dialogs

Implementation Details

  • Created edittext_border.xml drawable with 1dp stroke and 4dp corner radius
  • Modified VersatileTextPreference.kt to apply border and change button text
  • Updated string resource to use 'URL' instead of 'url'

Testing

  • Verified border appears on all EditText fields in preference dialogs
  • Confirmed 'Save' button replaces 'OK' button
  • Checked that URL capitalization is correct

Fixes #20487

- Capitalize 'url' to 'URL' in sync server dialog
- Add border to input boxes (URL, PEM, Directory)
- Replace 'OK' button with 'Save' button

Fixes ankidroid#20487
@github-actions
Copy link
Contributor

Important

Maintainers: This PR contains Strings changes

  1. Sync Translations before merging this PR and wait for the action to complete
  2. Review and merge the auto-generated PR in order to sync all user-submitted translations
  3. Sync Translations again and merge the PR so the huge automated string changes caused by merging this PR are by themselves and easy to review

@Ayush-Patel-56
Copy link
Contributor

Hi @dagangtj, kindly follow the standard PR template.

@DoomsCoder
Copy link
Contributor

Hi @dagangtj 👋

It would be good to follow the suggestion mentioned by @Ayush-Patel-56 and update the PR description according to the template.Since this PR changes some dialog UI, adding screenshots of the updated dialogs would also help. Its good practice for future PRs as well.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Improve the custom sync server and AnkiDroid directory dialogs

3 participants