-
-
Notifications
You must be signed in to change notification settings - Fork 863
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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
Improve page dialog (user) #8224
Improve page dialog (user) #8224
Conversation
Signed-off-by: Diogo Rodrigues <dmfrodrigues2000@gmail.com>
52dac49
to
141a7b3
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A few minor nits.
Co-authored-by: Alexander Borsuk <170263+biodranik@users.noreply.github.com> Signed-off-by: Diogo Rodrigues <dmfrodrigues2000@gmail.com>
Co-authored-by: Alexander Borsuk <170263+biodranik@users.noreply.github.com> Signed-off-by: Diogo Rodrigues <dmfrodrigues2000@gmail.com>
Co-authored-by: Alexander Borsuk <170263+biodranik@users.noreply.github.com> Signed-off-by: Diogo Rodrigues <dmfrodrigues2000@gmail.com>
Co-authored-by: Alexander Borsuk <170263+biodranik@users.noreply.github.com> Signed-off-by: Diogo Rodrigues <dmfrodrigues2000@gmail.com>
Co-authored-by: Alexander Borsuk <170263+biodranik@users.noreply.github.com> Signed-off-by: Diogo Rodrigues <dmfrodrigues2000@gmail.com>
Co-authored-by: Alexander Borsuk <170263+biodranik@users.noreply.github.com> Signed-off-by: Diogo Rodrigues <dmfrodrigues2000@gmail.com>
Signed-off-by: Diogo Rodrigues <dmfrodrigues2000@gmail.com>
3785fd2
to
795ad45
Compare
Signed-off-by: Diogo Rodrigues <dmfrodrigues2000@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! Hope it wasn't a waste of time ;-)
@biodranik Not at all, the implementation is better (less allocations) and is still understandable. Your feedback is welcome 馃檪 |
* Improve page dialog user Signed-off-by: Diogo Rodrigues <dmfrodrigues2000@gmail.com> Co-authored-by: Alexander Borsuk <170263+biodranik@users.noreply.github.com>
Points 2 and 4 are in preparation to move the page dialog to a panel to the left of the map 馃檪 (which I'll do in a separate PR). This will hopefully make the place page less of a nuisance when one specific place is selected but you still want to explore the surroundings, giving a more intuitive experience that is also consistent with the mobile version (where you can select a place, and then drag the map around to see the surroundings).
@biodranik @Ferenc-