BlockchainCommons / GordianSeedTool-iOS

Cryptographic Seed Manager for iOS
Other
36 stars 8 forks source link

Print is Wrong Color Under SSKr #196

Closed shannona closed 11 months ago

shannona commented 12 months ago

Under the SSKR Export, all the options are yellow until you hit "Print All Shares", then the next "Print" button is green.

wolfmcnally commented 11 months ago

This is as expected. The Print button has app accent color: green. This is just like all the other normal buttons like Done and Cancel. The distinction between green and yellow in buttons is made when the button exports potentially sensitive information. In the case of the Print button when printing a seed or SSKR share, the user has already passed that decision point here:

image

Once the user gets to the print preview page, the actual button to print is just the app accent color:

image

shannona commented 11 months ago

I personally think it's an inconsistency, but see your viewpoint.