Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

UPnP #113

Closed
wants to merge 1 commit into from
Closed

UPnP #113

wants to merge 1 commit into from

Conversation

TheBlueMatt
Copy link
Contributor

Finally a seemingly proper patch for UPnP Port Mapping.
Note that this one still does not have proper support in the makefiles for anything but UNIX.

Thanks to Luke Dashjr (luke-jr) for the help with the makefile.
@gavinandresen
Copy link
Contributor

Is github broken? The diff/commit looks completely messed up to me...

@TheBlueMatt
Copy link
Contributor Author

Oh goddamit, well after today I'm done trying to submit patches.

glv2 referenced this pull request in glv2/peercoin Sep 7, 2014
translation updates from getlocalization.com/peerunity
dexX7 added a commit to dexX7/bitcoin that referenced this pull request Jul 7, 2015
ee0bb5a RPC: Categorize calls (zathras-crypto)
3ad578b RPC: Standardize calls on omni_abcdefghij (zathras-crypto)
gavinandresen pushed a commit to gavinandresen/bitcoin-git that referenced this pull request Mar 1, 2016
Update README.md link to be relative.
lateminer pushed a commit to lateminer/bitcoin that referenced this pull request Feb 23, 2019
Fix accidental reverts from UI merge
jonasschnelli added a commit that referenced this pull request Oct 27, 2020
… window without loaded wallets

7b2e42e qt: Add WalletFrame::sizeHint (Hennadii Stepanov)

Pull request description:

  This PR fixes a bug in master (d67883d) and in 0.20.1 that could be easily reproduced with
  ```
  $ src/qt/bitcoin-qt -regtest -resetguisettings -nowallet
  ```

  ![Screenshot from 2020-10-25 21-21-27](https://user-images.githubusercontent.com/32963518/97117179-b1800100-170a-11eb-87c9-3120d39b9455.png)
  ![Screenshot from 2020-10-25 21-23-32](https://user-images.githubusercontent.com/32963518/97117186-b644b500-170a-11eb-8b5d-234ff7205003.png)

  **With this PR:**

  ![Screenshot from 2020-10-25 21-20-35](https://user-images.githubusercontent.com/32963518/97117226-f441d900-170a-11eb-8d66-98b7718a2bb1.png)
  ![Screenshot from 2020-10-25 21-23-03](https://user-images.githubusercontent.com/32963518/97117232-f99f2380-170a-11eb-85ed-c7b5ece926b2.png)

  ---

  Fix #104
  Fix #113

  This PR is an alternative to #107 without [hard-coding a size in pixels](bitcoin-core/gui#107 (comment)).

ACKs for top commit:
  jonasschnelli:
    Tested ACK 7b2e42e - I can confirm this fixes #104 (Ubuntu 20.04 - HiDPI 200%).

Tree-SHA512: eb0692dbeb3befdeecca0e41534c9783eab6637c14cc4f170ee42619235884f9354f8d22a10c20c08cc89dc5340a60b7dfa2523c12e64b3386b3fd2c6d5f934e
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Sep 8, 2021
This pull request was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants