Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

it would be nice if the back button was on the left side instead of the right #180

Closed
jbibla opened this issue Dec 5, 2017 · 5 comments · Fixed by #450
Closed

it would be nice if the back button was on the left side instead of the right #180

jbibla opened this issue Dec 5, 2017 · 5 comments · Fixed by #450
Assignees

Comments

@jbibla
Copy link
Collaborator

jbibla commented Dec 5, 2017

what do you think @nylira ?

@nylira
Copy link
Contributor

nylira commented Dec 6, 2017

I think that makes sense.

@jbibla
Copy link
Collaborator Author

jbibla commented Jan 23, 2018

where should we put it? 😕

@nylira
Copy link
Contributor

nylira commented Jan 26, 2018

Putting it to the left of the page title screws up the nice layout of the page header area.

I propose adding a text label to the back arrow, make it small, and then putting it on top of the page title. E.g.

<- Validators
FABIAN-THE-VALIDATOR

@jbibla
Copy link
Collaborator Author

jbibla commented Jan 26, 2018

good idea! but i think we'll have to increase the logo spacing / height i think.

@jbibla
Copy link
Collaborator Author

jbibla commented Feb 5, 2018

@nylira my mind has changed about this ticket. i am totally comfortable with the back button in the toolbar on the right. it seems to me that we don't have any routes that are more than one or two deep anyways, so using "back" will be just as easy clicking the route in the sidebar. also, i think the design is so much cleaner without the back button on the left.

all that being said, i will make sure it's on every page and in the same place. i think for now, this should be ok.

👍 😆 🙃

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

Successfully merging a pull request may close this issue.

3 participants