awsaml icon indicating copy to clipboard operation
awsaml copied to clipboard

Add account dropdown to the refresh container

Open asebastian-r7 opened this issue 6 years ago • 3 comments

This PR introduces a minor UI improvement which allows switching accounts from the refresh page. It adds a dropdown button at the top of the refresh page, populated with your accounts. Select your account from the dropdown and you are redirected through select-role and back to the refresh page.

Note that this doesn't actually reduce the amount of clicks you have to perform, it just streamlines the process and - for me - feels a lot smoother. It feels like less of a context switch. This is also the groundwork for switching your role directly from the refresh page. Coming soon.

I preemptively created a release and cask so I can continue to merge to master until concurrent account support is complete.

asebastian-r7 avatar Aug 07 '19 13:08 asebastian-r7

Coverage Status

Coverage remained the same at 90.625% when pulling 18c89dc65ffa23d12bf17b583bb7768f7d0d6c87 on add-account-dropdown into 5a2b43e96edeb10fb3b4578d855a70c1a0854b51 on master.

coveralls avatar Aug 07 '19 15:08 coveralls

I'll take a look as soon as I get the chance. In the meantime do you mind updating the changelog file with the 2.2.0 changes?

dgreene-r7 avatar Aug 08 '19 13:08 dgreene-r7

Updated changelog, thanks for the reminder!

asebastian-r7 avatar Aug 08 '19 14:08 asebastian-r7