Polish from the design review: messages, names, accessibility of the Resolve dialog, CLI help #192

Open
opened 2026-11-11 05:58:42 +00:00 by james.maes · 2 comments
Owner

From the release reviews of 1.1.1 (2026-11-10). Not part of 1.1.1; planned for 1.2.0 unless said otherwise.

Messages

  • The update check and "Sync all stopped" show an exception's own text. For the update check: "Could not reach git.kof22.com to check for updates. The latest release is at https://git.kof22.com/KofTwentyTwo/gclo/releases."
  • Rows that were never synced, and rows that are not selected, read "Queued"; the filter tab calls them "Pending".
  • One setting has three names: "Parallel clones", "Parallel clones and pulls", "Default parallelism".

Resolve paths dialog

  • Each row's rename box has no accessible name and every checkbox is only "Skip": name them for their path.
  • The close button "Skip this repo" means Cancel.
  • An active column filter in the table is shown by tint only.

Navigation pane

  • Edit and Delete stay available while that account syncs; saving or deleting cancels the sync without saying so.
  • "Sync all" turns into "Cancel sync all" in place, so a double-click starts and cancels.
  • With no token, the organization box of an account's page shows its placeholder although the account has an organization.

Command line

  • A usage error in a subcommand points to gclo --help instead of that command's help.
  • A missing --token-file prints the path twice, with the runtime's own text.
  • gclo accounts --help does not list exit code 1 (Ctrl+C).

Reaching the project

  • Help > Submit an issue and the CLI's "please report it" line lead to a sign-in page for people without an account on git.kof22.com (registration is closed).
From the release reviews of 1.1.1 (2026-11-10). Not part of 1.1.1; planned for 1.2.0 unless said otherwise. ## Messages - The update check and "Sync all stopped" show an exception's own text. For the update check: "Could not reach git.kof22.com to check for updates. The latest release is at https://git.kof22.com/KofTwentyTwo/gclo/releases." - Rows that were never synced, and rows that are not selected, read "Queued"; the filter tab calls them "Pending". - One setting has three names: "Parallel clones", "Parallel clones and pulls", "Default parallelism". ## Resolve paths dialog - Each row's rename box has no accessible name and every checkbox is only "Skip": name them for their path. - The close button "Skip this repo" means Cancel. - An active column filter in the table is shown by tint only. ## Navigation pane - Edit and Delete stay available while that account syncs; saving or deleting cancels the sync without saying so. - "Sync all" turns into "Cancel sync all" in place, so a double-click starts and cancels. - With no token, the organization box of an account's page shows its placeholder although the account has an organization. ## Command line - A usage error in a subcommand points to `gclo --help` instead of that command's help. - A missing `--token-file` prints the path twice, with the runtime's own text. - `gclo accounts --help` does not list exit code 1 (Ctrl+C). ## Reaching the project - Help > Submit an issue and the CLI's "please report it" line lead to a sign-in page for people without an account on git.kof22.com (registration is closed).
james.maes added the deferredenhancement labels 2026-11-11 05:58:42 +00:00
Author
Owner

More from the second pass of the release reviews, also not urgent:

  • A Sync all pass in which every account was skipped (busy, canceled in its own page, nothing selected) ends with the success icon, although its text says "0 of N accounts synced". An outcome between success and failure would fit.
  • In the pane, an account that could not be synced shows the count badge when it still has failed rows from an earlier run; the two meanings of a failed account are told apart in the shell, and a state of its own would keep that in the view model.
  • A repository named as git's own folder fails its row in the sync, but on the path where names are resolved for a caller-supplied list the message of an unusable name ends in "(Parameter 'repoName')".
More from the second pass of the release reviews, also not urgent: - A Sync all pass in which every account was skipped (busy, canceled in its own page, nothing selected) ends with the success icon, although its text says "0 of N accounts synced". An outcome between success and failure would fit. - In the pane, an account that could not be synced shows the count badge when it still has failed rows from an earlier run; the two meanings of a failed account are told apart in the shell, and a state of its own would keep that in the view model. - A repository named as git's own folder fails its row in the sync, but on the path where names are resolved for a caller-supplied list the message of an unusable name ends in "(Parameter 'repoName')".
Author
Owner

From the review of #207 (Gitea in the app's wizard), left for later:

  • ForgeTexts.TokenName / TokenNeeds word every forge that is not GitHub like Gitea, so an account on a forge this version does not know gets Gitea's advice on its (unusable) token step.
  • UiTestFixture.RepositoriesAt (clone addresses on the Gitea server) is exercised by no UI test: the new wizard test cancels on step 3.
  • The Share invitation and its subject still say "GitHub organization".
  • A radio group that reports no selection (-1) is ignored by PlaceIndex, but nothing puts the selection back on the control.
From the review of #207 (Gitea in the app's wizard), left for later: - `ForgeTexts.TokenName` / `TokenNeeds` word every forge that is not GitHub like Gitea, so an account on a forge this version does not know gets Gitea's advice on its (unusable) token step. - `UiTestFixture.RepositoriesAt` (clone addresses on the Gitea server) is exercised by no UI test: the new wizard test cancels on step 3. - The Share invitation and its subject still say "GitHub organization". - A radio group that reports no selection (-1) is ignored by `PlaceIndex`, but nothing puts the selection back on the control.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: KofTwentyTwo/gclo#192