BlockchainCommons / GordianSeedTool-iOS

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

FEATURE: Visible size of crypto-seed export while editing seed notes #125

Closed ChristopherA closed 2 years ago

ChristopherA commented 2 years ago

It would be useful if there was a little counter below the notes field that showed dynamically the resulting size of the final crypto-seed (or maybe in dev mode). For instance, there is a limit of 504 bytes for NTAG15 and 888 bytes for NTAG16, and other cheaper ones can be less and more expensive and rare ones can be more. Currently hard to tell if you are backing up.

For instance, my latest version of the 128-byte Seed Public Test Vector (Yimn Blue) 604b93f2 is now 451 bytes so I can test it on NTAG15.

Similarly, we may want to warn when the size of the notes will make QR to transfer it animated, such as "256-byte Seed Public Test Vector () 604b93f2" (below) or too large for even the largest QR.


Medium-sized crypto seed with notes is 451 bytes:

Medium-sized crypto seed with notes

ur:crypto-seed/oxadgdhkwzdtfthptokigtvwnnjsqzcxknsktdaosecyhnemfwrfaxkseceheyetdpidinjycxguihihiecxgdkpidjziniacxghihjkjycxhfihiajyjljpcxdehkinjtjnjtcxfwjzkpihdtcxendyeeideseoiyeyaaksjkghisinjkcxinjkcxjyisihcxeheyetdpidinjycxdeeheycxktjljpiedtcxjkihihiecxkpjkihiecxhsjkcxhscxjokpidjziniacxjpihiojpihjkjkinjljtcxjyihjkjycxkoihiajyjljpcxiyjljpcxgogmjkdmcxgoglgufefxgogmfeclcxgogufecxfggwgmcxghfegughgaglflcxgwglgshkcltdcpfwah


Overly large-sized crypto-seed with notes (256-bit Seed Public Test Vector (Khaki) de584efd) is 1361 bytes, and both can't fit in an NFC and animates when transferred using Gordian Seed Tool Version 1.3.3 (45).

ur:crypto-seed/oxadhdcxvlmdhhtndyfljsrtaxcsmdialbgosrpyvegyguspknrylyskckttgllepeoypebwaosezofptsbkwpgwotwlhnaxksdyeyecendpidinjycxguihihiecxgdkpidjziniacxghihjkjycxhfihiajyjljpcxdegrishsjeindtcxieiheceteeihiyieaakkaoeoghisinjkcxinjkcxjyisihcxeyecendpidinjycxdeeyeecxktjljpiedtcxjkihihiecxkpjkihiecxinjtcxjyisihcxcngujnhsjpjyfxkpjkjyjliekkcxieihjnjljkcxjliycxkohsjpinjlkpjkcxjoiskkjkiniahsjzcxjkihihiecxjkjyjljphsioihcxieihkoiniaihjkdwcxkpjkihiecxinjtcxkohsjpinjlkpjkcxjljyisihjpcxjokpidjziniacxieihjnjljtjkjyjphsjyinjljtjkcxjliycxjkihihiecxjkjyjljphsioihcxidihjkjycxjojphsiajyiniaihjkdwcxhsjtiecxinjkcxiakpjpjpihjtjyjzkkcxkpjkihiecxhsjkcxhscxjokpidjziniacxjpihiojpihjkjkinjljtcxjyihjkjycxkoihiajyjljpcxiyjljpcxgogmjkdwcxfljljpieinhsjtcxguihihiecxghjljljzdwcxhsjtiecxjkihihiejyjljljzdpiajzindmbkbkghisinjkcxjkihihiecxhsjtiecxhsjzjzcxjeihkkjkcxioihjtihjphsjyihiecxiyjpjljncxinjycxjkisjlkpjziecxidihcxiajljtjkinieihjpihiecxkpjtjkihiakpjpihcxhsjkcxjyisinjkcxjeihkkcxjnhsjyihjpinhsjzcxinjkcxidihinjtiocxjkishsjpihiecxjokpidjziniahsjzjzkkcxhsjkcxhscxjyihjkjycxkoihiajyjljpdmbkbkfwinjyiajlinjtcxgthsjkjyihjpcxgrihkkcxfginjtioihjpjojpinjtjyftcxendyeeideseoiyeybkfejyisihjpihkpjncxfpiaiajlkpjtjycxcndycxfpieiejpihjkjkcxhpjndleeeedidlendyvolanldldyvolanldldydldyhlftcxdyksiydyhsfpemeyecehfpdydyecfxesehidhseyeyeoideniaeceeieeceeeoeyecendyeyehfgeyehfpenbkcnihroke

wolfmcnally commented 2 years ago

The next release (51) has this:

image

wolfmcnally commented 2 years ago

Please re-test.

shannona commented 2 years ago

Length, QR size, and warnings dynamically update as you type. Very nice.