Helium314 / SCEE

OpenStreetMap surveyor app for experienced OSM contributors
GNU General Public License v3.0
154 stars 12 forks source link

Add POI in way menu doesn't load text for list properly #656

Closed RubenKelevra closed 2 months ago

RubenKelevra commented 2 months ago

Long pressing on a way will give the option to add a POI into the way itself at this position.

After confirmation of the way, based on its tags you're presented with what looks to be the same menu as the normal POI search list.

However the recent items only load their icons properly, but the text stays empty and the list also claims that "Nothing [was] found".

Clicking on one of the broken items with no text does however work.

Screenshot Screenshot_2024-09-17-06-46-04-490-edit_de westnordost streetcomplete expert

How to Reproduce

Expected Behavior The text should load properly and the list shouldn't display "Nothing found" if nothing was searched for

Does it happen in normal StreetComplete? SC doesn't have this function

Versions affected Android 12 SCEE 58.22

mcliquid commented 2 months ago
  • Select a POI from the list by searching

What did you enter in the search?

RubenKelevra commented 2 months ago

@mcliquid doesn't matter. Just search for anything and used it, to make it appear in the "recently used" list, you see on the screenshot.

I've used bollards and stop signs so far

mnalis commented 2 months ago

Clicking on one of the broken items with no text does however work. The text should load properly

Those icons on the far right are not "regular" search items in the list, those are most-recently-used (or MFU?) shortcuts for popular tags. They are always displayed just with icon without text, and are separate from regular search results. It works like that in Things overlay too (or long-press + Add POI), so I always considered it a feature, and not a bug, and it was quite useful when considering it in that way :smiley:

and the list shouldn't display "Nothing found" if nothing was searched for

Is that a fresh install of SCEE, or have you not used "Insert into way" functionality before? What should in your opinion be shown in the "search results" list if nothing was searched for?

For me, when search box is empty, it shows other most-recently-used-low-priority tags there too (58.2 for this screenshot):

small_Screenshot_20240917_144543_SCEE


So, to summarize my understanding: there are two most-recently-used-tags lists shown in that SCEE dialog:

In your case @RubenKelevra it seems you have any searched for two popular tags (bollards and stop signs), so they only got in the "popular" list (big icons always shown, without text).

If you instead search for non-popular tag (e.g. for "manhole") it will end in the "other" (i.e. non-popular) list (which is only shown if you have not started to type search term yet).

As you keep using SCEE, both lists will get more and more populated, but when you uninstall/clear data, they will start from scratch.

Does that explain the situation @RubenKelevra ?

[^1]: I have not researched how that "popularity" is determined, though

mnalis commented 2 months ago

Such behaviour is perhaps not obvious, but this is "Expert Edition" after all :smile_cat:

Perhaps someone should create SCEE usage instructions for several of less-obvious features on the SCEE wiki subpage. It would be much appreciated by many, I'm sure!

RubenKelevra commented 2 months ago

Ah, yeah that makes sense. Thanks!

I thought you replaced the standard list from SC with an MRU list and moved the icons to the right, but the text is not rendering correctly.

That's the "always staying the same" list in SC:

Screenshot_2024-09-17-16-39-45-184-edit_de.westnordost.streetcomplete.expert.jpg

I think the combination of no round circle behind it and no visual separation between the left search result and the right ribbon style list threw me off.

Maybe add at least one of them? 🤔

mnalis commented 2 months ago

I think the combination of no round circle behind it

I think it depends whether it is SC icon from quests (nice colored ones with round circle), or one acquired via id-tagging-schema (black & white, no circle)

and no visual separation between the left search result and the right ribbon style list

Yes, perhaps a thin vertical line between them would be nice, if there is an easy way to implement it.

mnalis commented 2 months ago

@RubenKelevra I've added vertical separator in https://github.com/Helium314/SCEE/pull/658; would you take a look how you like it?

If you want to test Dev build of SCEE it's available at https://github.com/mnalis/StreetComplete/actions/runs/10908602588 (you can use Dev build in parallel with SC and SCEE release builds, without each affecting the other)

RubenKelevra commented 2 months ago

@mnalis I think this amount of margin around the ribbons would help to make it look more uniform:

Screenshot_2024-09-18-01-08-00-892_de westnordost streetcomplete expert debug~2

mnalis commented 2 months ago

@RubenKelevra ok, updated pics and debug apk in https://github.com/Helium314/SCEE/pull/658. Looks better?

RubenKelevra commented 2 months ago

@mnalis yeah, but I recommend adding a margin on the right as well. Looks a bit odd right now:

Screenshot_2024-09-18-03-33-20-493-edit_de westnordost streetcomplete expert debug

nobkd commented 2 months ago

Also, what about a gap between the icons? They look kinda cramped (at least those with circle background)

RubenKelevra commented 2 months ago

Also, what about a gap between the icons? They look kinda cramped (at least those with circle background)

I'm fine with that. Looked a tad cramped, but you wanna cramp them there to get enough items in the list. So thats fine for my eyes.

If we do put gaps in between we might wanna scale the icons to 2/3 of the current size, to still put enough up there

mnalis commented 2 months ago

Updated pics/apk in https://github.com/Helium314/SCEE/pull/658 to add padding on the right side too. Hopefully that resolves this specific issue; I understand now why Helium314 hates having to deal with UI design :smiling_face_with_tear:

For other UI improvements, I'll leave it to other interested parties. PRs welcome! (hey, I didn't know anything about android layout design before I invested few hours yesterday and today to learn and try to improve this)

RubenKelevra commented 2 months ago

Looks perfect, good work! :)