[steem keychain] sender in the confirmation page / enable enter key for confirmation

in #utopian-io5 years ago (edited)

Repository

https://github.com/MattyIce/steem-keychain/

Note: For the comment style, I followed PO's style.

Two new features

  • add sender in the transfer confirmation page
  • enable enter key for adding account and key

New Feature - add sender in the transfer confirmation page

PR: https://github.com/MattyIce/steem-keychain/pull/97

Steem Keychain supports multiple accounts which I really like. But regarding the transfer, the confirmation page didn't show the sender information as follows:

Currently, no sender is shown.

Thus, a user may choose a wrong sender account by mistake.

And as it says, it can't be undone.

I changed the confirmation page so that it shows sender as follows:

Code

  • The current account name is stored in active_account.name Using this, "From" field has been inserted appropriately.

Benefits

Any transfer can't be cancelled. However, people often tend to make a mistake. Considering the fact that the Keychain supports multiple accounts, this feature is a must to prevent those mistakes.

New Feature - enable enter key for adding account and key

PR: https://github.com/MattyIce/steem-keychain/pull/104 (enable enter key)

Previously, enter key was working (for clicking the confirm button) only for unlocking and registration, and it didn't work for adding account and key, which was inconvenient and inconsistent.

Now enter key is supported in all (if I didn't forget) similar contexts.

enable enter key for adding account

enable enter key for adding key

Code

  • When the enter key is clicked, a desired confirm button should be clicked.

GitHub Account

https://github.com/economicstudio

ps. Thanks to @yabapmatt, @stoodkev, @aggroed for reviewing and merging and running the awesome keychain.

Sort:  
  • Great looking post, lots of details. Includes Images, code samples and explanations.
  • Great commit comments as well.

Your contribution has been evaluated according to Utopian policies and guidelines, as well as a predefined set of questions pertaining to the category.

To view those questions and the relevant answers related to your post, click here.


Need help? Chat with us on Discord.

[utopian-moderator]

Hi @helo, thanks for your review!

Thank you for your review, @helo! Keep up the good work!

Korean: Keychain에 두가지 작업했던 것 모아서 올립니다.

  1. 이미 소개드린 송금시 보내는 사람 보여줘서 다계정 사용시 실수 방지
  2. key를 추가할때 마우스 클릭대신 enter key가 안되는 곳들이 있더군요. 제 개인적으로 이런 걸 무지 불편해해서 엔터키 동작하게 수정했습니다. => 이건 아직 배포하진 않은 것 같습니다. 다른 수정사항 있을때 함께 내보낼 계획인듯.

@surpassinggoogle팀에서 개발자는 찾는다네요. 여기도 지난번에 유토피안 리뷰어가 저를 알려주는바람에 계속 개발 태스크 생길때 마다 연락이 오는데 이번에 좀 더 정식으로 보수를 지급하고 개발자를 찾고 있다합니다: Looking For A Dedicated Developer/Developers For A Duration Of 1 Month, For Ulogs & Steemgigs Alpha-Stage Development Tasks. (Budget - 1500 STEEM, 20,000 TEARDROPS TOKENS & Prospective Utopian-io Curation) 대충 의사소통될 정도의 영어면 충분할 것 같은데 관심있는 분들 지원해보세요. 보수가 스팀과 자체 토큰(?)으로 주는 것 같습니다. 사실 큰 보상 규모는 아닐 겁니다.

어그로드도 엊그제 다시 연락왔습니다. 오픈소스로 해둘테니 좀 작업해줄 수 있냐고. 여기야말로 개발자가 엄청 부족할텐데 영어 기본 소통만 가능하신 분 계시면 팀에 조인해보시는 건 어떨까요? 아 물론 저는 지난번 말씀드린 것처럼 오픈소스로 가끔 취미 삼아 하는것외에 정식 소속되어 할 계획은 전혀 없습니다. 관심있으신분 있으면 연결시켜드리겠습니다. 제느낌상 여긴 영어가 어느정도는 되야할거에요. 워낙 빠르게 변하고 이것저것 시도하는 팀이라.

Hi @surpassinggoogle,

Sure, I'll introduce it again next time. By the way, what about english skills?

unfortunately, I think there are not many in kr who are both good at programming and english.

이제 엔터키만 입력하면 되네요. ㅎㅎ 감사합니다.

워낙 단순한 수정이라 아직 배포하진 않은 것 같아요. 몇개씩 모아서 보내는 것 같아요^^ 만들고 계신것도 기대하고 있겠습니다. 아직 공개는 안한걸로 아는데 오픈소스로 하실 계획인가요?

확실한 목표를 가지고 만들고 있는게 아직 없습니다. 그래서 어떤걸 기대하는지는 잘모르겠네요.ㅋ
헤피베리보이님과 함께 작업하고 있는 크롬 익스텐션은 이미 깃헙에 소스가 공개되어있습니다.

곰돌이가 @anpigon님의 소중한 댓글에 $0.010을 보팅해서 $0.013을 살려드리고 가요. 곰돌이가 지금까지 총 4672번 $52.774을 보팅해서 $59.365을 구했습니다. @gomdory 곰도뤼~

zorba님이 blockchainstudio님을 멘션하셨습니당. 아래 링크를 누르시면 연결되용~ ^^
zorba님의 [2019/5/15] 가장 빠른 해외 소식! 해외 스티미언 소모임 회원들의 글을 소개해드립니다.

...an님, 태국에서 jisoooh0202님, 미국 캘리포니아에서 livelyshawnee님, 영국에서 blockchainstudio gomdory님, 일본에서 sizuko님, 프랑스에서 eric66님, laylador님, 네덜란드에...

Hi @blockchainstudio!

Your post was upvoted by @steem-ua, new Steem dApp, using UserAuthority for algorithmic post curation!
Your post is eligible for our upvote, thanks to our collaboration with @utopian-io!
Feel free to join our @steem-ua Discord server

Hey, @blockchainstudio!

Thanks for contributing on Utopian.
We’re already looking forward to your next contribution!

Get higher incentives and support Utopian.io!
Simply set @utopian.pay as a 5% (or higher) payout beneficiary on your contribution post (via SteemPlus or Steeditor).

Want to chat? Join us on Discord https://discord.gg/h52nFrV.

Vote for Utopian Witness!

Coin Marketplace

STEEM 0.17
TRX 0.13
JST 0.027
BTC 60812.47
ETH 2629.76
USDT 1.00
SBD 2.62