ChainSafe / simpleserialize.com

https://simpleserialize.com
MIT License
7 stars 6 forks source link

Netlify: Deserialize and 2 tabs #13

Closed 3xtr4t3rr3str14l closed 4 years ago

3xtr4t3rr3str14l commented 4 years ago

Created a section for deserializing ssz values and separated serialize & deserialize into separate tabs

3xtr4t3rr3str14l commented 4 years ago

Looking really good! Two additional asks:

  • Can you make the widths of the input and output sections consistent between the serialize and deserialize tabs? Perhaps both can have a 50/50 split, like in the deserialize tab.
  • When navigating from deserialize, back to serialize, the input type gets set to ssz. Can you retain the input type that was previously set, rather than automatically going to the ssz input type?

so, my thinking on the widths for the two sections is that you need the current width for the serialize section because of the 3 dropdowns. For example, the ssz type value of "SubscribeToCommitteeSubnetPayload", when selected, wouldn't be fully displayed if you reduced the width. My thinking for the width of the deserialize section was that you need different widths from the serialize sections because the input and output text values are essentially swapped. So, to me, it makes sense to display them as they are. If you have another reason to change the widths though, I'm all ears.

I agree about the input type getting set to ssz though. i'll change that.