Improve custom server sync dialogs

[manocormen]

Purpose / Description

In this PR, I updated the "Sync URL" and "Custom root certificate", and "AnkiDroid directory" [scope update 1] dialogs to:

• Style them consistently most notably, make their titles the same size [scope update 2].
• Add a dialog hint: i.e. a floating label on the input border.
• Rename the positive button from "OK" to "Save".

Additionally, I capitalized "url" in "Sync URL".

Fixes

• Fixes Improve the custom sync server and AnkiDroid directory dialogs #20487.

Approach

To enforce consistency across dialogs, I identified the shared classes: "Sync URL" and "Custom root certificate" use VersatileTextPreference/VersatileTextPreferenceDialogFragment; I updated "AnkiDroid directory" to use it too.

Then, I wired a dialogHint attribute, and I made shared layout display the hint and have a "Save" button. The dialog styling update follows the same approach as the already-merged #20520.

How Has This Been Tested?

I did some manual tests across the two dialogs on my physical device:

• Checked hint, title size, and "Save" button.
• Tried valid and invalid inputs.

I ran the following commands:

./gradlew jacocoUnitTestReport                      
./gradlew lintRelease ktlintCheck

Learning

Lessons learned:

• Understanding the preference/dialog class hierarchy helped identify the right shared implementation point.
• Reviewing sibling PRs under the same parent issue helped confirm the preferred dialog pattern to use.
• Favor view binding over findViewById (ref) and use extensions (ref) when possible.
• Favor default styles over custom (ref), unless there's a strong reason not to.

Checklist

• You have a descriptive commit message with a short title (first line, max 50 chars).
• You have commented your code, particularly in hard-to-understand areas
• You have performed a self-review of your own code
• UI changes: include screenshots of all affected screens (in particular showing any new or changed strings)
• UI Changes: You have tested your change using the Google Accessibility Scanner

[github-actions]

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

[ZornHadNoChoice]

Thanks, but the text fields shouldn't be rounded.

[manocormen]

Thanks, but the text fields shouldn't be rounded.

Corner radius removed and screenshots updated. I'd carried them over from dialog_incrementer_preference.xml, but I've dropped the overrides here.

[manocormen]

A couple of comments for whenever you get a chance to review this:

• This PR is chunkier than my previous ones. I had it locally as three commits but squashed before pushing, assuming that's what you'd want. But now I'm double-guessing myself on whether you'd prefer to review them separately first. If I should wait for the go-ahead to squash next time, let me know.

• I mention this in the description, but in case it goes unnoticed: the PR is largely based on the already-merged feat(preferences): improve timeout/backup number dialogs #20520. Hopefully that facilitates review.

[david-allison]

Please revert the AnkiDroid Directory changes, this screen will be improved in

• Feat: Add path picker GUI for AnkiDroid Directory setting #19855

[manocormen]

@david-allison Done: removed the AnkiDroid Directory changes and updated the PR description accordingly.

[david-allison]

LGTM, with optional improvements, thanks!!!

For a new issue: consider validating the URL as it's typed

[BrayanDSO]

I'm not sure why we aren't using the default style of Material dialogs.

Is the name size issue related to the string size?

[manocormen]

Is the name size issue related to the string size?

Yes, with default styles, beyond a certain length, the title gets shrunk:

So I enforced custom styles like here. If it gets too long, the title wraps to a newline:

[BrayanDSO]

I'd go with whatever Material does

[manocormen]

Done: removed the custom styles and updated the PR description accordingly.

[github-actions]

Maintainers: Please Sync Translations to produce a commit with only the automated changes from this PR.

Read more about updating strings on the wiki,

• localization-administration
• download-localized-strings