diff --git a/Makefile b/Makefile index a5ef3a20..8eabb82d 100644 --- a/Makefile +++ b/Makefile @@ -58,7 +58,9 @@ pre-build-tests-fast: check-for-non-ascii-urls check-for-wrong-filename-assignme check-for-missing-copyright-licenses \ check-bundle \ check-for-english-in-en-dir \ - check-for-consistent-bitcoin-core-titles + check-for-consistent-bitcoin-core-titles \ + check-for-too-many-wallets-on-one-platform + ## Post-build tests which, aggregated together, take less than 10 seconds to run on a typical PC post-build-tests-fast: check-for-build-errors ensure-each-svg-has-a-png check-for-liquid-errors \ @@ -259,7 +261,7 @@ check-bundle: ## Ensure all the dependencies are installed. If you build without this ## check, you'll get confusing error messages when your deps aren't up ## to date - $S ! bundle check | grep -v "The Gemfile's dependencies are satisfied" + $S ! bundle check | egrep -v "(Resolving dependencies...|The Gemfile's dependencies are satisfied)" travis-background-keepalive: $S { while ps aux | grep -q '[m]ake' ; do echo "Ignore me: Travis CI keep alive" ; sleep 1m ; done ; } & @@ -284,3 +286,10 @@ check-for-consistent-bitcoin-core-titles: ## Ensure all page titles in the en/bitcoin-core/ hierarchy mention ## Bitcoin Core $S grep -r -L '^title:.*Bitcoin Core' en/bitcoin-core/ | eval $(ERROR_ON_OUTPUT) + +check-for-too-many-wallets-on-one-platform: + $S for platform in desktop windows mac linux mobile android ios blackberry windowsphone web hardware \ + ; do count=$$( grep -c "compat:.*$$platform" _templates/choose-your-wallet.html ) \ + ; if [ $$count -gt 14 ] \ + ; then echo "ERROR: too many wallets in $$platform platform. Remove one or change layout" \ + ; fi ; done diff --git a/_less/screen.less b/_less/screen.less index 56dcd478..d20802a8 100644 --- a/_less/screen.less +++ b/_less/screen.less @@ -1694,7 +1694,7 @@ button.active { } .wallets{ - width:605px; + width:705px; height:250px; text-align:left; position:relative; @@ -1714,7 +1714,7 @@ button.active { vertical-align:top; font-size:16px; } -.wallets>div:nth-child(1n+13){ +.wallets>div:nth-child(1n+15){ display:none; } .wallets>div>a{ diff --git a/_templates/choose-your-wallet.html b/_templates/choose-your-wallet.html index 5ea641c9..cf87581b 100755 --- a/_templates/choose-your-wallet.html +++ b/_templates/choose-your-wallet.html @@ -116,7 +116,7 @@ wallets: - electrum: title: "Electrum" titleshort: "Electrum" - compat: "desktop windows mac linux" + compat: "desktop windows mac linux mobile android" level: 2 platform: desktop: @@ -139,7 +139,7 @@ wallets: privacyaddressreuse: "checkpassprivacyaddressrotation" privacydisclosure: "checkfailprivacydisclosurecentralized" privacynetwork: "checkpassprivacynetworksupporttorproxy" - android: + mobile: text: "walletelectrum" link: "https://play.google.com/store/apps/details?id=org.electrum.electrum" source: "https://github.com/spesmilo/electrum" @@ -803,7 +803,7 @@ wallets: - coinspace: title: "Coin.Space" titleshort: "Coin.Space" - compat: "mobile web android windowsphone" + compat: "mobile web android windowsphone ios" level: 3 platform: mobile: @@ -1215,7 +1215,7 @@ wallets:

{{ wallet[1].title }}

-
{% for os in platform.os %}{% translate platform{{os}} %}{% endfor %}
+
{% for os in platform.os %}{% translate platform{{os}} %}{% unless wallet[1].compat contains os %}{% die platform not listed in compat field %}{% endunless %}{% endfor %}
{% if platform.link == 'bitcoincore' %}{% translate walletdownload %}{% elsif platform.link contains 'play.google.com' or platform.link contains 'itunes.apple.com' or platform.link contains 'chrome.google.com' %}{% translate walletdownload %}{% else %}{% translate walletvisit %}{% endif %}{% if platform.source %}{% translate walletsourcecode %}{% endif %}
{% for check in platform.check %}{% if check[0] == 'privacy' and platform.privacycheck %}