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

Patch 1 #6

Closed
wants to merge 29 commits into from
Closed

Patch 1 #6

wants to merge 29 commits into from

Conversation

div72
Copy link
Owner

@div72 div72 commented May 8, 2021

No description provided.

jamescowens and others added 29 commits April 23, 2021 22:40
This sets the application's palette color for links to the shade of blue
from gridcoin-community#847. It improves the visibility of links when using the dark mode.
This sets the "Fusion" Qt theme as the application's base style for
Windows and macOS to normalize some minor appearance issues between
different platforms. Fusion is the default style used for Linux for
desktops without a custom window manager theme.

From what I read, the Windows theme still has some high-DPI issues,
and the Fusion styles support high-DPI displays very well.
Nobody asked for it but I did it anyway.
Don't know if this will have implication on Testnet if the added nodes don't run Testnet.
Added new nodes and removed dead ones where the hostname resolution failed according to https://addnode.cycy.me
…used_rpcs

rpc: Remove deprecated RPC commands
…color

gui: Fix dark theme link text color visibility
…style

gui: Set standard base Qt style on Windows and macOS
1. Changes the (un)select all button to a true select all/select none filp-flop.

2. Implements an input filter function to allow the selection of inputs either less than
or equal to or greater than or equal to a provided input value. This can be used for more
than just consolidations.

3. Implements a "Consolidate" button which limits the selected inputs to 600 to ensure
the consolidation transaction will be successful, presents a pick list to select the destination
address for the consolidation, and then automatically fills in the send transaction for
convenience.
…idate_unspent

gui: Implement GUI version of consolidateunspent (coin control part)
This implements a chain parameter function to get the maximum number of
inputs allowable for a UTXO consolidation transaction with either
the RPC consolidateunspent or the GUI "consolidate" button in coin
control.

The default value for the RPC consolidateunspent function has been
changed to be the same as the upper clamp at the value returned
by GetMaxInputsForConsolidationTxn(). The help returned for
consolidateunspent uses that value as well.
This removes the standard menu bar from the main application layout in
favor of a menu button placed at the top-left corner of the window. It
enables the visual style of the proposed GUI designs which do not have
a menu bar. The menu button does not appear on macOS because the macOS
standard menu bar at the top of the screen already separates the menus
from the main layout.
This replaces the "error messages" field on the overview page with a
section in the status bar to match the proposed GUI design.
This updates the overview page to match the appearance from the proposed
GUI design.

 - Add a header bar with status, magnitude, and available balance
 - Split content sections into cards
 - Change out of sync and researcher action needed labels to badges
 - Change beacon button to an icon
 - Limit the max content width for large window sizes
 - Update colors, layout, sizing, and spacing
This changes the text for the following phrases on the overview page:

 - the "Recent transactions" label to "Recent Transactions"
 - the "out of sync" label to "Out of Sync"
 - the "Current Poll:" label to "Current Polls"

...and updates the localization metadata keys. Because the meanings of the
phrases did not change, this preserves the existing translations for those
entries.
This adds a button next to the logo in the sidebar that switches the
active GUI theme between light and dark mode.
…nsolidateunspent_max_inputs

util, rpc, gui: Implement GetMaxInputsForConsolidationTxn()
This implements a three step wizard that leads the user through
the process of selecting inputs, selecting a destination, and
then reviewing the overall transaction.

Select Inputs:

The select inputs screen uses similar code to that in coincontroldialog
to support the consolidate button there. Pointers to the coincontrol
data structures constructed in sendcoinsdialog are passed into
both coincontrol and the consolidateunspentwizard to faciliate using
the underlying machinery in a unified manner. This is possible because
both coincontrol and consolidateunspendwizard are called with the
sendcoinsdialog and are modal.

Note that there is a stop sign and the next button is disabled if more than
600 outputs are selected. The next button is also disabled if less than 2
outputs are selected (as it makes no sense to consolidate when there is no
consolidation achievable based on the selection). If a filter operation
is applied based on the filter criteria, and that criteria would result in
more than 600 inputs being selected, the selection is reduced to 600 inputs
and a warning triangle is presented.

Select Destination Address:

If all of the inputs selected in the prior page are from one address, then
that address will already be preselected (but allow the opportunity for it
to be changed by the user prior to pressing next). If the inputs selected
correspond to more than one address, an address will NOT be pre-selected,
and the user will be required to pick an address to proceed. The next button
will be disabled until a valid address is selected for the destination.

Confirmation:

The final screen presents the details of the intended transaction for review
by the user. When the "Finish" button is pressed, the transaction to send
is filled in in the send dialog sreen, and the user can review the details
again if desired and press the send button to send.
This corrects the mapping of change to the CoinControlChangeLabel
field.
@div72 div72 closed this May 8, 2021
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