Skip to content
This repository has been archived by the owner on Apr 15, 2019. It is now read-only.

Forging tab should be always visible #218

Closed
karmacoma opened this issue May 15, 2017 · 4 comments
Closed

Forging tab should be always visible #218

karmacoma opened this issue May 15, 2017 · 4 comments

Comments

@karmacoma
Copy link
Contributor

Expected behaviour

User should be able to click on a visible "Forging" tab and then: "Register as a delegate".

Actual behaviour

No forging tab is displayed.

Secondly, by navigating to the route /#/main/forging for a non-delegate, the user is presented with a disabled button: "Become Delegate". Therefore a dead end.

image

Steps to reproduce

Log in as a non-delegate.

@slaweet
Copy link
Contributor

slaweet commented May 16, 2017

Forging tab always visible was the case when I first implemented it. But it was requested (by @karmacoma or @MaxKK) to make it NOT visible if not delegate.

I'm for keeping it NOT always visible because of UX. Delegates should be by definition just a small percentage of users and for the rest, it just hinders the UI.

The only think that IMO needs to be solved is that /#/main/forging should redirect to /#/main/transactions if not delegate account.

@karmacoma
Copy link
Contributor Author

@slaweet During testing I found the "Forging" tab popping up out of nowhere largely invisible to new users after registering as a delegate.

Regardless, the routing issue needs to be resolved, the dead end message I attached should be replaced with a redirect at the very least.

@reyraa
Copy link
Contributor

reyraa commented May 16, 2017

I would suggest,
The template:
We can add a message informing about the reason for the redirection instead.
The tab:
We can show it only after the delegate registration transaction is confirmed.

@maxkordek
Copy link

I agree to @slaweet, it doesn't need to be shown if the account is not a delegate.

Later on, we need to think about a way how to introduce changes to the UI. Maybe a slick animation, instead of just a random popup. For now, I would say no need to change in regards to the animation. There is more important things to do.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants