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

Functionality diary button: Access old entries? #171

Closed
zven opened this issue Mar 17, 2022 · 3 comments · Fixed by #176
Closed

Functionality diary button: Access old entries? #171

zven opened this issue Mar 17, 2022 · 3 comments · Fixed by #176
Assignees

Comments

@zven
Copy link
Contributor

zven commented Mar 17, 2022

As @thierna has noted while testing the app (#156): When the user clicks on the diary button, a menu for adding a new entry pops up.
However, the user might expect to have access to their previous diary entries as well. As an alternative, the diary button on the map would open the diary menu.

Before implementing this, we should discuss the implications etc.

@felixerdy felixerdy self-assigned this Mar 21, 2022
@felixerdy
Copy link
Collaborator

felixerdy commented Mar 21, 2022

We coud add an additional button which would open the diary page. I'm not sure about the placement though

Header Block Button Floating Action Button
Bildschirmfoto 2022-03-21 um 10 08 36 Bildschirmfoto 2022-03-21 um 10 12 27 Bildschirmfoto 2022-03-21 um 10 15 24

@zven
Copy link
Contributor Author

zven commented Mar 22, 2022

I would tend to the the "Block Button"-option.
I think the list icon in the header next to the X is a bit too small and the floating action button seems to be a bit out of place. But that's just my opinion. What do you think @thierna?

@thierna
Copy link
Collaborator

thierna commented Mar 22, 2022

I would have thought to open the diary menu but this would mean an additional click to create a new entry. I like the button that says "meine Einträge öffnen" but I would place it at the bottom of the page so that it doesnt distract from creating a new entry. also would make it less blue.

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