Skip to content

Add missing topMargin to Navigation Bar middle detail#334

Merged
hebasto merged 1 commit into
bitcoin-core:mainfrom
jarolrod:middledetail-topmargin
May 28, 2023
Merged

Add missing topMargin to Navigation Bar middle detail#334
hebasto merged 1 commit into
bitcoin-core:mainfrom
jarolrod:middledetail-topmargin

Conversation

@jarolrod
Copy link
Copy Markdown
Contributor

@jarolrod jarolrod commented May 26, 2023

This was missed in #264

master pr
Screen Shot 2023-05-26 at 4 03 33 PM Screen Shot 2023-05-26 at 3 57 54 PM

Windows
Intel macOS
Apple Silicon macOS
ARM64 Android
ARM32 Android

Copy link
Copy Markdown
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 72fe12f

@hebasto hebasto merged commit 960490b into bitcoin-core:main May 28, 2023
johnny9 pushed a commit to johnny9/bitcoin-core-app that referenced this pull request Jul 4, 2025
…ar middle detail

eca46cb qml: add missing topMargin to Navigation Bar middle detail (jarolrod)

Pull request description:

  This was missed in bitcoin-core/gui-qml#264

  | master | pr |
  | ------ | -- |
  | <img width="642" alt="Screen Shot 2023-05-26 at 4 03 33 PM" src="https://github.com/bitcoin-core/gui-qml/assets/23396902/fc1523fd-0742-4beb-abb4-850f72a80fe3"> | <img width="642" alt="Screen Shot 2023-05-26 at 3 57 54 PM" src="https://github.com/bitcoin-core/gui-qml/assets/23396902/d643d02e-4410-4267-9521-5405039b0f53"> |

  [![Windows](https://img.shields.io/badge/OS-Windows-green)](https://api.cirrus-ci.com/v1/artifact/github/bitcoin-core/gui-qml/win64/unsecure_win_gui.zip?branch=pull/334)
  [![Intel macOS](https://img.shields.io/badge/OS-Intel%20macOS-green)](https://api.cirrus-ci.com/v1/artifact/github/bitcoin-core/gui-qml/macos/unsecure_mac_gui.zip?branch=pull/334)
  [![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/unsecure_mac_arm64_gui.zip?branch=pull/334)
  [![ARM64 Android](https://img.shields.io/badge/OS-Android-green)](https://api.cirrus-ci.com/v1/artifact/github/bitcoin-core/gui-qml/android/unsecure_android_apk.zip?branch=pull/334)
  [![ARM32 Android](https://img.shields.io/badge/OS-Android%2032bit-green)](https://api.cirrus-ci.com/v1/artifact/github/bitcoin-core/gui-qml/android32/unsecure_android_32bit_apk.zip?branch=pull/334)

ACKs for top commit:
  johnny9:
    ACK eca46cb

Tree-SHA512: 9d94d0e031160b6dc33cf4f27d79880dfda6482527146ea129ed98460d91b2177b87404d2ac59e4ac728531444f244abd3e38b049a8ab3106db45bf8f7fbebdd
tx-signer450 added a commit to tx-signer450/gui-qml that referenced this pull request Oct 20, 2025
…ar middle detail

eca46cbea63ab68ae7c8bcd962fe9e10d738e5fd qml: add missing topMargin to Navigation Bar middle detail (jarolrod)

Pull request description:

  This was missed in bitcoin-core/gui-qml#264

  | master | pr |
  | ------ | -- |
  | <img width="642" alt="Screen Shot 2023-05-26 at 4 03 33 PM" src="https://github.com/bitcoin-core/gui-qml/assets/23396902/fc1523fd-0742-4beb-abb4-850f72a80fe3"> | <img width="642" alt="Screen Shot 2023-05-26 at 3 57 54 PM" src="https://github.com/bitcoin-core/gui-qml/assets/23396902/d643d02e-4410-4267-9521-5405039b0f53"> |

  [![Windows](https://img.shields.io/badge/OS-Windows-green)](https://api.cirrus-ci.com/v1/artifact/github/bitcoin-core/gui-qml/win64/unsecure_win_gui.zip?branch=pull/334)
  [![Intel macOS](https://img.shields.io/badge/OS-Intel%20macOS-green)](https://api.cirrus-ci.com/v1/artifact/github/bitcoin-core/gui-qml/macos/unsecure_mac_gui.zip?branch=pull/334)
  [![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/unsecure_mac_arm64_gui.zip?branch=pull/334)
  [![ARM64 Android](https://img.shields.io/badge/OS-Android-green)](https://api.cirrus-ci.com/v1/artifact/github/bitcoin-core/gui-qml/android/unsecure_android_apk.zip?branch=pull/334)
  [![ARM32 Android](https://img.shields.io/badge/OS-Android%2032bit-green)](https://api.cirrus-ci.com/v1/artifact/github/bitcoin-core/gui-qml/android32/unsecure_android_32bit_apk.zip?branch=pull/334)

ACKs for top commit:
  johnny9:
    ACK eca46cbea63ab68ae7c8bcd962fe9e10d738e5fd

Tree-SHA512: 9d94d0e031160b6dc33cf4f27d79880dfda6482527146ea129ed98460d91b2177b87404d2ac59e4ac728531444f244abd3e38b049a8ab3106db45bf8f7fbebdd
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