Skip to content

Conversation

@jarolrod
Copy link
Contributor

@jarolrod jarolrod commented Feb 1, 2023

This finishes setting up all current Settings to be able to respond to the Setting state.

Filled Hover Active
Screen Shot 2023-02-01 at 2 12 02 AM Screen Shot 2023-02-01 at 2 12 11 AM Screen Shot 2023-02-01 at 2 12 19 AM

Windows
Intel macOS
Apple Silicon macOS
ARM64 Android

@jarolrod jarolrod force-pushed the right-caret-button-states branch from 37d2019 to 33b68b3 Compare February 1, 2023 18:31
@jarolrod
Copy link
Contributor Author

jarolrod commented Feb 1, 2023

updated from 37d2019 to 33b68b3, compare

Changes: rebased over main

@hebasto
Copy link
Member

hebasto commented Feb 4, 2023

Based on #241

It has been just merged. Rebase?

This encapsulates a commonly used button into a reusable component.
@jarolrod jarolrod force-pushed the right-caret-button-states branch from 33b68b3 to 624d985 Compare February 4, 2023 17:49
@jarolrod
Copy link
Contributor Author

jarolrod commented Feb 4, 2023

Updated from 33b68b3 to 624d985 , Compare

Changes: rebased over main

Copy link
Collaborator

@johnny9 johnny9 left a comment

Choose a reason for hiding this comment

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

ACK 624d985

@hebasto hebasto merged commit 0dd350e into bitcoin-core:main Feb 9, 2023
johnny9 pushed a commit to johnny9/bitcoin-core-app that referenced this pull request Jul 4, 2025
…bind color to Setting stateColor

b880f30 qml: use CaretRightButton, bind to Setting state color (jarolrod)
5397e99 qml: add CaretRightButton component (jarolrod)

Pull request description:

  This finishes setting up all current Settings to be able to respond to the Setting state.

  | Filled | Hover | Active |
  | ------ | ----- | ------ |
  | <img width="454" alt="Screen Shot 2023-02-01 at 2 12 02 AM" src="https://user-images.githubusercontent.com/23396902/215975827-b4a3b60c-f19b-483d-94a9-8c2191e48253.png"> | <img width="454" alt="Screen Shot 2023-02-01 at 2 12 11 AM" src="https://user-images.githubusercontent.com/23396902/215975854-d038ee42-4bda-4a87-8e04-bf76ebabb5b5.png"> | <img width="454" alt="Screen Shot 2023-02-01 at 2 12 19 AM" src="https://user-images.githubusercontent.com/23396902/215975871-d6a5b120-e53b-4b00-b5c4-7be1872a9e05.png"> |

  [![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/242)
  [![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/242)
  [![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/242)
  [![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/242)

ACKs for top commit:
  johnny9:
    ACK b880f30

Tree-SHA512: 864312d8f8702ad30050cbb82ff7944e2bcbaedc29999fcf728e625214b59b26320ee600c674703246fb4e852f957b26bf7982716b14419501c14dcd38879e3f
tx-signer450 added a commit to tx-signer450/gui-qml that referenced this pull request Oct 20, 2025
…bind color to Setting stateColor

b880f3081758eaa1d654a702321ca53cb73c4568 qml: use CaretRightButton, bind to Setting state color (jarolrod)
5397e9945ca68c95456bbca1a2302220934a7614 qml: add CaretRightButton component (jarolrod)

Pull request description:

  This finishes setting up all current Settings to be able to respond to the Setting state.

  | Filled | Hover | Active |
  | ------ | ----- | ------ |
  | <img width="454" alt="Screen Shot 2023-02-01 at 2 12 02 AM" src="https://user-images.githubusercontent.com/23396902/215975827-b4a3b60c-f19b-483d-94a9-8c2191e48253.png"> | <img width="454" alt="Screen Shot 2023-02-01 at 2 12 11 AM" src="https://user-images.githubusercontent.com/23396902/215975854-d038ee42-4bda-4a87-8e04-bf76ebabb5b5.png"> | <img width="454" alt="Screen Shot 2023-02-01 at 2 12 19 AM" src="https://user-images.githubusercontent.com/23396902/215975871-d6a5b120-e53b-4b00-b5c4-7be1872a9e05.png"> |

  [![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/242)
  [![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/242)
  [![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/242)
  [![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/242)

ACKs for top commit:
  johnny9:
    ACK b880f3081758eaa1d654a702321ca53cb73c4568

Tree-SHA512: 864312d8f8702ad30050cbb82ff7944e2bcbaedc29999fcf728e625214b59b26320ee600c674703246fb4e852f957b26bf7982716b14419501c14dcd38879e3f
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