Skip to content

Support detail item inside option button#103

Merged
hebasto merged 2 commits into
bitcoin-core:mainfrom
promag:2022-01-qml-optionbutton-detail
Jan 6, 2022
Merged

Support detail item inside option button#103
hebasto merged 2 commits into
bitcoin-core:mainfrom
promag:2022-01-qml-optionbutton-detail

Conversation

@promag
Copy link
Copy Markdown
Contributor

@promag promag commented Jan 5, 2022

Allow a custom inlined item on the right-hand side of the OptionButton:

Screen.Recording.2022-01-05.at.01.08.34.mov

Links for Windows, macOS, and Android build artifacts.

Windows
macOS
Android

@promag
Copy link
Copy Markdown
Contributor Author

promag commented Jan 5, 2022

Best reviewed without whitespace changes.

@hebasto
Copy link
Copy Markdown
Member

hebasto commented Jan 5, 2022

Allow a custom inlined item on the right-hand side of the OptionButton:

Could there be an issue with RTL writing systems?

@promag
Copy link
Copy Markdown
Contributor Author

promag commented Jan 5, 2022

RTL writing systems

I don't think so. Maybe we should raise that concern in the next meeting with designers.

@promag promag force-pushed the 2022-01-qml-optionbutton-detail branch from 1f5cd07 to 898d672 Compare January 5, 2022 19:36
Copy link
Copy Markdown
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.

tACK 898d672

nice! cleaner implementation than the relevant changes in 5483b12

@jarolrod
Copy link
Copy Markdown
Contributor

jarolrod commented Jan 6, 2022

ni: misspelled word in 347f772 commit: s/suport/support

@promag promag force-pushed the 2022-01-qml-optionbutton-detail branch from 898d672 to a55553c Compare January 6, 2022 02:07
@jarolrod jarolrod mentioned this pull request Jan 6, 2022
@GBKS
Copy link
Copy Markdown
Contributor

GBKS commented Jan 6, 2022

@hebasto I am not sure what (if anything) is required for RTL writing systems for this type of component. I'll put it on my list and do a bit of research.

Copy link
Copy Markdown
Member

@hebasto hebasto left a comment

Choose a reason for hiding this comment

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

ACK a55553c

@hebasto hebasto merged commit bb64686 into bitcoin-core:main Jan 6, 2022
@promag promag deleted the 2022-01-qml-optionbutton-detail branch January 6, 2022 11:28
hebasto pushed a commit to hebasto/gui-qml that referenced this pull request Jun 7, 2025
hebasto pushed a commit to hebasto/gui-qml that referenced this pull request Jun 7, 2025
hebasto pushed a commit to hebasto/gui-qml that referenced this pull request Jun 8, 2025
hebasto pushed a commit to hebasto/gui-qml that referenced this pull request Jun 8, 2025
hebasto pushed a commit to hebasto/gui-qml that referenced this pull request Jun 8, 2025
hebasto pushed a commit to hebasto/gui-qml that referenced this pull request Jun 8, 2025
hebasto pushed a commit to hebasto/gui-qml that referenced this pull request Jun 9, 2025
hebasto pushed a commit to hebasto/gui-qml that referenced this pull request Jun 9, 2025
hebasto pushed a commit to hebasto/gui-qml that referenced this pull request Jun 9, 2025
hebasto pushed a commit to hebasto/gui-qml that referenced this pull request Jun 9, 2025
johnny9 pushed a commit to johnny9/bitcoin-core-app that referenced this pull request Jul 4, 2025
fde8465 qml: Demo OptionButton detail in wifi-only option (João Barbosa)
5f3f324 qml: Support detail item inside OptionButton (João Barbosa)

Pull request description:

  Allow a custom inlined item on the right-hand side of the `OptionButton`:

  https://user-images.githubusercontent.com/3534524/148144944-165f657a-e918-47ab-92f7-5641faf095f8.mov

  Links for Windows, macOS, and Android build artifacts.

  [![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/103)
  [![macOS](https://img.shields.io/badge/OS-macOS-green)](https://api.cirrus-ci.com/v1/artifact/github/bitcoin-core/gui-qml/macos/insecure_mac_gui.zip?branch=pull/103)
  [![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/103)

ACKs for top commit:
  hebasto:
    ACK fde8465

Tree-SHA512: 58fbea8f5eed58e227ef65f46b03133ec847d329acd3ff2ebd066f5659411608e63b061eff216863125368faf04fa4a3baf01b46dfbbf9e8ed0e39840141e081
tx-signer450 added a commit to tx-signer450/gui-qml that referenced this pull request Oct 20, 2025
fde84650d6f3842c81a24f4414103117f8b831b8 qml: Demo OptionButton detail in wifi-only option (João Barbosa)
5f3f324edbbc63f505dc38362b51b748275987f7 qml: Support detail item inside OptionButton (João Barbosa)

Pull request description:

  Allow a custom inlined item on the right-hand side of the `OptionButton`:

  https://user-images.githubusercontent.com/3534524/148144944-165f657a-e918-47ab-92f7-5641faf095f8.mov

  Links for Windows, macOS, and Android build artifacts.

  [![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/103)
  [![macOS](https://img.shields.io/badge/OS-macOS-green)](https://api.cirrus-ci.com/v1/artifact/github/bitcoin-core/gui-qml/macos/insecure_mac_gui.zip?branch=pull/103)
  [![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/103)

ACKs for top commit:
  hebasto:
    ACK fde84650d6f3842c81a24f4414103117f8b831b8

Tree-SHA512: 58fbea8f5eed58e227ef65f46b03133ec847d329acd3ff2ebd066f5659411608e63b061eff216863125368faf04fa4a3baf01b46dfbbf9e8ed0e39840141e081
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.

4 participants