organicmaps/organicmaps-tmp

Archived

Fork 1

In walking or cycling mode, truncates the length of the route to a two-digit number #3320

New issue

Closed

opened 2022-09-02 05:58:45 +00:00 by snooppr · 11 comments

snooppr commented

2022-09-02 05:58:45 +00:00

(Migrated from github.com)

From left to right: bike 12xx km; on walk 11xx km; walk 32x km.

In all cases, it is not known which route is thousands or hundreds of kilometers long and even its accuracy.

I suppose it is connected with the change in the latest version of "height difference" to "climb/descent height".

Phone screen resolution: 720x1440 Android 9.

A workaround is to rotate the screen into the landscape.

![2022-09-02](https://user-images.githubusercontent.com/61022210/188067937-d6d0c80f-df5a-4d15-95e1-a205ec704ffe.jpg) From left to right: bike 12xx km; on walk 11xx km; walk 32x km. In all cases, it is not known which route is thousands or hundreds of kilometers long and even its accuracy. I suppose it is connected with the change in the latest version of "height difference" to "climb/descent height". Phone screen resolution: 720x1440 Android 9. A workaround is to rotate the screen into the landscape.

biodranik commented

2022-09-02 11:16:59 +00:00

(Migrated from github.com)

@ArsenyN can you please check and fix it? It was introduced in #3062

Actually, let's redesign this panel a bit and make it more convenient:

Let's make the graph a full width and place the "Start" button above it.
Let's always show the black font info in one line, and automatically wrap blue ascent/descent under each other, if necessary, so the graph will take as much space as possible on most screens.
Or alternatively, in portrait, let's always display black info in two lines (time, and distance), and ascent/descent difference in two lines too.

@ArsenyN can you please check and fix it? It was introduced in #3062 Actually, let's redesign this panel a bit and make it more convenient: 1. Let's make the graph a full width and place the "Start" button above it. 2. Let's always show the black font info in one line, and automatically wrap blue ascent/descent under each other, if necessary, so the graph will take as much space as possible on most screens. 3. Or alternatively, in portrait, let's always display black info in two lines (time, and distance), and ascent/descent difference in two lines too.

ArsenyN commented

2022-09-02 12:16:30 +00:00

(Migrated from github.com)

@biodranik Will check and fix. But not too fast

👍 1

ArsenyN commented

2022-09-28 13:06:34 +00:00

(Migrated from github.com)

Due personal reasons I have no time for fix this bug now.
Sorry.
Maybe later, or someone else can fix it.

Due personal reasons I have no time for fix this bug now. Sorry. Maybe later, or someone else can fix it.

arnaudvergnet commented

2022-09-30 06:37:43 +00:00

(Migrated from github.com)

While we are refactoring this menu, we should migrate it to a bottom sheet like all the other menus. This would make the code clearer and make it easier to handle lots of data. The user would be able to scroll to display more data.

👍 1

biodranik commented

2022-09-30 21:04:50 +00:00

(Migrated from github.com)

That is a good idea, but let's hotfix it first, many users are complaining.

Another significant improvement would be to display altitudes of the lowest and the highest points on the graph, as some other apps are doing.

That is a good idea, but let's hotfix it first, many users are complaining. Another significant improvement would be to display altitudes of the lowest and the highest points on the graph, as some other apps are doing.

rtsisyk commented

2022-10-12 06:00:00 +00:00

Owner

I am not working on this issue at the moment. Please feel free to take it.

biodranik commented

2022-11-17 13:32:05 +00:00

(Migrated from github.com)

@arnaudvergnet can you please take a look at it? It requires rewriting the layout. The easiest way would be to use the same layout like on iOS:

[time][length][alt up][alt down]  [Start]
[            graph                      ]

With an option to use two lines instead of one if the screen is very narrow:

[time][length]                    [Start]
[alt up][alt down]  
[            graph                      ]

@arnaudvergnet can you please take a look at it? It requires rewriting the layout. The easiest way would be to use the same layout like on iOS: ``` [time][length][alt up][alt down] [Start] [ graph ] ``` With an option to use two lines instead of one if the screen is very narrow: ``` [time][length] [Start] [alt up][alt down] [ graph ] ```

biodranik commented

2023-01-01 19:14:14 +00:00

(Migrated from github.com)

@arnaudvergnet Many users are complaining about this one. It is easily reproducible if a larger font is selected in the device's settings.

arnaudvergnet commented

2023-01-01 20:46:22 +00:00

(Migrated from github.com)

As a quick fix we could disable font scaling while we rework properly the layout. I want to finish migrating the place page to fragment, and then I'll see to migrate the routing layout to fragments while fixing this issue.

👍 2

biodranik commented

2023-01-09 09:07:40 +00:00

(Migrated from github.com)

Another bad example:

Another bad example: ![image](https://user-images.githubusercontent.com/170263/211273087-d0ed1d59-4e2f-4821-b8eb-ba7b3bfef04e.png)

👍 1

Sergey82K commented

2023-01-09 13:46:45 +00:00

(Migrated from github.com)

For example, it is possible to move value of altitude change to the free space at right side under the button.

Or use the "black" and "blue" text in one paragraph section, not a separated cells of table layout.

For example, it is possible to move value of altitude change to the free space at right side under the button. ![msg117404550-22103](https://user-images.githubusercontent.com/19643563/211320992-b8e5fb94-5527-40e0-bded-108280786b05.jpg) Or use the "black" and "blue" text in one paragraph section, not a separated cells of table layout.

This repo is archived. You cannot comment on issues.