Skip to content

Conversation

@johnny9
Copy link
Collaborator

@johnny9 johnny9 commented Feb 27, 2023

Windows
Intel macOS
Apple Silicon macOS
ARM64 Android

@johnny9
Copy link
Collaborator Author

johnny9 commented Feb 27, 2023

Screenshot from 2023-02-26 22-56-18

Copy link
Contributor

@jarolrod jarolrod left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you group the following into this PR, and update the PR title to something like: "Correct BlockClock states text" (or something similar)

This should be "Please wait" per the design file

subText: "Please Wait"

@johnny9 johnny9 changed the title Use Estimating as the default string for remainingSyncTime Fix up of Block Clock strings Feb 28, 2023
Copy link
Contributor

@jarolrod jarolrod left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK 5ccd6ae

"Estimating" "Please wait"
Screen Shot 2023-02-27 at 11 30 45 PM Screen Shot 2023-02-27 at 11 30 05 PM

@hebasto hebasto merged commit bb4e261 into bitcoin-core:main Feb 28, 2023
tx-signer450 added a commit to tx-signer450/gui-qml that referenced this pull request Oct 20, 2025
628c4ded92b620cdd7610ed06888182330061f43 qml: fix capitalization on BlockClock Please wait string (johnny9)
49e709754237c024600791367095a2b41012513d qml: use Estimating as the default string for remainingSyncTime (johnny9)

Pull request description:

  [![Windows](https://img.shields.io/badge/OS-Windows-green)](https://api.cirrus-ci.com/v1/artifact/github/bitcoin-core/gui-qml/win64/insecure_win_gui.zip?branch=pull/275)
  [![Intel macOS](https://img.shields.io/badge/OS-Intel%20macOS-green)](https://api.cirrus-ci.com/v1/artifact/github/bitcoin-core/gui-qml/macos/insecure_mac_gui.zip?branch=pull/275)
  [![Apple Silicon macOS](https://img.shields.io/badge/OS-Apple%20Silicon%20macOS-green)](https://api.cirrus-ci.com/v1/artifact/github/bitcoin-core/gui-qml/macos_arm64/insecure_mac_arm64_gui.zip?branch=pull/275)
  [![ARM64 Android](https://img.shields.io/badge/OS-Android-green)](https://api.cirrus-ci.com/v1/artifact/github/bitcoin-core/gui-qml/android/insecure_android_apk.zip?branch=pull/275)

ACKs for top commit:
  jarolrod:
    ACK 628c4ded92b620cdd7610ed06888182330061f43

Tree-SHA512: 8eed140b96340c382bc47b170ec5e52879f21b6ed96f2bf63ccd7bde730361df7191a809f869c9ea76cdec1afa5ee6e1666d3637987059552908dc5d5c26cd49
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.

3 participants